openstack/openstacksdk
Code Issues Proposed changes

Remove default values of router's is_ha and is_distributed arguments

Browse Source 
Those attributes of Neutron's router are by default blocked by
policy.json for regular users. So in such case even if router is ha
or is distributed, regular user would get False values for both
parameters.
This patch changes it by removing default value set to False for those
attributes. Now in such case both those parameters will be set to
None which should be less confusing for users.

Change-Id: I6908000bfc542478775a5f873e4660a9206ce38a
Closes-Bug: #1689510
This commit is contained in:
Sławek Kapłoński
2018-05-10 16:36:55 +02:00
committed by Slawek Kaplonski
parent daf889a315
commit 9242148cd9
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
openstack/network/v2/router.py
View File

@@ -59,11 +59,11 @@ class Router(resource.Resource, tag.TagMixin):
#: or down ``False``. *Type: bool*
is_admin_state_up = resource.Body('admin_state_up', type=bool)
#: The distributed state of the router, which is distributed ``True``
#: or not ``False``. *Type: bool* *Default: False*
is_distributed = resource.Body('distributed', type=bool, default=False)
#: or not ``False``. *Type: bool*
is_distributed = resource.Body('distributed', type=bool)
#: The highly-available state of the router, which is highly available
#: ``True`` or not ``False``. *Type: bool* *Default: False*
is_ha = resource.Body('ha', type=bool, default=False)
#: ``True`` or not ``False``. *Type: bool*
is_ha = resource.Body('ha', type=bool)
#: The router name.
name = resource.Body('name')
#: The ID of the project this router is associated with.