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

fedora-coreos-base.yaml: nuke systemd-gpt-auto-generator #119

Merged
merged 1 commit into from
Jul 17, 2019

Conversation

jlebon
Copy link
Member

@jlebon jlebon commented Jul 17, 2019

Otherwise, systemd-gpt-auto-generator's generated boot.automount
will cause a funky mount structure:

├─/boot           systemd-1    autofs
│ └─/boot         /dev/vda1    ext4
│   └─/boot/efi   /dev/vda2    vfat

The nature of autofs then causes issues for us around shutdown
ordering. See also the linked systemd issue.

Really, we don't need the GPT generator at all for now, so just nuke it
from the tree.

Closes: coreos/fedora-coreos-tracker#215

Otherwise, `systemd-gpt-auto-generator`'s generated `boot.automount`
will cause a funky mount structure:

```
├─/boot           systemd-1    autofs
│ └─/boot         /dev/vda1    ext4
│   └─/boot/efi   /dev/vda2    vfat
```

The nature of `autofs` then causes issues for us around shutdown
ordering. See also the linked systemd issue.

Really, we don't need the GPT generator at all for now, so just nuke it
from the tree.

Closes: coreos/fedora-coreos-tracker#215
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.

LGTM

@@ -63,6 +63,10 @@ remove-from-packages:
/usr/lib/systemd/system/systemd-resolved.service,
/usr/lib/systemd/system/sysinit.target.wants/systemd-firstboot.service]

# We don't want auto-generated mount units. See also
# https://github.com/systemd/systemd/issues/13099
- [systemd-udev, /usr/lib/systemd/system-generators/systemd-gpt-auto-generator]
Copy link
Contributor

Choose a reason for hiding this comment

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

LGTM, but why are generators in the udev package?

Copy link
Member Author

Choose a reason for hiding this comment

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

Not sure. Not all of them are in there, just a few.

@jlebon jlebon merged commit 5402a4a into coreos:testing-devel Jul 17, 2019
@jlebon jlebon deleted the pr/drop-gpt-generator branch April 23, 2023 23:28
dustymabe pushed a commit to jbtrystram/fedora-coreos-config that referenced this pull request Apr 19, 2024
This adds a FAQ entry explaining that Ignition and fcct configurations
are different interfaces (not just JSON-vs-YAML for the same content).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OSTree finalization on shutdown errors out
2 participants