-
Notifications
You must be signed in to change notification settings - Fork 155
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
coreos-boot-mount-generator: key off of /etc/fstab #247
Conversation
(Still need to test this.) |
overlay.d/05core/usr/lib/systemd/system-generators/coreos-boot-mount-generator
Outdated
Show resolved
Hide resolved
Hm, sorry about this, I certainly didn't even think at all about really old FCOS. Change seems reasonable to me as a better "detect build from Anaconda", but I think we can do better, and it's the same solution as we've been using in RHCOS, which is to key off of the |
Hmm, I think it's cleaner/more obviously correct to key off the thing we actually care about though. In this case, it's whether we need to emit boot mounts or whether |
Sure, that's fine by me too. |
OK, tested working from a node that started at |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two nits otherwise LGTM.
overlay.d/05core/usr/lib/systemd/system-generators/coreos-boot-mount-generator
Outdated
Show resolved
Hide resolved
overlay.d/05core/usr/lib/systemd/system-generators/coreos-boot-mount-generator
Show resolved
Hide resolved
Just look at `/etc/fstab` for `/boot` mounts to determine whether to generate them ourselves or let the `systemd-fstab-generator` do it. See also: coreos/fedora-coreos-tracker#322
Just look at
/etc/fstab
for/boot
mounts to determine whether togenerate them ourselves or let the
systemd-fstab-generator
do it.See also: coreos/fedora-coreos-tracker#322