You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
So i was discovering rootless options in containers/podman#18375 podman issue and when i discovered rootless mode, i of course wanted to change runroot path, but looking through code, it seems like theres no code related to changing runroot in rootless mode...
The related function is getRootlessStorageOpts, which feeds output from underlying functions (and as such, getRootlessRuntimeDirIsolated), which doesn't use runroot from configuration. opts.RunRoot = rootlessRuntime changes it's runroot to be used from the getRootlessRuntimeDirIsolated, and later on never updated to be used from system env if it was overriden by conf files, like it is done with opts.GraphRoot, err = expandEnvPath(systemOpts.RootlessStoragePath, rootlessUID).
So it seems like there should be code related if runRoot is being overriden from configuration file, it should be placed instead of defaults.
The text was updated successfully, but these errors were encountered:
So i was discovering rootless options in containers/podman#18375 podman issue and when i discovered rootless mode, i of course wanted to change runroot path, but looking through code, it seems like theres no code related to changing runroot in rootless mode...
The related function is
getRootlessStorageOpts
, which feeds output from underlying functions (and as such,getRootlessRuntimeDirIsolated
), which doesn't use runroot from configuration.opts.RunRoot = rootlessRuntime
changes it's runroot to be used from thegetRootlessRuntimeDirIsolated
, and later on never updated to be used from system env if it was overriden by conf files, like it is done withopts.GraphRoot, err = expandEnvPath(systemOpts.RootlessStoragePath, rootlessUID)
.So it seems like there should be code related if runRoot is being overriden from configuration file, it should be placed instead of defaults.
The text was updated successfully, but these errors were encountered: