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

Handle MACHINE_ID=uninitialized #25407

Merged
merged 1 commit into from Nov 16, 2022

Conversation

schaefi
Copy link
Contributor

@schaefi schaefi commented Nov 16, 2022

systemd supports /etc/machine-id to be set to: uninitialized In this case the expectation is that systemd creates a new machine ID and replaces the value 'uninitialized' with the effective machine id. In the scope of kernel-install we should also enforce the creation of a new machine id in this condition

systemd supports /etc/machine-id to be set to: uninitialized
In this case the expectation is that systemd creates a new
machine ID and replaces the value 'uninitialized' with the
effective machine id. In the scope of kernel-install we
should also enforce the creation of a new machine id in this
condition
@DaanDeMeyer DaanDeMeyer added the good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed label Nov 16, 2022
@DaanDeMeyer
Copy link
Contributor

We really need to rewrite kernel-install in C so we don't have to reimplement all the helpers we already have :(

@keszybz
Copy link
Member

keszybz commented Nov 16, 2022

LGTM.

@yuwata yuwata merged commit 305dd91 into systemd:main Nov 16, 2022
@keszybz keszybz removed good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed needs-stable-backport labels Nov 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

5 participants