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

infra: adjust the containers to use full systemd #6147

Merged
merged 1 commit into from
Feb 11, 2025

Conversation

KKoukiou
Copy link
Contributor

@KKoukiou KKoukiou commented Feb 7, 2025

The systemd-standalone-sysusers package, which is pre-installed in the containers, conflicts with systemd. Since Anaconda requires systemd as a dependency, this conflict prevents installation during tests.

This update ensures that containers use the full systemd package to resolve the dependency issue.

@github-actions github-actions bot added infrastructure Changes affecting mostly infrastructure f42 Fedora 42 labels Feb 7, 2025
@KKoukiou KKoukiou changed the title infra: Adjust the Containers to Use Full systemd infra: adjust the containers to use full systemd Feb 7, 2025
@KKoukiou
Copy link
Contributor Author

This fixes CI unit tests runs but RPM tests are still broken.

@M4rtinK
Copy link
Contributor

M4rtinK commented Feb 10, 2025

This fixes CI unit tests runs but RPM tests are still broken.

Yeah, just tired locally & I don't see that issue anymore. So some progress. :)

Copy link
Contributor

@M4rtinK M4rtinK left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks! :)

@KKoukiou KKoukiou force-pushed the systemd-standalone branch 4 times, most recently from bc16efb to 8e3a9c9 Compare February 10, 2025 15:00
The systemd-standalone-sysusers package, which is pre-installed in the containers,
conflicts with systemd. Since Anaconda requires systemd as a dependency,
this conflict prevents installation during tests.

This update ensures that containers use the full systemd package to resolve the dependency issue.
@KKoukiou
Copy link
Contributor Author

/kickstart-tests --waive infra

@jkonecny12
Copy link
Member

I like this but it would be great to have FIXME: Remove this when <bug link> is resolved so it is visible that this is a workaround.

@jkonecny12
Copy link
Member

jkonecny12 commented Feb 11, 2025

After looking into the issue more closely, I found that there are too many indirect dependencies on systemd so removing systemd as dependency would be too problematic.

Also I misunderstood the issue before. Feel free to ignore my previous comment.

@KKoukiou KKoukiou merged commit d36dffe into rhinstaller:main Feb 11, 2025
16 of 17 checks passed
@KKoukiou KKoukiou deleted the systemd-standalone branch February 11, 2025 13:12
@jkonecny12 jkonecny12 added f43 and removed f42 Fedora 42 labels Feb 12, 2025
rvykydal added a commit to rvykydal/kickstart-tests that referenced this pull request Feb 14, 2025
The systemd-standalone-sysusers package, which is pre-installed in the
rawhide container conflicts with systemd. Since lorax requires systemd
as a dependency, this conflict prevents lorax installation.

See also rhinstaller/anaconda#6147
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
f43 infrastructure Changes affecting mostly infrastructure
Development

Successfully merging this pull request may close these issues.

3 participants