openstack/rally
Code Issues Proposed changes

Merge "Improve restricted_parameters validator to cover None case"

Browse Source
This commit is contained in:
Zuul 2020-05-07 14:19:20 +00:00 committed by Gerrit Code Review
commit 266d6bc53f
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
rally/plugins/common/validators.py
View File

@ -294,10 +294,10 @@ class RestrictedParametersValidator(validation.Validator):
def validate(self, context, config, plugin_cls, plugin_cfg):
restricted_params = []
for param_name in self.params:
args = config.get("args", {})
a_dict, a_key = (args, self.subdict) if self.subdict else (
config, "args")
if param_name in a_dict.get(a_key, {}):
source = config.get("args", {})
if self.subdict:
source = source.get(self.subdict) or {}
if param_name in source:
restricted_params.append(param_name)
if restricted_params:
self.fail("You can't specify parameters '%s' in '%s'" % (