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

fix(systemd-sysusers): override systemd-sysusers.service #1658

Merged
merged 1 commit into from
Dec 10, 2021

Conversation

Cornelicorn
Copy link
Contributor

Fixes a regression with systemd not running units with ConditionNeedsUpdate set in initrds

Checklist

  • I have tested it locally
  • I have reviewed and updated any documentation if relevant
  • I am providing new code and test(s) for it

Fixes #1656

@github-actions github-actions bot added the modules Issue tracker for all modules label Nov 29, 2021
Copy link
Collaborator

@johannbg johannbg left a comment

Choose a reason for hiding this comment

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

In your commit message you need to seperate the subject and body with an empty line as well as fix the spelling error sytemd-sysusers.service should be systemd-sysusers.service

Fixes a regression with systemd not running units with ConditionNeedsUpdate set in initrds
@Cornelicorn Cornelicorn changed the title fix(systemd-sysusers): override sytemd-sysusers.service fix(systemd-sysusers): override systemd-sysusers.service Nov 29, 2021
@johannbg johannbg self-requested a review November 29, 2021 21:11
@johannbg johannbg enabled auto-merge (rebase) November 29, 2021 21:12
@johannbg johannbg merged commit dcbe23c into dracutdevs:master Dec 10, 2021
@Cornelicorn Cornelicorn deleted the systemd-sysusers branch December 10, 2021 13:11
aafeijoo-suse added a commit to aafeijoo-suse/dracut that referenced this pull request Nov 21, 2022
When generating the initrd, dracut copies all the sysusers dropins provided by
its modules to it, and systemd-sysusers runs at every boot to always create the
same users (`ConditionNeedsUpdate` removed in dracutdevs#1658).

This patch is intended to create the system users only when the initrd is
generated, without including the systemd-sysusers stuff in the initrd, as it is
not necessary.

To control this functionality and allow to use the old method, added the
`create_sysusers` configuration option (default to yes) and the
`--no-create-sysusers` parameter.

Fixes dracutdevs#1830
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
modules Issue tracker for all modules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

systemd-resolve user missing
3 participants