diff options
| author | Zuul <zuul@review.opendev.org> | 2023-01-27 17:30:30 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2023-01-27 17:30:30 +0000 |
| commit | a49526c278e52823080c7f3fcb72785b93fd4dcb (patch) | |
| tree | 80e0cfd592af942374a2a346ce0c55c015ce0ebd | |
| parent | d3ce7baf320606055a42872c80481d288d142841 (diff) | |
| parent | 1305a3152f75c6e62ec5094ea2bfc38f165204cf (diff) | |
| download | heat-a49526c278e52823080c7f3fcb72785b93fd4dcb.tar.gz | |
Merge "Honor 'hidden' parameter in 'stack environment show' command"
| -rw-r--r-- | heat/engine/service.py | 12 | ||||
| -rw-r--r-- | heat/tests/test_engine_service.py | 3 | ||||
| -rw-r--r-- | releasenotes/notes/honor-hidden-parameter-in-stack-env-show-cmd-062065545dfef82a.yaml | 4 |
3 files changed, 17 insertions, 2 deletions
diff --git a/heat/engine/service.py b/heat/engine/service.py index fdd497503..9019ddbea 100644 --- a/heat/engine/service.py +++ b/heat/engine/service.py @@ -12,6 +12,7 @@ # under the License. import collections +import copy import datetime import functools import itertools @@ -1354,7 +1355,16 @@ class EngineService(service.ServiceBase): :rtype: dict """ s = self._get_stack(cnxt, stack_identity, show_deleted=True) - return s.raw_template.environment + tmpl = templatem.Template.load(cnxt, s.raw_template_id, s.raw_template) + param_schemata = tmpl.all_param_schemata(tmpl.files) + env = copy.deepcopy(s.raw_template.environment) + for section in [env_fmt.PARAMETERS, env_fmt.PARAMETER_DEFAULTS]: + for param_name in env.get(section, {}).keys(): + if (param_name not in param_schemata + or not param_schemata[param_name].hidden): + continue + env[section][param_name] = str('******') + return env @context.request_context def get_files(self, cnxt, stack_identity): diff --git a/heat/tests/test_engine_service.py b/heat/tests/test_engine_service.py index 875d44dcd..9ce13e308 100644 --- a/heat/tests/test_engine_service.py +++ b/heat/tests/test_engine_service.py @@ -978,11 +978,12 @@ class StackServiceTest(common.HeatTestCase): env = {'parameters': {'KeyName': 'EnvKey'}} tmpl = templatem.Template(t) stack = parser.Stack(self.ctx, 'get_env_stack', tmpl) + stack.store() mock_get_stack = self.patchobject(self.eng, '_get_stack') mock_get_stack.return_value = mock.MagicMock() mock_get_stack.return_value.raw_template.environment = env - self.patchobject(parser.Stack, 'load', return_value=stack) + self.patchobject(templatem.Template, 'load', return_value=tmpl) # Test found = self.eng.get_environment(self.ctx, stack.identifier()) diff --git a/releasenotes/notes/honor-hidden-parameter-in-stack-env-show-cmd-062065545dfef82a.yaml b/releasenotes/notes/honor-hidden-parameter-in-stack-env-show-cmd-062065545dfef82a.yaml new file mode 100644 index 000000000..385e3d4d5 --- /dev/null +++ b/releasenotes/notes/honor-hidden-parameter-in-stack-env-show-cmd-062065545dfef82a.yaml @@ -0,0 +1,4 @@ +--- +fixes: + - | + Honor ``hidden`` parameter in ``stack environment show`` command |