By: Tom Sydney Kerckhove <syd@cs-syd.eu>
Address the review on the pull request A chassis and an export are not optional: a check that quietly does not run is worse than one that was never written, so checkedFor requires both and says which router is missing which. The asserts move here from deployFor, since everything that deploys goes through this. The comments the review pointed at are gone. The one on vmtest-for's chassis argument had also gone stale: a chassis is five things now, not the two it claimed.
Total
8s
Created Started Finished
| Time to Start | Worker time | Duration | Time to finish | Idle | |
|---|---|---|---|---|---|
| Config | 0s | 1s | 1s | 1s | 0s |
| Eval | 1s | 6s | 6s | 8s | 0s |
| Build | 7s | 0s | 0s | 7s | 0s |
| Suite | 0s | 8s | 8s | 8s | 0s |