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

emergency.target.d/core-override.conf seems to be incorrect #33

Open
mvo5 opened this issue Aug 4, 2021 · 6 comments
Open

emergency.target.d/core-override.conf seems to be incorrect #33

mvo5 opened this issue Aug 4, 2021 · 6 comments

Comments

@mvo5
Copy link
Contributor

mvo5 commented Aug 4, 2021

During the boot there are a bunch of warnings like this:

emergency.target: Requested dependency OnFailure=reboot.target ignored (target units cannot fail).

which indicates that the factory/usr/lib/systemd/system/emergency.target.d/core-override.conf file is not quite correct.

@vorlonofportland
Copy link
Contributor

@xnox this file was your commit, do you have insight here?

@xnox
Copy link
Contributor

xnox commented Aug 10, 2021

@vorlonofportland target units can fail to be reached though. I think it is a regression in systemd upstream if they stopped allowing OnFailure with target units (which is not even a unit type in systemd).

@alfonsosanchezbeato
Copy link
Member

Apparently this fixed the problem in systemd v246: systemd/systemd@94d1ddb

@alfonsosanchezbeato
Copy link
Member

alfonsosanchezbeato commented Oct 22, 2021

Closing as this is fixed in systemd upstream, and it does not look like doing an SRU is worth it, as we can live without the system rebooting when emergency.target fails.

@mvo5
Copy link
Contributor Author

mvo5 commented Oct 22, 2021

Fwiw, given how trivial the fix and how confusing/irritating the warnings I would ask if the next systemd SRU can incldue this one line fix for this.

@alfonsosanchezbeato
Copy link
Member

Fair enough, I've created https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/1948476

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

No branches or pull requests

4 participants