diff options
| author | Daniel J Walsh <dwalsh@redhat.com> | 2021-05-19 08:22:13 -0400 |
|---|---|---|
| committer | Daniel J Walsh <dwalsh@redhat.com> | 2021-05-19 15:35:59 -0400 |
| commit | 55f00bac02fcde7fbe960a9a80131dbc72630b5b (patch) | |
| tree | b1d1c8b9357acd4d065de3cd5f8d98f3266c0150 /pkg/domain | |
| parent | 4c756268e6d0dbe90e61a59d9644e34a0f2d5f36 (diff) | |
| download | podman-55f00bac02fcde7fbe960a9a80131dbc72630b5b.tar.gz podman-55f00bac02fcde7fbe960a9a80131dbc72630b5b.tar.bz2 podman-55f00bac02fcde7fbe960a9a80131dbc72630b5b.zip | |
Clear the storage-options from the graphdriver if users specifies --root
Fixes: https://github.com/containers/podman/issues/10393
Currently if a user specifies a --root flag to override the location of
the container storage, we still enforce the storage-opts from
storage.conf. This causes issues with people trying to intereact with
the additional stores feature, and then forces them to use the obscure
--storage-opt="" option. I belive this should be the default and we
already do this when the user specifies the --storage-driver option.
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Diffstat (limited to 'pkg/domain')
| -rw-r--r-- | pkg/domain/infra/runtime_libpod.go | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/pkg/domain/infra/runtime_libpod.go b/pkg/domain/infra/runtime_libpod.go index a98c9168a..5cbee2e76 100644 --- a/pkg/domain/infra/runtime_libpod.go +++ b/pkg/domain/infra/runtime_libpod.go @@ -129,6 +129,7 @@ func getRuntime(ctx context.Context, fs *flag.FlagSet, opts *engineOpts) (*libpo if fs.Changed("root") { storageSet = true storageOpts.GraphRoot = cfg.Engine.StaticDir + storageOpts.GraphDriverOptions = []string{} } if fs.Changed("runroot") { storageSet = true |