Skip to content
This repository has been archived by the owner on May 30, 2023. It is now read-only.

sys-kernel/bootengine: add cryptsetup in initramfs #1760

Merged
merged 2 commits into from
Mar 24, 2022
Merged

Conversation

tormath1
Copy link
Contributor

@tormath1 tormath1 commented Mar 24, 2022

this is required to run luks encryption with ignition

Signed-off-by: Mathieu Tortuyaux mtortuyaux@microsoft.com


it pulls:

@JAORMX
Copy link
Contributor

JAORMX commented Mar 24, 2022

Is this only meant for the CI run right now? I'd expect the CROS_WORKON_COMMIT to keep pointing to flatcar-master

@tormath1
Copy link
Contributor Author

Is this only meant for the CI run right now? I'd expect the CROS_WORKON_COMMIT to keep pointing to flatcar-master

Yes - exactly, it's only CI for now, that's why this PR is a "draft" . We target the commit: flatcar/bootengine#41 we actually want to test. :)

@pothos
Copy link
Contributor

pothos commented Mar 24, 2022

Is this only meant for the CI run right now? I'd expect the CROS_WORKON_COMMIT to keep pointing to flatcar-master

The final commit ID used has to be the one from the merge commit, to make sure that concurrent PRs don't discard other's changes by still pointing to the commit before merge

@tormath1 tormath1 marked this pull request as ready for review March 24, 2022 13:39
@tormath1 tormath1 requested a review from a team March 24, 2022 13:39
tormath1 and others added 2 commits March 24, 2022 17:00
this is required to run luks encryption with ignition

Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
Co-authored-by: Kai Lüke <pothos@users.noreply.github.com>
@tormath1 tormath1 merged commit dbadd20 into main Mar 24, 2022
@tormath1 tormath1 deleted the tormath1/bootengine branch March 24, 2022 16:01
@tormath1
Copy link
Contributor Author

cherry-picked to:

  • flatcar-3185

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants