Add bash helper script for loading encryption password for LUKS#482
Open
mariaa144 wants to merge 1 commit intoopenzfs:masterfrom
Open
Add bash helper script for loading encryption password for LUKS#482mariaa144 wants to merge 1 commit intoopenzfs:masterfrom
mariaa144 wants to merge 1 commit intoopenzfs:masterfrom
Conversation
I thought it would be nice if there was some check to make sure the password is correct because getting it wrong can be a real pain.
|
The intention is good, but the inplementation is overcomplicated. Just remove all printf commands, the pipe sign, and the single dash sign trailing each line.
This would let cryptsetup handle password input and validation.
|
|
And you will need to keep an automated version for CI/CD testing. See other sections for examples. Remember to get indentation right.
于 2023年12月16日 GMT+01:00 上午1:45:21, Maria ***@***.***> 写到:
…I thought it would be nice if there was some check to make sure the password is correct because getting it wrong can be a real pain.
You can view, comment on, or merge this pull request online at:
#482
-- Commit Summary --
* Add bash helper script for loading encryption password for LUKS
-- File Changes --
M docs/Getting Started/NixOS/Root on ZFS.rst (7)
-- Patch Links --
https://github.com/openzfs/openzfs-docs/pull/482.patch
https://github.com/openzfs/openzfs-docs/pull/482.diff
--
Reply to this email directly or view it on GitHub:
#482
You are receiving this because you are subscribed to this thread.
Message ID: ***@***.***>
|
Contributor
Author
|
If we do that then you have to enter the passphrase 3 times for each disk instead of 2 times total. I did expect the prompts to break the automation but I didn't expect the REHL guide to break. I assume the checks didn't get to NixOS. |
|
That check failed due to a file removal on RHEL part: I currently do not have time to fix this. |
Member
|
Unfortunately, we don't have a maintainer of NixOS guide now, so I'll pick up at least reviews, if you're ready to walk this through - please rebase it and I'll merge it, thank you! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I thought it would be nice if there was some check to make sure the password is correct because getting it wrong can be a real pain.