search

facebookarchive / contest

Run continuous and on-demand system testing for real and virtual hardware
MIT License
32 stars 15 forks source link

Add target manager state to the target #272

Closed rojer9-fb closed 3 years ago

rojer9-fb commented 3 years ago

It's state that target manager may need to store to facilitate release of the target across server restarts.

Signed-off-by: Deomid "rojer" Ryabkov rojer9@fb.com

codecov[bot] commented 3 years ago

Codecov Report

Merging #272 (86baade) into master (ecf9e96) will increase coverage by 0.02%. The diff coverage is 68.96%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #272      +/-   ##
==========================================
+ Coverage   66.29%   66.32%   +0.02%     
==========================================
  Files         125      126       +1     
  Lines        5999     6025      +26     
==========================================
+ Hits         3977     3996      +19     
- Misses       1553     1557       +4     
- Partials      469      472       +3
Flag Coverage Δ
e2e 45.32% <68.96%> (+0.10%) :arrow_up:
integration 59.91% <50.00%> (+<0.01%) :arrow_up:
unittests 43.49% <100.00%> (+0.20%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
cmds/plugins/plugins.go 9.09% <ø> (ø)
...ins/targetlist_with_state/targetlist_with_state.go 64.00% <64.00%> (ø)
pkg/target/target.go 60.86% <100.00%> (+3.72%) :arrow_up:
plugins/targetmanagers/targetlist/targetlist.go 50.00% <100.00%> (-2.18%) :arrow_down:
pkg/runner/job_runner.go 64.50% <0.00%> (-1.30%) :arrow_down:
pkg/runner/test_runner.go 92.72% <0.00%> (+1.17%) :arrow_up:

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 ecf9e96...86baade. Read the comment docs.