Merge ~morphis/snappy-hwe-snaps/+git/build-scripts:bug/check-deb-install-status into ~snappy-hwe-team/snappy-hwe-snaps/+git/build-scripts:master
Proposed by
Simon Fels
Status: | Merged |
---|---|
Approved by: | Jim Hodapp |
Approved revision: | b1c057b794d54f10dc416d872247e507cf2a2b9a |
Merged at revision: | 776581b6f9fd5fb4f5ab15a994f4a082915a8ab6 |
Proposed branch: | ~morphis/snappy-hwe-snaps/+git/build-scripts:bug/check-deb-install-status |
Merge into: | ~snappy-hwe-team/snappy-hwe-snaps/+git/build-scripts:master |
Diff against target: |
24 lines (+12/-1) 1 file modified
jobs/generic-run-snap-spread-tests (+12/-1) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jim Hodapp (community) | Approve | ||
Alfonso Sanchez-Beato | Approve | ||
Konrad Zapałowicz (community) | code | Approve | |
System Enablement Bot | continuous-integration | Approve | |
Review via email: mp+316937@code.launchpad.net |
Description of the change
Check deb package install status before trying to install again
This removes the chance of having a conflict with an already locked deb database when we run the job multiple times. For the last tiny tiny chance of having a conflict we simply ignore the failing apt install call and let the build fail later on when it attempts to use one of the missing tools.
To post a comment you must log in.
PASSED: Continuous integration, rev:b1c057b794d 54f10dc416d8722 47e507cf2a2b9a /jenkins. canonical. com/system- enablement/ job/generic- build-snap/ 889/ /jenkins. canonical. com/system- enablement/ job/generic- run-snap- spread- tests/595 /jenkins. canonical. com/system- enablement/ job/generic- update- snap-mp/ 795/console
https:/
Executed test runs:
SUCCESS: https:/
None: https:/
Click here to trigger a rebuild: /jenkins. canonical. com/system- enablement/ job/generic- build-snap/ 889/rebuild
https:/