Merge lp:~cypressyew/checkbox/dependency-fix into lp:checkbox
Proposed by
Po-Hsu Lin
Status: | Merged |
---|---|
Approved by: | Zygmunt Krynicki |
Approved revision: | 3707 |
Merged at revision: | 3707 |
Proposed branch: | lp:~cypressyew/checkbox/dependency-fix |
Merge into: | lp:checkbox |
Diff against target: |
25 lines (+2/-2) 2 files modified
providers/plainbox-provider-checkbox/jobs/esata.txt.in (+1/-1) providers/plainbox-provider-checkbox/jobs/firewire.txt.in (+1/-1) |
To merge this branch: | bzr merge lp:~cypressyew/checkbox/dependency-fix |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Zygmunt Krynicki (community) | Approve | ||
Po-Hsu Lin | Needs Resubmitting | ||
Review via email: mp+256641@code.launchpad.net |
Description of the change
Change the dependency for esata / firewire remove job from storage job to insert job
To post a comment you must log in.
Resubmit with more information in the commit message.
The reason for this change is that if the storage test failed, the removal test will be skipped automatically. For the tool we have now, we're unable to re-run those automatically skipped tests.
Also, we could still do the removal test even if it failed with the storage test.