Skip to content

Commit

Permalink
Merge pull request #7156 from zhangguanzhang/master
Browse files Browse the repository at this point in the history
fix podman system df format error
  • Loading branch information
openshift-merge-robot authored Jul 30, 2020
2 parents ca2bda6 + b4fa2c2 commit 4132b71
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 3 deletions.
9 changes: 9 additions & 0 deletions cmd/podman/system/df.go
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import (
"fmt"
"io"
"os"
"strconv"
"strings"
"text/tabwriter"
"text/template"
Expand Down Expand Up @@ -69,6 +70,14 @@ func printSummary(reports *entities.SystemDfReport, userFormat string) error {

// Images
if len(userFormat) > 0 {
if !strings.HasSuffix(userFormat, `\n`) {
userFormat += `\n`
}
// should be Unquoto from cmd line
userFormat, err := strconv.Unquote(`"` + userFormat + `"`)
if err != nil {
return err
}
format = userFormat
}

Expand Down
7 changes: 4 additions & 3 deletions test/system/320-system-df.bats
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,10 @@ function teardown() {
run_podman run -d -v /myvol2 --name c2 $IMAGE \
sh -c 'while ! test -e /stop; do sleep 0.1;done'

run_podman system df --format '{{ .Type }}:{{ .Total }}:{{ .Active }}--'
# FIXME: if/when #7149 gets fixed, split this into three tests (i.e. test "${lines[0]}", [1], [2] )
is "$output" "Images:1:1--Containers:2:1--Local Volumes:2:1--"
run_podman system df --format '{{ .Type }}:{{ .Total }}:{{ .Active }}'
is "${lines[0]}" "Images:1:1" "system df : Images line"
is "${lines[1]}" "Containers:2:1" "system df : Containers line"
is "${lines[2]}" "Local Volumes:2:1" "system df : Volumes line"

# Try -v. (Grrr. No way to specify individual formats)
#
Expand Down

0 comments on commit 4132b71

Please sign in to comment.