]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
nvp plugin rxtx_factor readonly update port
authorAaron Rosen <arosen@nicira.com>
Wed, 17 Jul 2013 21:56:16 +0000 (14:56 -0700)
committerAaron Rosen <arosen@nicira.com>
Wed, 17 Jul 2013 22:02:55 +0000 (15:02 -0700)
The following change done in nova 7de916 started passing in the
rxtx_factor on update port rather than just on create_port which
is what we only originally supported. Therefore currently when
booting a vm and specifying --nic port-id it will fail to boot with:
Cannot update read-only attribute rxtx_factor. This patch is a work
around to allow the rxtx_factor value to be passed in on update port even
though we ignore the value. Later we'll implement updating rxtx_factor on
ports but this is a good work around for now.

Fixes bug: 1202406

Change-Id: Iedd488b7bdc9b1a1317000d249f03b0eafbea419

neutron/plugins/nicira/extensions/nvp_qos.py

index 5891be6ecf1651705244d2863b05739bdbf4018e..ade87a5ac7316be61d517ff75abf03c1cdbfed9e 100644 (file)
@@ -127,7 +127,9 @@ RXTX_FACTOR = 'rxtx_factor'
 EXTENDED_ATTRIBUTES_2_0 = {
     'ports': {
         RXTX_FACTOR: {'allow_post': True,
-                      'allow_put': False,
+                      # FIXME(arosen): the nvp plugin currently does not
+                      # implement updating rxtx factor on port.
+                      'allow_put': True,
                       'is_visible': False,
                       'default': 1,
                       'enforce_policy': True,