Skip to content

fix(systemd): avoid rechunker local-fs ordering cycle - #919

Open
castrojo wants to merge 1 commit into
projectbluefin:mainfrom
castrojo:fix/rechunker-local-fs-cycle
Open

fix(systemd): avoid rechunker local-fs ordering cycle#919
castrojo wants to merge 1 commit into
projectbluefin:mainfrom
castrojo:fix/rechunker-local-fs-cycle

Conversation

@castrojo

@castrojo castrojo commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Closes #918

Summary

  • remove Wants=local-fs.target and After=local-fs.target from the early-boot rechunker-group-fix.service bridge;
  • preserve ordering after bootc-sysusers-shadow-sync.service and before systemd-sysusers.service;
  • add Bats regression coverage for the unit ordering and document the invariant.

These local-fs edges can create an ordering cycle through local-fs-pre.target, systemd-tmpfiles-setup-dev.service, and systemd-sysusers.service when upgrading legacy-rechunked images. The bridge only operates on /etc and does not require all local filesystems to be mounted.

Validation

  • git diff --check passed.
  • Focused Bats test could not run locally because bats is not installed in the environment.
  • Full just test / pre-commit run --all-files could not run because just, pre-commit, and pytest are not installed.

Remove local-fs.target ordering edges from the early-boot migration bridge while preserving ordering after bootc shadow synchronization and before systemd-sysusers. Add regression coverage and document the ordering invariant.

Assisted-by: GPT-5 via GitHub Copilot

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@castrojo
castrojo requested review from a team, ahmedadan, hanthor, inffy, ledif and repires as code owners August 3, 2026 03:50
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.

fix(systemd): rechunker-group-fix creates local-fs ordering cycle on upgrades

1 participant