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

Ensure that libvirtd is started after any enabled tcp/tls sockets #76

Closed
wants to merge 1 commit into from

Conversation

m-bull
Copy link
Contributor

@m-bull m-bull commented Feb 4, 2025

This role restarts systemd services in the correct order, but after a subsequent reboot, libvirtd may start before the socket services, resulting in a broken libvirtd. Ensure that the correct startup order is followed by creating a systemd drop-in that forces libvirtd to start after any enabled systemd sockets.

@m-bull m-bull force-pushed the systemd-socket-deps branch 2 times, most recently from e58efb5 to 202d0c4 Compare February 4, 2025 17:03
This role restarts systemd services in the correct
order, but after a subsequent reboot, libvirtd may
start before the socket services, resulting in a
broken libvirtd. Ensure that the correct startup
order is followed by creating a systemd drop-in
that forces libvirtd to start after any enabled
systemd sockets.
@m-bull m-bull force-pushed the systemd-socket-deps branch from 202d0c4 to a6dc366 Compare February 4, 2025 17:23
@jovial
Copy link
Contributor

jovial commented Feb 5, 2025

This is similar to #66. Did you have that commit: https://github.com/stackhpc/ansible-role-libvirt-host/pull/67/files when you hit this?

@m-bull
Copy link
Contributor Author

m-bull commented Feb 5, 2025

This doesn't work - just chucked it up for testing in the end.

The real answer is modular vs monolithic daemon systemd services - working through this now.

Closing.

@m-bull m-bull closed this Feb 5, 2025
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