Fix quota-update issue in LBaaS

Reedip · Reedip · commit 91c4509afe1b · 2016-09-16T10:57:02.000Z
Currently L7Policies cannot be updated( it was missing
in implementation in neutronclient). The same has been
taken care in the current patch.

Also, currently quota doesnt support updating the members
in an LBaaS pool. This patch temporarily removes it, till
it is not confirmed that LBaaS v2 needs to support quotas
for members or not.

Change-Id: I25a54a57debb762a32a280ece8c081fc52365f0f
Closes-Bug: #1624097
diff --git a/openstackclient/common/quota.py b/openstackclient/common/quota.py
@@ -74,8 +74,8 @@
     'rbac_policy': 'rbac-policies',
     'vip': 'vips',
     'subnetpool': 'subnetpools',
-    'member': 'members',
     'healthmonitor': 'health-monitors',
+    'l7policy': 'l7policies',
 }
 
 
diff --git a/openstackclient/tests/unit/common/test_quota.py b/openstackclient/tests/unit/common/test_quota.py
@@ -249,8 +249,8 @@ def test_quota_set_network(self):
             '--rbac-policies', str(network_fakes.QUOTA['rbac_policy']),
             '--ports', str(network_fakes.QUOTA['port']),
             '--vips', str(network_fakes.QUOTA['vip']),
-            '--members', str(network_fakes.QUOTA['member']),
             '--health-monitors', str(network_fakes.QUOTA['healthmonitor']),
+            '--l7policies', str(network_fakes.QUOTA['l7policy']),
             identity_fakes.project_name,
         ]
         verifylist = [
@@ -265,8 +265,8 @@ def test_quota_set_network(self):
             ('rbac_policy', network_fakes.QUOTA['rbac_policy']),
             ('port', network_fakes.QUOTA['port']),
             ('vip', network_fakes.QUOTA['vip']),
-            ('member', network_fakes.QUOTA['member']),
             ('healthmonitor', network_fakes.QUOTA['healthmonitor']),
+            ('l7policy', network_fakes.QUOTA['l7policy']),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
@@ -283,8 +283,8 @@ def test_quota_set_network(self):
             'rbac_policy': network_fakes.QUOTA['rbac_policy'],
             'port': network_fakes.QUOTA['port'],
             'vip': network_fakes.QUOTA['vip'],
-            'member': network_fakes.QUOTA['member'],
             'healthmonitor': network_fakes.QUOTA['healthmonitor'],
+            'l7policy': network_fakes.QUOTA['l7policy'],
         }
         self.network_mock.update_quota.assert_called_once_with(
             identity_fakes.project_id,
diff --git a/openstackclient/tests/unit/network/v2/fakes.py b/openstackclient/tests/unit/network/v2/fakes.py
@@ -34,6 +34,7 @@
     "vip": 10,
     "member": 10,
     "healthmonitor": 10,
+    "l7policy": 5,
 }
 
 

Original file line number	Diff line number	Diff line change
`@@ -74,8 +74,8 @@`
`74`	`74`	`'rbac_policy': 'rbac-policies',`
`75`	`75`	`'vip': 'vips',`
`76`	`76`	`'subnetpool': 'subnetpools',`
`77`		`- 'member': 'members',`
`78`	`77`	`'healthmonitor': 'health-monitors',`
	`78`	`+ 'l7policy': 'l7policies',`
`79`	`79`	`}`
`80`	`80`
`81`	`81`
Original file line number	Diff line number	Diff line change
`@@ -34,6 +34,7 @@`
`34`	`34`	`"vip": 10,`
`35`	`35`	`"member": 10,`
`36`	`36`	`"healthmonitor": 10,`
	`37`	`+ "l7policy": 5,`
`37`	`38`	`}`
`38`	`39`
`39`	`40`