Fix creating a private flavor with ID auto

Jens Rosenboom · Jens Rosenboom · commit d8749f9148f2 · 2017-01-05T15:58:47.000+01:00
When a private flavor is created with ID auto (=default) and a project
is specified for it, instead of trying to add the project to the flavor
called "auto" the ID of the newly created project should be used.

Change-Id: I19f7a0ec26bd5d147f00ecba3312240e3601567e
Closes-Bug: 1654221
diff --git a/openstackclient/compute/v2/flavor.py b/openstackclient/compute/v2/flavor.py
@@ -167,7 +167,7 @@ def take_action(self, parsed_args):
                     parsed_args.project_domain,
                 ).id
                 compute_client.flavor_access.add_tenant_access(
-                    parsed_args.id, project_id)
+                    flavor.id, project_id)
             except Exception as e:
                 msg = _("Failed to add project %(project)s access to "
                         "flavor: %(e)s")
diff --git a/openstackclient/tests/unit/compute/v2/test_flavor.py b/openstackclient/tests/unit/compute/v2/test_flavor.py
@@ -160,7 +160,7 @@ def test_flavor_create_other_options(self):
 
         self.flavor.is_public = False
         arglist = [
-            '--id', self.flavor.id,
+            '--id', 'auto',
             '--ram', str(self.flavor.ram),
             '--disk', str(self.flavor.disk),
             '--ephemeral', str(self.flavor.ephemeral),
@@ -174,7 +174,6 @@ def test_flavor_create_other_options(self):
             self.flavor.name,
         ]
         verifylist = [
-            ('id', self.flavor.id),
             ('ram', self.flavor.ram),
             ('disk', self.flavor.disk),
             ('ephemeral', self.flavor.ephemeral),
@@ -193,7 +192,7 @@ def test_flavor_create_other_options(self):
             self.flavor.ram,
             self.flavor.vcpus,
             self.flavor.disk,
-            self.flavor.id,
+            'auto',
             self.flavor.ephemeral,
             self.flavor.swap,
             self.flavor.rxtx_factor,
diff --git a/releasenotes/notes/bug-1654221-a564ab75a6afc332.yaml b/releasenotes/notes/bug-1654221-a564ab75a6afc332.yaml
@@ -0,0 +1,6 @@
+---
+fixes:
+  - |
+    Fix ``--project`` option for ``flavor create`` command when the
+    ID for the new flavor is auto generated.
+    [Bug `1654221 <https://bugs.launchpad.net/bugs/1654221>`_]
