summaryrefslogtreecommitdiff
path: root/cmd/podman
diff options
context:
space:
mode:
authorValentin Rothberg <rothberg@redhat.com>2020-09-22 10:54:23 +0200
committerValentin Rothberg <rothberg@redhat.com>2020-09-22 17:04:38 +0200
commit78afe67e00a78b98483a9bb1cb55f58dffc4c001 (patch)
tree87fcd9b6be2863d79c25fbebdca950aab9285178 /cmd/podman
parentfffcc25d8dfab8c6059c229177286f462d909b8d (diff)
downloadpodman-78afe67e00a78b98483a9bb1cb55f58dffc4c001.tar.gz
podman-78afe67e00a78b98483a9bb1cb55f58dffc4c001.tar.bz2
podman-78afe67e00a78b98483a9bb1cb55f58dffc4c001.zip
podman save: fix redirect of multi-images
Fix a bug when saving multi-image archives where the progress bars from copying images was accidentally written to the archive and hence corrupted it. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
Diffstat (limited to 'cmd/podman')
-rw-r--r--cmd/podman/images/save.go7
1 files changed, 1 insertions, 6 deletions
diff --git a/cmd/podman/images/save.go b/cmd/podman/images/save.go
index c57f61221..b164a2534 100644
--- a/cmd/podman/images/save.go
+++ b/cmd/podman/images/save.go
@@ -94,6 +94,7 @@ func save(cmd *cobra.Command, args []string) (finalErr error) {
return errors.Errorf("--compress can only be set when --format is either 'oci-dir' or 'docker-dir'")
}
if len(saveOpts.Output) == 0 {
+ saveOpts.Quiet = true
fi := os.Stdout
if terminal.IsTerminal(int(fi.Fd())) {
return errors.Errorf("refusing to save to terminal. Use -o flag or redirect")
@@ -122,12 +123,6 @@ func save(cmd *cobra.Command, args []string) (finalErr error) {
tags = args[1:]
}
- // Decide whether c/image's progress bars should use stderr or stdout.
- // If the output is set of stdout, any log message there would corrupt
- // the tarfile.
- if saveOpts.Output == os.Stdout.Name() {
- saveOpts.Quiet = true
- }
err := registry.ImageEngine().Save(context.Background(), args[0], tags, saveOpts)
if err == nil {
succeeded = true