don't bother building a safe extra vars namespace; it's a file path now

This commit is contained in:
Ryan Petrello
2018-04-17 10:24:14 -04:00
parent 7074dcd677
commit 7304301948

View File

@@ -581,10 +581,8 @@ class CredentialType(CommonModelNameNotUnique):
safe_env[env_var] = Template(tmpl).render(**safe_namespace)
extra_vars = {}
safe_extra_vars = {}
for var_name, tmpl in self.injectors.get('extra_vars', {}).items():
extra_vars[var_name] = Template(tmpl).render(**namespace)
safe_extra_vars[var_name] = Template(tmpl).render(**safe_namespace)
def build_extra_vars_file(vars, private_dir):
handle, path = tempfile.mkstemp(dir = private_dir)
@@ -594,12 +592,9 @@ class CredentialType(CommonModelNameNotUnique):
os.chmod(path, stat.S_IRUSR)
return path
path = build_extra_vars_file(extra_vars, private_data_dir)
if extra_vars:
path = build_extra_vars_file(extra_vars, private_data_dir)
args.extend(['-e', '@%s' % path])
if safe_extra_vars:
path = build_extra_vars_file(safe_extra_vars, private_data_dir)
safe_args.extend(['-e', '@%s' % path])