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

feat: bring disk encryption support #105

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

smira
Copy link
Member

@smira smira commented Aug 22, 2024

Mostly copy-paste from v1 of go-blockdevice, but with some fixes to adapt to new code in other places.

encryption/luks/token.go Outdated Show resolved Hide resolved
encryption/luks/token.go Outdated Show resolved Hide resolved
encryption/luks/luks.go Outdated Show resolved Hide resolved
encryption/luks/luks.go Outdated Show resolved Hide resolved
encryption/luks/luks.go Outdated Show resolved Hide resolved
@smira smira force-pushed the feat/encryption-support branch from 6bdbb7d to 8ea1320 Compare August 22, 2024 12:59
@smira
Copy link
Member Author

smira commented Aug 22, 2024

thanks, fixed up all of that

Mostly copy-paste from `v1` of `go-blockdevice`, but with some fixes to
adapt to new code in other places.

Signed-off-by: Andrey Smirnov <[email protected]>
@smira smira force-pushed the feat/encryption-support branch from 8ea1320 to fa9291f Compare August 22, 2024 14:45

// Token defines LUKS2 token.
type Token[UserData any] struct {
// UserData has a strange JSON tag, but this keeps it backwards compatible with v1 library.
Copy link
Member Author

Choose a reason for hiding this comment

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

added some clarity and tests here

@smira
Copy link
Member Author

smira commented Aug 22, 2024

/m

@talos-bot talos-bot merged commit fa9291f into siderolabs:v2 Aug 22, 2024
14 checks passed
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.

5 participants