Merge ~ogayot/curtin:recovery-key into curtin:master
Proposed by
Olivier Gayot
Status: | Merged |
---|---|
Approved by: | Olivier Gayot |
Approved revision: | 2bfebabc9aa8be019b05d9c2eb0077f83f1a96de |
Merge reported by: | Server Team CI bot |
Merged at revision: | not available |
Proposed branch: | ~ogayot/curtin:recovery-key |
Merge into: | curtin:master |
Diff against target: |
97 lines (+66/-0) 2 files modified
curtin/commands/block_meta.py (+14/-0) tests/unittests/test_commands_block_meta.py (+52/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Server Team CI bot | continuous-integration | Needs Fixing | |
Dan Bungert | Approve | ||
Michael Hudson-Doyle | Approve | ||
Review via email: mp+449362@code.launchpad.net |
Commit message
block_meta: add luks recovery key if requested
If the storage configuration contains the key "recovery_keyfile", call
cryptsetup luksAddKey after cryptsetup luksFormat so that the key
specified is added as a recovery key (i.e., a normal key in the second
key slot).
Signed-off-by: Olivier Gayot <email address hidden>
Description of the change
This adds support for adding a LUKS recovery key. Something special probably needs to be done for s390x but that's all I have at the moment.
To post a comment you must log in.
PASSED: Continuous integration, rev:5d6aed93658 f5fb85b4277bdf2 1c9f10c80d8ebb /jenkins. canonical. com/server- team/job/ curtin- ci/171/ /jenkins. canonical. com/server- team/job/ curtin- ci/nodes= metal-amd64/ 171/ /jenkins. canonical. com/server- team/job/ curtin- ci/nodes= metal-arm64/ 171/ /jenkins. canonical. com/server- team/job/ curtin- ci/nodes= metal-ppc64el/ 171/ /jenkins. canonical. com/server- team/job/ curtin- ci/nodes= metal-s390x/ 171/
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/171/ /rebuild
https:/