summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJhon Honce <jhonce@redhat.com>2018-11-28 15:44:46 -0700
committerJhon Honce <jhonce@redhat.com>2018-11-28 15:44:46 -0700
commit57f7b794002abb5f89d037eea30c964c92aca449 (patch)
treeaa8c4407ee378a1ef6d95efdd51e611bb86d54b2
parent3af62f620a4d4ed21db58e71f9ba7ace242ec742 (diff)
downloadpodman-57f7b794002abb5f89d037eea30c964c92aca449.tar.gz
podman-57f7b794002abb5f89d037eea30c964c92aca449.tar.bz2
podman-57f7b794002abb5f89d037eea30c964c92aca449.zip
Only include container SizeRootFs when requested
* API always returns value, so we remove it if not asked for Fixes #1876 Signed-off-by: Jhon Honce <jhonce@redhat.com>
-rw-r--r--contrib/python/pypodman/pypodman/lib/actions/inspect_action.py14
1 files changed, 10 insertions, 4 deletions
diff --git a/contrib/python/pypodman/pypodman/lib/actions/inspect_action.py b/contrib/python/pypodman/pypodman/lib/actions/inspect_action.py
index 514b4702a..a581e7e4e 100644
--- a/contrib/python/pypodman/pypodman/lib/actions/inspect_action.py
+++ b/contrib/python/pypodman/pypodman/lib/actions/inspect_action.py
@@ -23,9 +23,9 @@ class Inspect(AbstractActionBase):
help='Type of object to inspect',
)
parser.add_argument(
- 'size',
+ '--size',
action='store_true',
- default=True,
+ default=False,
help='Display the total file size if the type is a container.'
' Always True.')
parser.add_argument(
@@ -59,7 +59,7 @@ class Inspect(AbstractActionBase):
def inspect(self):
"""Inspect provided podman objects."""
- output = {}
+ output = []
try:
for ident in self._args.objects:
obj = None
@@ -78,7 +78,13 @@ class Inspect(AbstractActionBase):
msg = 'Object "{}" not found'.format(ident)
print(msg, file=sys.stderr, flush=True)
else:
- output.update(obj._asdict())
+ fields = obj._asdict()
+ if not self._args.size:
+ try:
+ del fields['sizerootfs']
+ except KeyError:
+ pass
+ output.append(fields)
except podman.ErrorOccurred as e:
sys.stdout.flush()
print(