Merge ~mwhudson/curtin:swap-exit into curtin:master
Proposed by
Michael Hudson-Doyle
Status: | Merged |
---|---|
Approved by: | Dan Bungert |
Approved revision: | 9ccf75c478e96802da41c2e4392c093bff7e2474 |
Merge reported by: | Server Team CI bot |
Merged at revision: | not available |
Proposed branch: | ~mwhudson/curtin:swap-exit |
Merge into: | curtin:master |
Diff against target: |
13 lines (+1/-1) 1 file modified
curtin/commands/swap.py (+1/-1) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Dan Bungert | Approve | ||
Server Team CI bot | continuous-integration | Approve | |
Review via email: mp+433683@code.launchpad.net |
Commit message
fix "curtin swap" exit code
Today I discovered that there is a "curtin swap" command that just does
the "configure swap" part of curthooks and then I fairly quickly after
that discovered that probably noone has used it because it
unconditionally exits with code 2 even if everything succeeded.
To post a comment you must log in.
PASSED: Continuous integration, rev:9ccf75c478e 96802da41c2e439 2c093bff7e2474 /jenkins. canonical. com/server- team/job/ curtin- ci/57/ /jenkins. canonical. com/server- team/job/ curtin- ci/nodes= metal-amd64/ 57/ /jenkins. canonical. com/server- team/job/ curtin- ci/nodes= metal-arm64/ 57/ /jenkins. canonical. com/server- team/job/ curtin- ci/nodes= metal-ppc64el/ 57/ /jenkins. canonical. com/server- team/job/ curtin- ci/nodes= metal-s390x/ 57/
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
Click here to trigger a rebuild: /jenkins. canonical. com/server- team/job/ curtin- ci/57// rebuild
https:/