Added 'openstack image set --visibility'

M V P Nitesh · Dean Troyer · commit 3468ea1ca429 · 2017-06-30T20:35:01.000Z
This feature will allow image owners to share images across multiple
tenants/projects without explicitly creating members individually
through the glance API V2. “Community images” will not appear iu
user's default image listings

Change-Id: Ic02bf44cca5d6d793d68a8737d3cfe3f44407d88
Closes-Bug: #1626837
diff --git a/doc/source/cli/command-objects/image.rst b/doc/source/cli/command-objects/image.rst
@@ -57,7 +57,7 @@ Create/upload an image
         [--force]
         [--checksum <checksum>]
         [--protected | --unprotected]
-        [--public | --private]
+        [--public | --private | --community | --shared]
         [--property <key=value> [...] ]
         [--tag <tag> [...] ]
         [--project <project> [--project-domain <project-domain>]]
@@ -143,6 +143,14 @@ Create/upload an image
 
     Image is inaccessible to the public (default)
 
+.. option:: --community
+
+    Image is accessible to the community
+
+.. option:: --shared
+
+    Image can be shared
+
 .. option:: --property <key=value>
 
     Set a property on this image (repeat for multiple values)
@@ -310,7 +318,7 @@ Set image properties
         [--disk-format <disk-format>]
         [--size <size>]
         [--protected | --unprotected]
-        [--public | --private]
+        [--public | --private | --community | --shared]
         [--store <store>]
         [--location <image-url>]
         [--copy-from <image-url>]
@@ -376,6 +384,14 @@ Set image properties
 
     Image is inaccessible to the public (default)
 
+.. option:: --community
+
+    Image is accessible to the community
+
+.. option:: --shared
+
+    Image can be shared
+
 .. option:: --store <store>
 
     Upload image to this store
diff --git a/openstackclient/image/v2/image.py b/openstackclient/image/v2/image.py
@@ -205,6 +205,16 @@ def get_parser(self, prog_name):
             action="store_true",
             help=_("Image is inaccessible to the public (default)"),
         )
+        public_group.add_argument(
+            "--community",
+            action="store_true",
+            help=_("Image is accessible to the community"),
+        )
+        public_group.add_argument(
+            "--shared",
+            action="store_true",
+            help=_("Image can be shared"),
+        )
         parser.add_argument(
             "--property",
             dest="properties",
@@ -260,7 +270,7 @@ def take_action(self, parsed_args):
         kwargs = {}
         copy_attrs = ('name', 'id',
                       'container_format', 'disk_format',
-                      'min_disk', 'min_ram', 'tags')
+                      'min_disk', 'min_ram', 'tags', 'visibility')
         for attr in copy_attrs:
             if attr in parsed_args:
                 val = getattr(parsed_args, attr, None)
@@ -288,7 +298,10 @@ def take_action(self, parsed_args):
             kwargs['visibility'] = 'public'
         if parsed_args.private:
             kwargs['visibility'] = 'private'
-
+        if parsed_args.community:
+            kwargs['visibility'] = 'community'
+        if parsed_args.shared:
+            kwargs['visibility'] = 'shared'
         # Handle deprecated --owner option
         project_arg = parsed_args.project
         if parsed_args.owner:
@@ -698,6 +711,16 @@ def get_parser(self, prog_name):
             action="store_true",
             help=_("Image is inaccessible to the public (default)"),
         )
+        public_group.add_argument(
+            "--community",
+            action="store_true",
+            help=_("Image is accessible to the community"),
+        )
+        public_group.add_argument(
+            "--shared",
+            action="store_true",
+            help=_("Image can be shared"),
+        )
         parser.add_argument(
             "--property",
             dest="properties",
@@ -817,7 +840,7 @@ def take_action(self, parsed_args):
         copy_attrs = ('architecture', 'container_format', 'disk_format',
                       'file', 'instance_id', 'kernel_id', 'locations',
                       'min_disk', 'min_ram', 'name', 'os_distro', 'os_version',
-                      'prefix', 'progress', 'ramdisk_id', 'tags')
+                      'prefix', 'progress', 'ramdisk_id', 'tags', 'visibility')
         for attr in copy_attrs:
             if attr in parsed_args:
                 val = getattr(parsed_args, attr, None)
@@ -845,7 +868,10 @@ def take_action(self, parsed_args):
             kwargs['visibility'] = 'public'
         if parsed_args.private:
             kwargs['visibility'] = 'private'
-
+        if parsed_args.community:
+            kwargs['visibility'] = 'community'
+        if parsed_args.shared:
+            kwargs['visibility'] = 'shared'
         # Handle deprecated --owner option
         project_arg = parsed_args.project
         if parsed_args.owner:
diff --git a/releasenotes/notes/image_set_visibility-babf4ff2f687d465.yaml b/releasenotes/notes/image_set_visibility-babf4ff2f687d465.yaml
@@ -0,0 +1,7 @@
+---
+fixes:
+  - Add ``--community`` and ``--shared`` options to the ``image create`` and
+    ``image set`` commands to allow image owners to share images across
+    multiple projects without explicitly creating image members.
+    “Community images” will not appear in user’s default
+    image listings.
