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

test containers: add systemd to tang image #3983

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

jbtrystram
Copy link
Contributor

In the F41 fedora-minimal image, systemctl stopped working. We need to enable tang.service in the container so we need systemd in the container.

Adding systemd in the container.

In the F41 fedora-minimal image, systemctl stopped working.
We need to enable tang.service in the container so we need systemd
in the container.

Adding systemd in the container.
@travier
Copy link
Member

travier commented Dec 4, 2024

In the F41 fedora-minimal image, systemctl stopped working.

Do you mean that systemd was removed in the minimal images?

@@ -1,6 +1,6 @@
FROM registry.fedoraproject.org/fedora-minimal:41
Copy link
Member

Choose a reason for hiding this comment

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

would switching to the non-minimal image here be a better move? i.e. would less rpms get downloaded in the dnf install below?

Copy link
Contributor Author

@jbtrystram jbtrystram Dec 4, 2024

Choose a reason for hiding this comment

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

it's not enough to fix this issue though, systemd is not in the non-minimal image either. So I'd rather stick with the smaller image

@jbtrystram
Copy link
Contributor Author

In the F41 fedora-minimal image, systemctl stopped working.

Do you mean that systemd was removed in the minimal images?

yeah sorry. But it was also removed from the non-minimal image so I just kept the minimal image and added systemd back

@dustymabe dustymabe merged commit b3b8bd7 into coreos:main Dec 4, 2024
5 checks passed
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.

3 participants