]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Forbid update of HA property of routers
authorAssaf Muller <amuller@redhat.com>
Tue, 7 Oct 2014 19:45:41 +0000 (22:45 +0300)
committerThomas Goirand <thomas@goirand.fr>
Thu, 16 Oct 2014 15:16:21 +0000 (15:16 +0000)
While the HA property is update-able, and resulting router-get
invocations suggest that the router is HA, the migration
itself fails on the agent. This is deceiving and confusing
and should be blocked until the migration itself is fixed
in a future patch.

Change-Id: I4171ab481e3943e0110bd9a300d965bbebe44871
Related-Bug: #1365426
Closes-Bug: #1378525
(cherry picked from commit 1fd7dd99ca7e5e9736200360aa354cada7fb43ff)

neutron/extensions/l3_ext_ha_mode.py

index f8487bb5b9a30cf9047875530529832212ceb901..115a8db81409e7ea73b4fe624607b1724eb30dda 100644 (file)
@@ -21,7 +21,7 @@ from neutron.common import exceptions
 HA_INFO = 'ha'
 EXTENDED_ATTRIBUTES_2_0 = {
     'routers': {
-        HA_INFO: {'allow_post': True, 'allow_put': True,
+        HA_INFO: {'allow_post': True, 'allow_put': False,
                   'default': attributes.ATTR_NOT_SPECIFIED, 'is_visible': True,
                   'enforce_policy': True,
                   'convert_to': attributes.convert_to_boolean_if_not_none}