Merge "Ensure nvp resources are tagged with an up-to-date version of Neutron"
This commit is contained in:
commit
b3e68ed5f6
@ -35,6 +35,7 @@ from neutron.openstack.common import log
|
||||
from neutron.plugins.nicira.common import (
|
||||
exceptions as nvp_exc)
|
||||
from neutron.plugins.nicira import NvpApiClient
|
||||
from neutron.version import version_info
|
||||
|
||||
|
||||
LOG = log.getLogger(__name__)
|
||||
@ -55,7 +56,7 @@ LROUTERNAT_RESOURCE = "nat/lrouter"
|
||||
LQUEUE_RESOURCE = "lqueue"
|
||||
GWSERVICE_RESOURCE = "gateway-service"
|
||||
# Current neutron version
|
||||
NEUTRON_VERSION = "2013.1"
|
||||
NEUTRON_VERSION = version_info.release_string()
|
||||
# Other constants for NVP resource
|
||||
MAX_DISPLAY_NAME_LEN = 40
|
||||
# Constants for NAT rules
|
||||
|
@ -376,7 +376,7 @@ class TestNvplibExplicitLRouters(NvplibTestCase):
|
||||
|
||||
router = {'display_name': router_name,
|
||||
'uuid': router_id,
|
||||
'tags': [{'scope': 'quantum', 'tag': '2013.1'},
|
||||
'tags': [{'scope': 'quantum', 'tag': nvplib.NEUTRON_VERSION},
|
||||
{'scope': 'os_tid', 'tag': '%s' % tenant_id}],
|
||||
'distributed': False,
|
||||
'routing_config': {'type': 'RoutingTableRoutingConfig',
|
||||
@ -416,7 +416,8 @@ class TestNvplibExplicitLRouters(NvplibTestCase):
|
||||
'type': 'RouterNextHop'},
|
||||
'type': 'SingleDefaultRouteImplicitRoutingConfig'},
|
||||
'tags': [{'scope': 'os_tid', 'tag': 'fake_tenant_id'},
|
||||
{'scope': 'quantum', 'tag': '2013.1'}],
|
||||
{'scope': 'quantum',
|
||||
'tag': nvplib.NEUTRON_VERSION}],
|
||||
'type': 'LogicalRouterConfig'}
|
||||
self.assertEqual(expected, body)
|
||||
|
||||
@ -429,7 +430,8 @@ class TestNvplibExplicitLRouters(NvplibTestCase):
|
||||
expected = {'display_name': 'fake_router_name',
|
||||
'routing_config': {'type': 'RoutingTableRoutingConfig'},
|
||||
'tags': [{'scope': 'os_tid', 'tag': 'fake_tenant_id'},
|
||||
{'scope': 'quantum', 'tag': '2013.1'}],
|
||||
{'scope': 'quantum',
|
||||
'tag': nvplib.NEUTRON_VERSION}],
|
||||
'type': 'LogicalRouterConfig'}
|
||||
self.assertEqual(expected, body)
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user