image list: Add Checksum column

msabramo · msabramo · commit 8bbf30498eb3 · 2016-07-11T20:31:13.000-07:00
The checksum could potentially be useful for spotting duplicated images or
checking if images with the same name are identical or different.

Closes-Bug: #1602073

Change-Id: Ia0c41970c846d550de14297e18bc738e847e5a3b
diff --git a/openstackclient/image/v1/image.py b/openstackclient/image/v1/image.py
@@ -378,6 +378,7 @@ def take_action(self, parsed_args):
                 'Disk Format',
                 'Container Format',
                 'Size',
+                'Checksum',
                 'Status',
                 'is_public',
                 'protected',
@@ -390,6 +391,7 @@ def take_action(self, parsed_args):
                 'Disk Format',
                 'Container Format',
                 'Size',
+                'Checksum',
                 'Status',
                 'Visibility',
                 'Protected',
diff --git a/openstackclient/image/v2/image.py b/openstackclient/image/v2/image.py
@@ -490,6 +490,7 @@ def take_action(self, parsed_args):
                 'Disk Format',
                 'Container Format',
                 'Size',
+                'Checksum',
                 'Status',
                 'visibility',
                 'protected',
@@ -502,6 +503,7 @@ def take_action(self, parsed_args):
                 'Disk Format',
                 'Container Format',
                 'Size',
+                'Checksum',
                 'Status',
                 'Visibility',
                 'Protected',
diff --git a/openstackclient/tests/image/v1/test_image.py b/openstackclient/tests/image/v1/test_image.py
@@ -363,6 +363,7 @@ def test_image_list_long_option(self):
             'Disk Format',
             'Container Format',
             'Size',
+            'Checksum',
             'Status',
             'Visibility',
             'Protected',
@@ -378,6 +379,7 @@ def test_image_list_long_option(self):
             '',
             '',
             '',
+            '',
             'public',
             False,
             image_fakes.image_owner,
diff --git a/openstackclient/tests/image/v2/test_image.py b/openstackclient/tests/image/v2/test_image.py
@@ -643,6 +643,7 @@ def test_image_list_long_option(self):
             'Disk Format',
             'Container Format',
             'Size',
+            'Checksum',
             'Status',
             'Visibility',
             'Protected',
@@ -658,6 +659,7 @@ def test_image_list_long_option(self):
             '',
             '',
             '',
+            '',
             self._image.visibility,
             self._image.protected,
             self._image.owner,
diff --git a/releasenotes/notes/bug_1602073-5deb58deeafbc8be.yaml b/releasenotes/notes/bug_1602073-5deb58deeafbc8be.yaml
@@ -0,0 +1,4 @@
+---
+features:
+  - Add "Checksum" column to output of "image list --long"
+    [Bug `1602073 <https://bugs.launchpad.net/bugs/1602073>`_]
