From c244c347b1e326d57a456424f7821ccf4cdb1bde Mon Sep 17 00:00:00 2001
From: Qi Wang <qiwan@redhat.com>
Date: Tue, 16 Jul 2019 16:17:59 -0400
Subject: podman-remote make --size optional in ps

Close #3578 Add `size` field to PsOpts in podman remote to receive size as an option.

Signed-off-by: Qi Wang <qiwan@redhat.com>
---
 pkg/adapter/containers_remote.go |  1 +
 pkg/varlinkapi/containers.go     | 54 +++++++++++++++++++++-------------------
 pkg/varlinkapi/util.go           |  2 +-
 3 files changed, 30 insertions(+), 27 deletions(-)

(limited to 'pkg')

diff --git a/pkg/adapter/containers_remote.go b/pkg/adapter/containers_remote.go
index 5836d0788..c816fd980 100644
--- a/pkg/adapter/containers_remote.go
+++ b/pkg/adapter/containers_remote.go
@@ -493,6 +493,7 @@ func (r *LocalRuntime) Ps(c *cliconfig.PsValues, opts shared.PsOptions) ([]share
 		NoTrunc: &c.NoTrunct,
 		Pod:     &c.Pod,
 		Quiet:   &c.Quiet,
+		Size:    &c.Size,
 		Sort:    &c.Sort,
 		Sync:    &c.Sync,
 	}
diff --git a/pkg/varlinkapi/containers.go b/pkg/varlinkapi/containers.go
index 6855a7231..83743351c 100644
--- a/pkg/varlinkapi/containers.go
+++ b/pkg/varlinkapi/containers.go
@@ -66,32 +66,34 @@ func (i *LibpodAPI) Ps(call iopodman.VarlinkCall, opts iopodman.PsOpts) error {
 
 	for _, ctr := range psContainerOutputs {
 		container := iopodman.PsContainer{
-			Id:         ctr.ID,
-			Image:      ctr.Image,
-			Command:    ctr.Command,
-			Created:    ctr.Created,
-			Ports:      ctr.Ports,
-			Names:      ctr.Names,
-			IsInfra:    ctr.IsInfra,
-			Status:     ctr.Status,
-			State:      ctr.State.String(),
-			PidNum:     int64(ctr.Pid),
-			RootFsSize: ctr.Size.RootFsSize,
-			RwSize:     ctr.Size.RwSize,
-			Pod:        ctr.Pod,
-			CreatedAt:  ctr.CreatedAt.Format(time.RFC3339Nano),
-			ExitedAt:   ctr.ExitedAt.Format(time.RFC3339Nano),
-			StartedAt:  ctr.StartedAt.Format(time.RFC3339Nano),
-			Labels:     ctr.Labels,
-			NsPid:      ctr.PID,
-			Cgroup:     ctr.Cgroup,
-			Ipc:        ctr.Cgroup,
-			Mnt:        ctr.MNT,
-			Net:        ctr.NET,
-			PidNs:      ctr.PIDNS,
-			User:       ctr.User,
-			Uts:        ctr.UTS,
-			Mounts:     ctr.Mounts,
+			Id:        ctr.ID,
+			Image:     ctr.Image,
+			Command:   ctr.Command,
+			Created:   ctr.Created,
+			Ports:     ctr.Ports,
+			Names:     ctr.Names,
+			IsInfra:   ctr.IsInfra,
+			Status:    ctr.Status,
+			State:     ctr.State.String(),
+			PidNum:    int64(ctr.Pid),
+			Pod:       ctr.Pod,
+			CreatedAt: ctr.CreatedAt.Format(time.RFC3339Nano),
+			ExitedAt:  ctr.ExitedAt.Format(time.RFC3339Nano),
+			StartedAt: ctr.StartedAt.Format(time.RFC3339Nano),
+			Labels:    ctr.Labels,
+			NsPid:     ctr.PID,
+			Cgroup:    ctr.Cgroup,
+			Ipc:       ctr.Cgroup,
+			Mnt:       ctr.MNT,
+			Net:       ctr.NET,
+			PidNs:     ctr.PIDNS,
+			User:      ctr.User,
+			Uts:       ctr.UTS,
+			Mounts:    ctr.Mounts,
+		}
+		if ctr.Size != nil {
+			container.RootFsSize = ctr.Size.RootFsSize
+			container.RwSize = ctr.Size.RwSize
 		}
 		containers = append(containers, container)
 	}
diff --git a/pkg/varlinkapi/util.go b/pkg/varlinkapi/util.go
index e8f74e6aa..d3a41f7ab 100644
--- a/pkg/varlinkapi/util.go
+++ b/pkg/varlinkapi/util.go
@@ -191,7 +191,7 @@ func makePsOpts(inOpts iopodman.PsOpts) shared.PsOptions {
 		Latest:    derefBool(inOpts.Latest),
 		NoTrunc:   derefBool(inOpts.NoTrunc),
 		Pod:       derefBool(inOpts.Pod),
-		Size:      true,
+		Size:      derefBool(inOpts.Size),
 		Sort:      derefString(inOpts.Sort),
 		Namespace: true,
 		Sync:      derefBool(inOpts.Sync),
-- 
cgit v1.2.3-54-g00ecf