summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKevin Downey <hiredman@thelastcitadel.com>2022-05-15 23:04:30 -0700
committerMatthew Heon <mheon@redhat.com>2022-06-14 13:42:35 -0400
commit2b8a5843ecf3baa2b3bf3d60de9798fd0c562454 (patch)
treecab5dca7e48a48c3f981e6672e462c4002f7d8bb
parent41e6eff62b4a65b73fafb00a796af7409a3e2b4f (diff)
downloadpodman-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.go2
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 {