From c5e443ae5223ae2a7f91294587212ab45792907e Mon Sep 17 00:00:00 2001
From: Kevin Hannon <kehannon@redhat.com>
Date: Fri, 8 Sep 2023 09:13:06 -0400
Subject: [PATCH] adding imagefsinfo endpoint for both container and image
 filesystems

Signed-off-by: Kevin Hannon <kehannon@redhat.com>
---
 cmd/crictl/image.go | 47 +++++++++++++++++++++++++++------------------
 1 file changed, 28 insertions(+), 19 deletions(-)

diff --git a/cmd/crictl/image.go b/cmd/crictl/image.go
index 3568a81e3e..5830550b46 100644
--- a/cmd/crictl/image.go
+++ b/cmd/crictl/image.go
@@ -494,28 +494,37 @@ var imageFsInfoCommand = &cli.Command{
 		if err != nil {
 			return fmt.Errorf("image filesystem info request: %w", err)
 		}
-		for _, info := range r.ImageFilesystems {
-			status, err := protobufObjectToJSON(info)
-			if err != nil {
-				return fmt.Errorf("marshal image filesystem info to json: %w", err)
-			}
+		fileSystemReport := func(filesystem []*pb.FilesystemUsage) error {
+			for _, info := range filesystem {
+				status, err := protobufObjectToJSON(info)
+				if err != nil {
+					return fmt.Errorf("marshal filesystem info to json: %w", err)
+				}
 
-			switch output {
-			case "json", "yaml", "go-template":
-				if err := outputStatusInfo(status, nil, output, tmplStr); err != nil {
-					return fmt.Errorf("output image filesystem info: %w", err)
+				switch output {
+				case "json", "yaml", "go-template":
+					if err := outputStatusInfo(status, nil, output, tmplStr); err != nil {
+						return fmt.Errorf("output filesystem info: %w", err)
+					}
+					continue
+				case "table": // table output is after this switch block
+				default:
+					return fmt.Errorf("output option cannot be %s", output)
 				}
-				continue
-			case "table": // table output is after this switch block
-			default:
-				return fmt.Errorf("output option cannot be %s", output)
-			}
 
-			// otherwise output in table format
-			fmt.Printf("TimeStamp: %d\n", info.Timestamp)
-			fmt.Printf("Disk: %s\n", units.HumanSize(float64(info.UsedBytes.GetValue())))
-			fmt.Printf("Inodes: %d\n", info.InodesUsed.GetValue())
-			fmt.Printf("Mountpoint: %s\n", info.FsId.Mountpoint)
+				// otherwise output in table format
+				fmt.Printf("TimeStamp: %d\n", info.Timestamp)
+				fmt.Printf("Disk: %s\n", units.HumanSize(float64(info.UsedBytes.GetValue())))
+				fmt.Printf("Inodes: %d\n", info.InodesUsed.GetValue())
+				fmt.Printf("Mountpoint: %s\n", info.FsId.Mountpoint)
+			}
+			return nil
+		}
+		if err := fileSystemReport(r.ImageFilesystems); err != nil {
+			return err
+		}
+		if err := fileSystemReport(r.ContainerFilesystems); err != nil {
+			return err
 		}
 
 		return nil