diff options
| author | Jenkins <jenkins@review.openstack.org> | 2016-02-25 03:08:32 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2016-02-25 03:08:32 +0000 |
| commit | 06b2ef0e3ccd0e224c85b6d49a1ff7aa475c581d (patch) | |
| tree | a12bd11e5a43b4d4d28d7b82ebcb48219c28d3e0 | |
| parent | f9a2aee026081af9e3d1408c5d1f47aaddb6a75c (diff) | |
| parent | d6c3e09680404d3e0fc7bb0daca9160bcf0dc5b4 (diff) | |
| download | python-heatclient-06b2ef0e3ccd0e224c85b6d49a1ff7aa475c581d.tar.gz | |
Merge "Ignore restricted_actions key"
| -rw-r--r-- | heatclient/common/template_utils.py | 2 | ||||
| -rw-r--r-- | heatclient/tests/unit/test_template_utils.py | 22 |
2 files changed, 23 insertions, 1 deletions
diff --git a/heatclient/common/template_utils.py b/heatclient/common/template_utils.py index a14f35c..b80878d 100644 --- a/heatclient/common/template_utils.py +++ b/heatclient/common/template_utils.py @@ -261,7 +261,7 @@ def resolve_environment_urls(resource_registry, files, env_base_url, # Built in providers like: "X::Compute::Server" # don't need downloading. return True - if key == 'hooks': + if key in ['hooks', 'restricted_actions']: return True get_file_contents(rr, files, base_url, ignore_if, diff --git a/heatclient/tests/unit/test_template_utils.py b/heatclient/tests/unit/test_template_utils.py index 1e1eee9..c99086e 100644 --- a/heatclient/tests/unit/test_template_utils.py +++ b/heatclient/tests/unit/test_template_utils.py @@ -52,6 +52,28 @@ class ShellEnvironmentTest(testtools.TestCase): if url: self.assertEqual(content.decode('utf-8'), files[url]) + def test_ignore_env_keys(self): + self.m.StubOutWithMock(request, 'urlopen') + env_file = '/home/my/dir/env.yaml' + env = b''' + resource_registry: + resources: + bar: + hooks: pre_create + restricted_actions: replace + ''' + request.urlopen('file://%s' % env_file).AndReturn( + six.BytesIO(env)) + self.m.ReplayAll() + _, env_dict = template_utils.process_environment_and_files( + env_file) + self.assertEqual( + {u'resource_registry': {u'resources': { + u'bar': {u'hooks': u'pre_create', + u'restricted_actions': u'replace'}}}}, + env_dict) + self.m.VerifyAll() + def test_process_environment_file(self): self.m.StubOutWithMock(request, 'urlopen') |
