Skip to content

Commit

Permalink
[NO TESTS NEEDED] Drop Warning to Info, if cgroups not mounted
Browse files Browse the repository at this point in the history
If I run podman within a container without /sys/fs/cgroups mounted,
we get this warning, which the user can not do much about.  I don't
see a reason for requiring this warning to show by default, so we
should drop it to Info.

User can not do anything about it either, and code works well with
the somewhat expected error.

Helps fix: containers#9753

Signed-off-by: Daniel J Walsh <[email protected]>
  • Loading branch information
rhatdan authored and jmguzik committed Apr 26, 2021
1 parent 1e3e7e7 commit 28cb8ca
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/domain/infra/abi/system.go
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ func (ic *ContainerEngine) SetupRootless(_ context.Context, cmd *cobra.Command)
if os.Geteuid() == 0 {
ownsCgroup, err := cgroups.UserOwnsCurrentSystemdCgroup()
if err != nil {
logrus.Warnf("Failed to detect the owner for the current cgroup: %v", err)
logrus.Infof("Failed to detect the owner for the current cgroup: %v", err)
}
if !ownsCgroup {
conf, err := ic.Config(context.Background())
Expand Down

0 comments on commit 28cb8ca

Please sign in to comment.