Skip to content

Comments

sys-kernel/bootengine: pull ignition changes#2308

Merged
tormath1 merged 2 commits intomainfrom
tormath1/ignition
Sep 16, 2024
Merged

sys-kernel/bootengine: pull ignition changes#2308
tormath1 merged 2 commits intomainfrom
tormath1/ignition

Conversation

@tormath1
Copy link
Contributor

@tormath1 tormath1 commented Sep 11, 2024

this pulls Ignition reordering to fix Equinix Metal issues. (flatcar/bootengine#99) (upstream: coreos/ignition#1940)


Ignition will POST a status update to Equinix Metal at each stage (disk, mount, files, etc.). The failure occurs in mount service, because systemd-networkd is shutted down, as all other services because initrd-cleanup.service is started.
initrd-cleanup.service is started because Ignition has reported success (last stage files is executed with success) so the Ignition complete target is reached.

But the mount service has an ExecStop condition:

ExecStop=/usr/bin/ignition --root=/sysroot --platform=${PLATFORM_ID} --stage=umount ${IGNITION_ARGS}

which will fail on Equinix Metal because as said: we need the network to POST updates to Equinix Metal

Testing done

Related to: flatcar/Flatcar#1536

this pulls Ignition reordering to fix Equinix Metal issues.

Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
@tormath1 tormath1 marked this pull request as ready for review September 16, 2024 07:52
@tormath1 tormath1 requested a review from a team September 16, 2024 07:52
@tormath1 tormath1 merged commit 7234433 into main Sep 16, 2024
@tormath1 tormath1 deleted the tormath1/ignition branch September 16, 2024 12:21
@tormath1
Copy link
Contributor Author

cherry-picked to:

For Stable, I'm still considering if we should or not create a backport branch on bootengine to avoid shipping the PXE/ISO fix in Stable directly.

@github-actions
Copy link

Build action triggered: https://github.com/flatcar/scripts/actions/runs/10883888286

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants