Repository navigation
Include polkit in systemd RPM package selection - #116
Merged
Jiri Appl (jiria) merged 1 commit intoOct 19, 2026
Merged
Conversation
Select polkit explicitly with systemd so RPM images retain the PolicyKit daemon even when Portage dependency enumeration omits it. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: deaf3826-2e46-4f65-8960-6449244e09f9
There was a problem hiding this comment.
🟢 Approval recommended
The narrow catalog change has no identified blocking issues, though full image and runtime validation remain outstanding.
No open issues 🎉
What changed in this PR
Ensures ACL RPM images explicitly include the PolicyKit daemon through systemd package selection.
Changes:
- Adds
polkitto the systemd RPM list. - Preserves the standalone polkit mapping without resolver or runtime configuration changes.
| File | Description |
|---|---|
build_library/rpm/package_catalog.yaml |
Selects polkit alongside systemd. |
Review effort: Balanced
9 of 16 tasks
SeanDougherty
approved these changes
Oct 19, 2026
Lanze Liu (liulanze)
approved these changes
Oct 19, 2026
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Explicitly select the
polkitRPM alongside systemd so ACL RPM images include the PolicyKit daemon. The existing standalonesys-auth/polkitmapping only takes effect when that Portage package reaches catalog resolution; it does not independently request installation.Change Log
polkitto thesys-apps/systemdRPM list inbuild_library/rpm/package_catalog.yaml.Type of Change
Does this affect the image build?
Associated Issues
N/A.
Test Methodology
python build_library/rpm/parse_catalog.py build_library/rpm/package_catalog.yaml— passed; the systemd entry includespolkitand the standalone polkit mapping is preserved.source build_library/rpm/package_catalog.sh; get_rpm_package_name sys-apps/systemd— passed forBOARD=amd64-usrandBOARD=arm64-usr, each withBOOTLOADER_MODE=grubandBOOTLOADER_MODE=uki; assertions confirmed exactly onepolkitentry. Native Python stdout line endings were normalized for the Windows/Git Bash check.git diff --check— passed.Merge Checklist
All applicable boxes should be checked before merging