threefoldtecharchive / 0-robot

Distributed live cycle management system
Apache License 2.0
0 stars 0 forks source link

provide API to get other robot clients #45

Closed zaibon closed 6 years ago

zaibon commented 6 years ago

This api provide an unified way to get othe robot client from within a template. It uses a queue to serialize the access to the low level configuration to ensure that we don't have data race if a template tries to modify the robot client configuration

All the templates that was dealing with robot client manually will have to be updated to use this api

codecov-io commented 6 years ago

Codecov Report

Merging #45 into development will increase coverage by <1%. The diff coverage is 83%.

Impacted file tree graph

@@             Coverage Diff             @@
##           development    #45    +/-   ##
===========================================
+ Coverage           71%    72%   +<1%     
===========================================
  Files              154    155     +1     
  Lines             6495   6522    +27     
===========================================
+ Hits              4662   4708    +46     
+ Misses            1833   1814    -19
Impacted Files Coverage Δ
zerorobot/template/base.py 96% <100%> (ø) :arrow_up:
zerorobot/dsl/ZeroRobotAPI.py 92% <100%> (+24%) :arrow_up:
zerorobot/dsl/__init__.py 100% <100%> (ø)
...pscaleZrobot/clients/zerorobot/ZeroRobotFactory.py 62% <100%> (+10%) :arrow_up:
zerorobot/service_proxy.py 82% <50%> (+8%) :arrow_up:
zerorobot/dsl/ZeroRobotManager.py 78% <58%> (-9%) :arrow_down:
tests/test_zrobot_api.py 91% <76%> (-3%) :arrow_down:
zerorobot/dsl/config_mgr.py 88% <88%> (ø)
...caleZrobot/clients/zerorobot/client/http_client.py 85% <0%> (-7%) :arrow_down:
zerorobot/server/auth/auth.py 60% <0%> (-6%) :arrow_down:
... and 12 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update db8fe76...6889a5e. Read the comment docs.