-
Notifications
You must be signed in to change notification settings - Fork 674
Make sure to install podman package from repo #2990
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
Conversation
I thought we could put something in
What do you mean by "like before"? Did |
I'm not sure if I would suggest |
Ok, I just realized that the issue is that Fedora comes with podman already installed, which disables our code that used to install the latest version on first boot. So this PR does restore the old behaviour; the only reason I'm not merging is because I think the sentinel file should be in |
Like with older versions of Fedora, what was I referring to (but that info was only in issue) The Originally I used /tmp instead of /var/tmp, but that is also broken because Fedora (tmpfs) Using /etc is slightly better, but it is not ideal that programs are creating config on their own... |
Newer versions of Fedora come with an older pre-installed Podman, make sure to install the package from the repository like before. Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
8c0e426
to
ab544b5
Compare
@@ -43,7 +43,7 @@ provision: | |||
mkdir -p /etc/tmpfiles.d | |||
echo "d /run/podman 0700 {{.User}} -" > /etc/tmpfiles.d/podman.conf | |||
fi | |||
dnf -y install podman | |||
dnf -y install --best podman && touch /etc/lima-podman |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
--best
is nice, but touch /etc/lima-podman
seems controversial
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see how using /etc/lima-podman
can create any problems. Every service nowadays seems to store their config in there (including apps, e.g. nanorc
or screenrc
):
$ lima ls /etc | wc -l
189
But if you don't like it, how about mkdir -p /var/lib/lima
then? I don't really care; I was only pointing out that a temp dir that a user may reasonably wipe clean is a poor location for a file that should never be deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, LGTM.
Not merging myself as there doesn't seem to be consensus about the sentinel location yet.
My concern was not about the location. |
That's what this is all about: how do you make sure that we are not running Before we relied on Fedora not including So we want to upgrade to the latest How would you solve the issue of running |
So old podman is installed by default and |
Unless I misunderstand, this is simply due to the latest Fedora 41 1.4 image being from Oct 24 containing podman 5.2.5 released on Oct 23. Since then there have been podman releases 5.3.0 (Nov 13) and 5.3.1 (Nov 21) that you can install via |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
Newer versions of Fedora come with an older pre-installed Podman,
make sure to install the package from the repository like before.
Closes #2978
Fixes the issue on my machine:
It needed --best too, or it would exit:
Package "podman-5:5.2.5-1.fc41.x86_64" is already installed.