Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for creating LUKS HW-OPAL devices #5826

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

vojtechtrefny
Copy link
Contributor

This adds support for creating encrypted devices using the OPAL self-encrypting drives. Anaconda only needs to make sure to pass the OPAL administrator passphrase to Blivet when creating the LUKS device, everything else will be taken care by either Blivet or pykickstart -- whether a "normal" or OPAL LUKS device will be created is controlled by the "--luks-version" kickstart option specified by the user which is passed "as is" to Blivet.

@vojtechtrefny vojtechtrefny added the blocked Don't merge this pull request! label Aug 13, 2024
@pep8speaks
Copy link

pep8speaks commented Aug 13, 2024

Hello @vojtechtrefny! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2024-08-21 16:45:01 UTC

@vojtechtrefny
Copy link
Contributor Author

Marking as blocked for now: libblockdev (storaged-project/libblockdev#1047), blivet (storaged-project/blivet#1266) and pykickstart (pykickstart/pykickstart#502) support need to merged first and we'll also need to coordinate the releases in Fedora.

@vojtechtrefny
Copy link
Contributor Author

The blivet and libblockdev support is now merged and making its way to Fedora.

@vojtechtrefny
Copy link
Contributor Author

/kickstart-test --testtype smoke

@vojtechtrefny vojtechtrefny marked this pull request as ready for review August 21, 2024 06:14
@vojtechtrefny
Copy link
Contributor Author

/kickstart-test --testtype storage

@vojtechtrefny vojtechtrefny removed the blocked Don't merge this pull request! label Aug 21, 2024
@vojtechtrefny
Copy link
Contributor Author

Removing the blocked label, all the dependencies are now available in the latest F41 compose (Fedora-41-20240820.n.0).

Copy link
Contributor

@rvykydal rvykydal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@vojtechtrefny
Copy link
Contributor Author

/kickstart-test --testtype smoke

This adds support for creating encrypted devices using the OPAL
self-encrypting drives. Anaconda only needs to make sure to pass
the OPAL administrator passphrase to Blivet when creating the
LUKS device, everything else will be taken care by either Blivet
or pykickstart -- whether a "normal" or OPAL LUKS device will be
created is controlled by the "--luks-version" kickstart option
specified by the user which is passed "as is" to Blivet.
@KKoukiou
Copy link
Contributor

/kickstart-test --testtype smoke

Copy link
Contributor

@KKoukiou KKoukiou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New containers are now available, test passed.

@vojtechtrefny vojtechtrefny merged commit c672868 into rhinstaller:master Aug 22, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

4 participants