diff options
author | Kevin Downey <hiredman@thelastcitadel.com> | 2022-05-15 23:04:30 -0700 |
---|---|---|
committer | Matthew Heon <mheon@redhat.com> | 2022-06-14 13:42:35 -0400 |
commit | 2b8a5843ecf3baa2b3bf3d60de9798fd0c562454 (patch) | |
tree | cab5dca7e48a48c3f981e6672e462c4002f7d8bb | |
parent | 41e6eff62b4a65b73fafb00a796af7409a3e2b4f (diff) | |
download | podman-2b8a5843ecf3baa2b3bf3d60de9798fd0c562454.tar.gz podman-2b8a5843ecf3baa2b3bf3d60de9798fd0c562454.tar.bz2 podman-2b8a5843ecf3baa2b3bf3d60de9798fd0c562454.zip |
Don't complain about XDG_RUNTIME_DIR, Closes #1424
Code is not directly reading XDG_RUNTIME_DIR, it is reading a value in
the state that may initially be from XDG_RUNTIME_DIR, but then is
overriden by a value from the boltdb that podman stores some state in.
XDG_RUNTIME_DIR and the RunRoot path may not have the same value, so
complaining about XDG_RUNTIME_DIR here may cause confusion when trying
to debug things.
[NO TESTS NEEDED]
Signed-off-by: Kevin Downey <hiredman@thelastcitadel.com>
-rw-r--r-- | libpod/runtime.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libpod/runtime.go b/libpod/runtime.go index 78ff8ce94..c9f489033 100644 --- a/libpod/runtime.go +++ b/libpod/runtime.go @@ -345,7 +345,7 @@ func makeRuntime(runtime *Runtime) (retErr error) { // it will try to use existing XDG_RUNTIME_DIR // if current user has no write access to XDG_RUNTIME_DIR we will fail later if err := unix.Access(runtime.storageConfig.RunRoot, unix.W_OK); err != nil { - msg := "XDG_RUNTIME_DIR is pointing to a path which is not writable. Most likely podman will fail." + msg := fmt.Sprintf("RunRoot is pointing to a path (%s) which is not writable. Most likely podman will fail.", runtime.storageConfig.RunRoot) if errors.Is(err, os.ErrNotExist) { // if dir does not exists try to create it if err := os.MkdirAll(runtime.storageConfig.RunRoot, 0700); err != nil { |