Merge ~raharper/curtin:fix/uefi-mount-check-fails-no-path into curtin:master
Proposed by
Ryan Harper
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Paride Legovini | ||||
Approved revision: | 23e25643fa8aae59b4b5315afabfa6119d4e8e8e | ||||
Merge reported by: | Server Team CI bot | ||||
Merged at revision: | not available | ||||
Proposed branch: | ~raharper/curtin:fix/uefi-mount-check-fails-no-path | ||||
Merge into: | curtin:master | ||||
Diff against target: |
30 lines (+7/-1) 2 files modified
curtin/commands/curthooks.py (+1/-1) tests/unittests/test_curthooks.py (+6/-0) |
||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Paride Legovini | Approve | ||
Server Team CI bot | continuous-integration | Approve | |
Review via email: mp+389571@code.launchpad.net |
Commit message
curthooks: uefi_find_
When scanning a curtin storage config for efi mount entries we failed
to handle the case where an entry does not contain the 'path' key.
Curtin storage config type:mount does not require a 'path' element.
Change to using .get('path') when looking for efi mount path.
LP: #1892242
To post a comment you must log in.
FAILED: Continuous integration, rev:23e25643fa8 aae59b4b5315afa bfa6119d4e8e8e /jenkins. ubuntu. com/server/ job/curtin- ci/170/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-amd64/ 170/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-arm64/ 170/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-ppc64el/ 170/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= metal-s390x/ 170/
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
FAILURE: https:/
SUCCESS: https:/
Click here to trigger a rebuild: /jenkins. ubuntu. com/server/ job/curtin- ci/170/ /rebuild
https:/