Skip to content

Dont panic if no unlock method available#3355

Merged
mulkieran merged 2 commits intostratis-storage:masterfrom
mulkieran:dont-panic-if-no-unlock
May 31, 2023
Merged

Dont panic if no unlock method available#3355
mulkieran merged 2 commits intostratis-storage:masterfrom
mulkieran:dont-panic-if-no-unlock

Conversation

@mulkieran
Copy link
Copy Markdown
Member

@mulkieran mulkieran commented May 31, 2023

Cherry-picked from #3352

mulkieran added 2 commits May 31, 2023 09:33
It is not logically necessary for an unlock method to be discoverable,
the LUKS metadata may have been tampered with in such a way that it is
still clear that the LUKS device is a Stratis device but neither a key
nor a Clevis configuration can be obtained.

Signed-off-by: mulhern <amulhern@redhat.com>
Signed-off-by: mulhern <amulhern@redhat.com>
@mulkieran mulkieran self-assigned this May 31, 2023
@mulkieran mulkieran changed the title Dont panic if no unlock Dont panic if no unlock method available May 31, 2023
@mulkieran mulkieran requested a review from jbaublitz May 31, 2023 13:40
@mulkieran mulkieran marked this pull request as ready for review May 31, 2023 13:40
@mulkieran mulkieran merged commit cc9af09 into stratis-storage:master May 31, 2023
@mulkieran mulkieran deleted the dont-panic-if-no-unlock branch May 31, 2023 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants