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

[spec2x] Add a fetch stage #863

Merged
merged 1 commit into from
Oct 1, 2019
Merged

Conversation

cgwalters
Copy link
Member

This stage just triggers the internal code which fetches the
config and writes /run/ignition.json; from there it'd be picked
up by later stages the way it was implicitly before.

This is prep for adding support for redeploying the rootfs for
Fedora CoreOS; we need to have a separate process determine
whether the rootfs is being replaced, and if so do a save/restore
across ignition-disks.service.

See: coreos/ignition-dracut#107
(cherry picked from commit 8926f0f)

@cgwalters
Copy link
Member Author

cgwalters commented Sep 26, 2019

Per openshift/enhancements#15 the plan is to change RHCOS to come LUKS by default short term, and not do ignition-rootfs moves.

However, we need a way to configure the LUKS inside the initramfs to pass that config to a custom dracut module.

In combination with the fact that Ignition ignores unknown keys, this split out fetch stage gives us a way to extend Ignition in a sneaky fashion without going all the way to adding a public API for it. (The public API for encryption for OpenShift/RHCOS will be much higher level to start but the idea is that something like that would also end up as Ignition or sugar for Ignition)

@ajeddeloh
Copy link
Contributor

Imports need fixing to not import v2 stuff, otherwise lgtm

@ajeddeloh
Copy link
Contributor

Gofmt failed.

@dustymabe dustymabe changed the title Add a fetch stage [spec2x] Add a fetch stage Sep 30, 2019
This stage just triggers the internal code which fetches the
config and writes `/run/ignition.json`; from there it'd be picked
up by later stages the way it was implicitly before.

This is prep for adding support for redeploying the rootfs for
Fedora CoreOS; we need to have a separate process determine
whether the rootfs is being replaced, and if so do a save/restore
across `ignition-disks.service`.

See: coreos/ignition-dracut#107
(cherry picked from commit 8926f0f)
@cgwalters
Copy link
Member Author

Made gofmt happy!

@ajeddeloh ajeddeloh merged commit 3625532 into coreos:spec2x Oct 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants