diff options
author | Jenkins <jenkins@review.openstack.org> | 2015-04-06 22:38:34 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2015-04-06 22:38:34 +0000 |
commit | 7a6e5f4914a139647d3203eee6fd6f1f7df9dbb0 (patch) | |
tree | fababd4c19a949ac2627b0330eab25081ae4ff9e | |
parent | c875ba4e973aca0e727af38622047a9f9a06f804 (diff) | |
parent | c0f3f851c3787fc95fa450430db7a52314ee6d01 (diff) | |
download | heat-7a6e5f4914a139647d3203eee6fd6f1f7df9dbb0.tar.gz |
Merge "Do not resolve neutron resource attributes when resource_id is None"
-rw-r--r-- | heat/engine/resources/openstack/neutron/neutron.py | 19 | ||||
-rw-r--r-- | heat/tests/neutron/test_neutron.py | 6 |
2 files changed, 16 insertions, 9 deletions
diff --git a/heat/engine/resources/openstack/neutron/neutron.py b/heat/engine/resources/openstack/neutron/neutron.py index 926161749..17b4dc206 100644 --- a/heat/engine/resources/openstack/neutron/neutron.py +++ b/heat/engine/resources/openstack/neutron/neutron.py @@ -119,15 +119,16 @@ class NeutronResource(resource.Resource): result=_('Resource is not built')) def _resolve_attribute(self, name): - try: - attributes = self._show_resource() - except Exception as ex: - self.client_plugin().ignore_not_found(ex) - return None - if name == 'show': - return attributes - - return attributes[name] + if self.resource_id: + try: + attributes = self._show_resource() + except Exception as ex: + self.client_plugin().ignore_not_found(ex) + return None + if name == 'show': + return attributes + + return attributes[name] def FnGetRefId(self): return six.text_type(self.resource_id) diff --git a/heat/tests/neutron/test_neutron.py b/heat/tests/neutron/test_neutron.py index 5dc201816..9b55e1f5e 100644 --- a/heat/tests/neutron/test_neutron.py +++ b/heat/tests/neutron/test_neutron.py @@ -17,6 +17,7 @@ from neutronclient.v2_0 import client as neutronclient import six from heat.common import exception +from heat.engine.clients.os import neutron from heat.engine.hot import functions from heat.engine import properties from heat.engine import resource @@ -114,14 +115,19 @@ class NeutronTest(common.HeatTestCase): mock_show_resource.side_effect = [{'attr1': 'val1', 'attr2': 'val2'}, {'attr1': 'val1', 'attr2': 'val2'}, {'attr1': 'val1', 'attr2': 'val2'}, + qe.NotFound, qe.NeutronClientException] res._show_resource = mock_show_resource + nclientplugin = neutron.NeutronClientPlugin(mock.MagicMock()) + res.client_plugin = mock.Mock(return_value=nclientplugin) self.assertEqual({'attr1': 'val1', 'attr2': 'val2'}, res._resolve_attribute('show')) self.assertEqual('val2', res._resolve_attribute('attr2')) self.assertRaises(KeyError, res._resolve_attribute, 'attr3') self.assertIsNone(res._resolve_attribute('attr2')) + res.resource_id = None + self.assertIsNone(res._resolve_attribute('show')) class GetSecGroupUuidTest(common.HeatTestCase): |