Allow setting quotas for server groups and server group members

eezhova · eezhova · commit b50c2b6a8847 · 2016-07-27T13:04:36.000+03:00
Adds support of --server-groups and --server-group-members options
to the "quota set" command.

Change-Id: I178d1e267d010be7e908adefcf3b15abdafd9da4
Closes-Bug: #1602223
diff --git a/doc/source/command-objects/quota.rst b/doc/source/command-objects/quota.rst
@@ -25,6 +25,8 @@ Set quotas for project
         [--key-pairs <num-key-pairs>]
         [--properties <num-properties>]
         [--ram <ram-mb>]
+        [--server-groups <num-server-groups>]
+        [--server-group-members <num-server-group-members>]
 
         # Block Storage settings
         [--gigabytes <new-gigabytes>]
@@ -64,6 +66,8 @@ Set quotas for class
         [--key-pairs <num-key-pairs>]
         [--properties <num-properties>]
         [--ram <ram-mb>]
+        [--server-groups <num-server-groups>]
+        [--server-group-members <num-server-group-members>]
 
         # Block Storage settings
         [--gigabytes <new-gigabytes>]
@@ -108,6 +112,14 @@ Set quotas for class
 
     New value for the injected-file-size quota
 
+.. option:: --server-groups <new-server-groups>
+
+    New value for the server-groups quota
+
+.. option:: --server-group-members <new-server-group-members>
+
+    New value for the server-group-members quota
+
 .. option:: --floating-ips <new-floating-ips>
 
     New value for the floating-ips quota
diff --git a/openstackclient/common/quota.py b/openstackclient/common/quota.py
@@ -38,6 +38,8 @@
     'key_pairs': 'key-pairs',
     'metadata_items': 'properties',
     'ram': 'ram',
+    'server_groups': 'server-groups',
+    'server_group_members': 'server-group-members',
 }
 
 VOLUME_QUOTAS = {
diff --git a/openstackclient/tests/common/test_quota.py b/openstackclient/tests/common/test_quota.py
@@ -115,6 +115,8 @@ def test_quota_set(self):
             '--properties', str(compute_fakes.property_num),
             '--secgroup-rules', str(compute_fakes.secgroup_rule_num),
             '--secgroups', str(compute_fakes.secgroup_num),
+            '--server-groups', str(compute_fakes.servgroup_num),
+            '--server-group-members', str(compute_fakes.servgroup_members_num),
             identity_fakes.project_name,
         ]
         verifylist = [
@@ -131,6 +133,8 @@ def test_quota_set(self):
             ('metadata_items', compute_fakes.property_num),
             ('security_group_rules', compute_fakes.secgroup_rule_num),
             ('security_groups', compute_fakes.secgroup_num),
+            ('server_groups', compute_fakes.servgroup_num),
+            ('server_group_members', compute_fakes.servgroup_members_num),
             ('project', identity_fakes.project_name),
         ]
 
@@ -153,6 +157,8 @@ def test_quota_set(self):
             'metadata_items': compute_fakes.property_num,
             'security_group_rules': compute_fakes.secgroup_rule_num,
             'security_groups': compute_fakes.secgroup_num,
+            'server_groups': compute_fakes.servgroup_num,
+            'server_group_members': compute_fakes.servgroup_members_num,
         }
 
         self.quotas_mock.update.assert_called_with(
diff --git a/openstackclient/tests/compute/v2/fakes.py b/openstackclient/tests/compute/v2/fakes.py
@@ -36,6 +36,8 @@
 property_num = 128
 secgroup_rule_num = 20
 secgroup_num = 10
+servgroup_num = 10
+servgroup_members_num = 10
 project_name = 'project_test'
 QUOTA = {
     'project': project_name,
@@ -51,6 +53,8 @@
     'properties': property_num,
     'secgroup_rules': secgroup_rule_num,
     'secgroups': secgroup_num,
+    'server-groups': servgroup_num,
+    'server-group-members': servgroup_members_num
 }
 
 QUOTA_columns = tuple(sorted(QUOTA))
diff --git a/releasenotes/notes/add-server-group-quotas-b67fcba98619f0c9.yaml b/releasenotes/notes/add-server-group-quotas-b67fcba98619f0c9.yaml
@@ -0,0 +1,6 @@
+---
+features:
+  - Added support of --server-groups --server-group-members options
+    to ``quota set`` command.
+
+    [Bug `1602223 <https://bugs.launchpad.net/python-openstackclient/+bug/1602223>`_]

Original file line number	Diff line number	Diff line change
`@@ -38,6 +38,8 @@`
`38`	`38`	`'key_pairs': 'key-pairs',`
`39`	`39`	`'metadata_items': 'properties',`
`40`	`40`	`'ram': 'ram',`
	`41`	`+ 'server_groups': 'server-groups',`
	`42`	`+ 'server_group_members': 'server-group-members',`
`41`	`43`	`}`
`42`	`44`
`43`	`45`	`VOLUME_QUOTAS = {`