Skip to content
This repository was archived by the owner on Aug 25, 2021. It is now read-only.

ignition-firstboot-complete.service: Run as part of basic.target#124

Merged
miabbott merged 1 commit intocoreos:masterfrom
cgwalters:firstboot-complete-early
Oct 23, 2019
Merged

ignition-firstboot-complete.service: Run as part of basic.target#124
miabbott merged 1 commit intocoreos:masterfrom
cgwalters:firstboot-complete-early

Conversation

@cgwalters
Copy link
Member

See: openshift/installer#2554

Basically the OpenShift installer on vSphere injects a service which reboots
the first time, but I believe this could easily race with ignition-firstboot-complete.

Let's mark the boot complete much earlier.

See: openshift/installer#2554

Basically the OpenShift installer on vSphere injects a service which reboots
the first time, but I believe this could easily race with `ignition-firstboot-complete`.

Let's mark the boot complete much earlier.
@ajeddeloh
Copy link
Contributor

We've talked in the past about moving this into the initramfs like it is on CL. Any reason not to move it there?

@cgwalters
Copy link
Member Author

I can't think of a reason not to do that. Though it'd break #117 (since zipl isn't in the initramfs) but we could probably address that by just adding it.

@cgwalters
Copy link
Member Author

So...let's get this easy fix in first though?

Copy link
Contributor

@ajeddeloh ajeddeloh left a comment

Choose a reason for hiding this comment

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

Sure

@miabbott miabbott merged commit 9a25a5e into coreos:master Oct 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants