erabo/spiegel_podman
1
0
Fork 0
mirror of https://github.com/podman-container-tools/podman.git synced 2026-09-05 07:07:51 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

volumes: do not pass mount opt as formatter string

Browse source 
otherwise passing a formatter string as an option causes a weird
error message:

$ podman run --mount type=devpts,destination=/dev/pts,%sfoo ...
Error: %!s(MISSING)foo: invalid mount option

Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
This commit is contained in:
Giuseppe Scrivano 2021-10-28 09:53:27 +02:00
parent 3bc449371c
commit e6286fbace
No known key found for this signature in database
GPG key ID: E4730F97F60286ED
1 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
pkg/specgenutil/volumes.go
View file

@ -360,7 +360,7 @@ func getBindMount(args []string) (spec.Mount, error) {
// Since Docker ignores this option so shall we.
continue
default:
return newMount, errors.Wrapf(util.ErrBadMntOption, kv[0])
return newMount, errors.Wrapf(util.ErrBadMntOption, "%s", kv[0])
}
}
@ -460,7 +460,7 @@ func getTmpfsMount(args []string) (spec.Mount, error) {
// Since Docker ignores this option so shall we.
continue
default:
return newMount, errors.Wrapf(util.ErrBadMntOption, kv[0])
return newMount, errors.Wrapf(util.ErrBadMntOption, "%s", kv[0])
}
}
@ -493,7 +493,7 @@ func getDevptsMount(args []string) (spec.Mount, error) {
newMount.Destination = filepath.Clean(kv[1])
setDest = true
default:
return newMount, errors.Wrapf(util.ErrBadMntOption, kv[0])
return newMount, errors.Wrapf(util.ErrBadMntOption, "%s", kv[0])
}
}
@ -573,7 +573,7 @@ func getNamedVolume(args []string) (*specgen.NamedVolume, error) {
// Since Docker ignores this option so shall we.
continue
default:
return nil, errors.Wrapf(util.ErrBadMntOption, kv[0])
return nil, errors.Wrapf(util.ErrBadMntOption, "%s", kv[0])
}
}
@ -624,7 +624,7 @@ func getImageVolume(args []string) (*specgen.ImageVolume, error) {
// Since Docker ignores this option so shall we.
continue
default:
return nil, errors.Wrapf(util.ErrBadMntOption, kv[0])
return nil, errors.Wrapf(util.ErrBadMntOption, "%s", kv[0])
}
}