-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
rawhide, rootless: spurious warning messages #4012
Comments
I have already brought up this issue. rm ~/.config/containers/libpod.conf And you should be all set. The problem is on upgrades, anyone who has run rootless podman in the past will have a libpod.conf in their homedir pointing at runc. |
This is a duplicate of |
I'm not sure it's a dup: I remember checking podman config, and in fact still see this in my scrollback buffer:
(this was a fresh virt, and had never run an old podman as nonroot). Plus it surprised me that podman-78 did not have the error, but it appeared in -79. Both use crun. But it's bedtime here, and I won't have time tomorrow morning to bring up a new VM to test, so I'm respectfully reopening so @giuseppe will have a chance to have a quick look. If it really is a dup, I apologize in advance for wasting time. |
Did we remove |
I remember doing Oh another data point suggesting (to me) that this is not a dup of 3976: gating tests are failing, and those are fresh VMs. (Click the '8m' bar to expand; I have no idea why it's not auto-expanding and why it's not shown in red). |
There is no hard dependency on runc, it has switched to crun. Why is podman (code) requiring runc? If the libpod.conf is set to crun, runc should be ignored. |
We don't require it but will warn if it is in the configuration but not
available
…On Fri, Sep 13, 2019, 06:15 Daniel J Walsh ***@***.***> wrote:
There is no hard dependency on runc, it has switched to crun. Why is
podman (code) requiring runc? If the libpod.conf is set to crun, runc
should be ignored.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#4012>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AB3AOCF6DCVSVD2CWANCFZDQJNR4BANCNFSM4IWJPYLA>
.
|
Please don't neglect this message:
I've just run a quick test and see this message whether runc is installed or not. So in short:
|
Why are we showing these errors by default? I thought we defaulted to ERROR. |
@giuseppe I think that systemd CGroup error was added by you, with the recent PR to join systemd cgroups? |
@rhatdan We do, AFAIK |
@mheon yes my recent PR enabled systemd cgroups. I did a mistake in the patch though and the error is overwritten, I'll prepare a PR to address it |
do not shadow the err variable so that the correct error message can be reported when utils.RunUnderSystemdScope fails. Closes: containers#4012 Signed-off-by: Giuseppe Scrivano <[email protected]>
I've opened a PR here: #4018 |
The
This is causing Rawhide gating tests to fail; which is a pretty good indicator that this will cause a bad user experience in rawhide. Reopening. |
These are logrus.Warnf and our default log level should be error; they shouldn't be printing by default. I do not want to remove the |
Can we just fix it somehow? Fix the default configuration, fix whatever needs to be fixed, just fix things so that these error messages stop showing up? As I mentioned, they are causing rawhide system tests to fail. The system tests use BATS, which (love it or not) combines stdout and stderr streams. I am not going to spend effort hacking the system tests to filter out all instances of |
Why are they being printed? They should not be displayed by default. Are you forcing a non-default log level? |
No. Not intentionally, not to my memory either, and the words 'level' and 'log' do not appear in any related context under the |
Do you see this happen, anywhere but in the test? |
I saw it last week on a rawhide test setup. I'll bring up another rawhide virt now and confirm. |
This code seems curios. It looks to me like the outputError would output to stderr if the logLevel is not set to debug? |
Well I guess it does not come from this, since |
Sorry for the delay; there's been an openstack outage until just now. It is not log-level:
(FWIW, the messages also appear with podman-1.5.2-0.90.dev.git1f5514e.fc32.x86_64 |
Dum de dum. Out of desperation I edited libpod.conf, removed all references to runc:
|
Did you remove it from /etc/containers/libpod.conf and /usr/share/containers/libpod.conf |
No, but I just tried, no difference: still the same three warnings. |
Where the frig is it finding runc... |
@giuseppe Any ideas |
Already did (system migrate) (sorry for not saying so). Also confirmed, via ps auxww, that there are no podman processes running. Also tried Also tried |
@edsantiago We can pull runtimes from both system and local libpod.conf - can you confirm you removed (Just now realizing how much of an ordeal this actually is...) |
There is no
In case it's helpful (it isn't to me),
PATH is If you can find where that |
...that being |
It's falling back to $PATH because it can't find |
I wonder if the default configuration isn't being applied regardless of |
I think it is the default configuration being used. If there are no configuration files then we fallback to: https://github.com/containers/libpod/blob/master/libpod/runtime.go#L302-L313 I've tried configuring
then as rootless The configuration is created with only |
I run into this out of the box with Fedora 31 Workstation beta 1.1 which first has had podman updated to podman-1.5.1-3.git0005792.fc31.x86_64, and has not previously been run until now.
There is no libpod.conf in /etc but there is in ~/.config
If I run the command a second time, the problem does not occur.
If I delete the user libpod.conf and retry, the error still does not return. And if I delete ~/.local/share/containers/* the problem also does not recur. So it's a bit confusing. UPDATE: And if I reboot and run the command again, error happens. But only once. |
ping, this is still causing gating test failures on rawhide |
If we don't do this, we print WARN level messages that we should not be printing by default. Up one WARN message to ERROR so it still shows up by default. Fixes: containers#4115 Fixes: containers#4012 Signed-off-by: Matthew Heon <[email protected]>
On rawhide, rootless:
Adding
|& cat
converts the messages to debug format, with time= / level=warning / msg=...podman-1.5.2-0.79.dev.gitaf8fedc.fc32.x86_64.rpm , but 78.dev.gitafa3d11 does not elicit this, implying that the problem was introduced between afa3d11 and af8fedc
The text was updated successfully, but these errors were encountered: