x/quark
Code Issues Proposed changes

Explicitly define allow_put in neutron extensions

Browse Source 
allow_put is not explicitly defined in quark
extensions to existing neutron resources. Defining
allow_put in the resource definitions will now
throw the proper 400 error.

Change-Id: Idc427271c47155ea8ce3310e74028fd64c2a83d2
JIRA:NCP-1989
Closes-Bug: 1597912
This commit is contained in:
Kyle Haley
2016-06-30 13:46:21 -07:00
parent 15671d45ef
commit aa060e2154
3 changed files with 12 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
quark/api/extensions/networks_quark.py
View File

@@ -20,11 +20,12 @@ RESOURCE_NAME = "network"
RESOURCE_COLLECTION = RESOURCE_NAME + "s" RESOURCE_COLLECTION = RESOURCE_NAME + "s"
EXTENDED_ATTRIBUTES_2_0 = { EXTENDED_ATTRIBUTES_2_0 = {
RESOURCE_COLLECTION: { RESOURCE_COLLECTION: {
"ipam_strategy": {"allow_post": True, "is_visible": True, "ipam_strategy": {"allow_post": True, "allow_put": False,
"default": False}, "is_visible": True, "default": False},
"network_plugin": {"allow_post": True, "is_visible": False, "network_plugin": {"allow_post": True, "allow_put": False,
"default": False}, "is_visible": False, "default": False},
"id": {"allow_post": True, "is_visible": True, "default": False}}} "id": {"allow_post": True, "allow_put": False,
"is_visible": True, "default": False}}}
class Networks_quark(extensions.ExtensionDescriptor): class Networks_quark(extensions.ExtensionDescriptor):

8
quark/api/extensions/ports_quark.py
View File

@@ -21,10 +21,10 @@ RESOURCE_NAME = "port"
RESOURCE_COLLECTION = RESOURCE_NAME + "s" RESOURCE_COLLECTION = RESOURCE_NAME + "s"
EXTENDED_ATTRIBUTES_2_0 = { EXTENDED_ATTRIBUTES_2_0 = {
RESOURCE_COLLECTION: { RESOURCE_COLLECTION: {
"network_id": {"allow_post": True, "default": '', "network_id": {"allow_post": True, "allow_put": False,
"is_visible": True}, "default": '', "is_visible": True},
"tenant_id": {"allow_post": True, "default": '', "tenant_id": {"allow_post": True, "allow_put": False,
"is_visible": True}, "default": '', "is_visible": True},
"segment_id": {"allow_post": True, "default": False}, "segment_id": {"allow_post": True, "default": False},
"bridge": {'allow_post': False, 'allow_put': False, "bridge": {'allow_post': False, 'allow_put': False,
'default': False, 'is_visible': True}, 'default': False, 'is_visible': True},

3
quark/api/extensions/subnets_quark.py
View File

@@ -17,7 +17,8 @@ from neutron.api import extensions
EXTENDED_ATTRIBUTES_2_0 = { EXTENDED_ATTRIBUTES_2_0 = {
"subnets": { "subnets": {
"segment_id": {"allow_post": True, "default": None}, "segment_id": {"allow_post": True, "allow_put": False,
"default": None},
"ip_policy_id": {"allow_post": False, "allow_put": False, "ip_policy_id": {"allow_post": False, "allow_put": False,
"default": None, "is_visible": True} "default": None, "is_visible": True}
} }