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

Added flatcar systemd-tmpfiles-setup-dev-early.service #85

Merged
merged 1 commit into from
Feb 13, 2024

Conversation

ader1990
Copy link
Contributor

@ader1990 ader1990 commented Feb 9, 2024

systemd-tmpfiles-setup-dev-early.service is required to be added instead of systemd-tmpfiles-setup-dev.service, as the kmod-static-nodes service needs to run before the systemd-tmpfiles-setup-dev service.

Otherwise, the static links like /dev/vhost-net, will not be created.

Once systemd >= 253 is used, the systemd-tmpfiles-setup-dev-early.service definition can be removed, as it is already built in.

--gracefuly flag is not supported for the systemd 252 version, so it had to be removed.

See systemd upstream definition:
https://github.com/systemd/systemd/blob/main/units/systemd-tmpfiles-setup-dev-early.service

Testing done

[Describe the testing you have done before submitting this PR. Please include both the commands you issued as well as the output you got.]

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
  • Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

@ader1990 ader1990 force-pushed the fix_kmod_static_nodes branch from 4baca82 to 22a1bcc Compare February 9, 2024 13:47
ader1990 added a commit to ader1990/scripts that referenced this pull request Feb 9, 2024
Update the bootengine commit to use the fix from:
flatcar/bootengine#85

Fixes: flatcar/Flatcar#1336

Signed-off-by: Adrian Vladu <[email protected]>
ader1990 added a commit to ader1990/scripts that referenced this pull request Feb 9, 2024
Update the bootengine commit to use the fix from:
flatcar/bootengine#85

Fixes: flatcar/Flatcar#1336

Signed-off-by: Adrian Vladu <[email protected]>
@ader1990 ader1990 force-pushed the fix_kmod_static_nodes branch from 22a1bcc to b0a38d5 Compare February 13, 2024 10:09
@ader1990 ader1990 marked this pull request as ready for review February 13, 2024 10:54
@ader1990 ader1990 force-pushed the fix_kmod_static_nodes branch from b0a38d5 to cc3e5a3 Compare February 13, 2024 10:55
systemd-tmpfiles-setup-dev-early.service is required to be added
instead of systemd-tmpfiles-setup-dev.service, as the kmod-static-nodes
service needs to run before the systemd-tmpfiles-setup-dev service.

Otherwise, the static links like /dev/vhost-net, will not be created.

Once systemd >= 253 is used, the systemd-tmpfiles-setup-dev-early.service
definition can be removed, as it is already built in.

`--gracefuly` flag is not supported for the systemd 252 version, so it
had to be removed.

See systemd upstream definition:
https://github.com/systemd/systemd/blob/main/units/systemd-tmpfiles-setup-dev-early.service

Signed-off-by: Adrian Vladu <[email protected]>
@ader1990 ader1990 force-pushed the fix_kmod_static_nodes branch from cc3e5a3 to 5f73716 Compare February 13, 2024 10:57
@ader1990
Copy link
Contributor Author

While testing this PR, an issue was seen with oem-cloudinit service failing. Fixed by #86.

ader1990 added a commit to flatcar/scripts that referenced this pull request Feb 13, 2024
Update the bootengine commit id to use the fix from:
flatcar/bootengine#85

Fixes: flatcar/Flatcar#1336

Signed-off-by: Adrian Vladu <[email protected]>
@pothos
Copy link
Member

pothos commented Feb 13, 2024

You can merge (and then update the scripts PR to use the merge commit ID)

@ader1990 ader1990 merged commit a85e197 into flatcar:flatcar-master Feb 13, 2024
ader1990 added a commit to flatcar/scripts that referenced this pull request Feb 13, 2024
Update the bootengine commit id to use the fix from:
flatcar/bootengine#85

Fixes: flatcar/Flatcar#1336

Signed-off-by: Adrian Vladu <[email protected]>
ader1990 added a commit to flatcar/scripts that referenced this pull request Feb 14, 2024
Update the bootengine commit id to use the fix from:
flatcar/bootengine#85

Fixes kubevirt vm creation by ensuring that /dev/vhost-net static node gets created
Fixes: flatcar/Flatcar#1336

Signed-off-by: Adrian Vladu <[email protected]>
pothos pushed a commit to flatcar/scripts that referenced this pull request Mar 5, 2024
Update the bootengine commit id to use the fix from:
flatcar/bootengine#85

Fixes kubevirt vm creation by ensuring that /dev/vhost-net static node gets created
Fixes: flatcar/Flatcar#1336

Signed-off-by: Adrian Vladu <[email protected]>
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.

2 participants