openstack/python-tripleoclient
Code Issues Proposed changes

Merge "Minor fixes for container image prepare command"

Browse Source
This commit is contained in:
Jenkins 2017-08-03 14:46:30 +00:00 committed by Gerrit Code Review
commit 51ae069be8
1 changed files with 11 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
tripleoclient/v1/container_image.py
View File

@ -244,20 +244,15 @@ class PrepareImageFiles(command.Command):
) )
return parser return parser
def write_env_file(self, result, env_file): def write_env_file(self, params, env_file):
params = {}
for entry in result:
imagename = entry.get('imagename', '')
if 'params' in entry:
for p in entry.pop('params'):
params[p] = imagename
with os.fdopen(os.open(env_file, with os.fdopen(os.open(env_file,
os.O_CREAT | os.O_TRUNC | os.O_WRONLY, 0o666), os.O_CREAT | os.O_TRUNC | os.O_WRONLY, 0o666),
'w') as f: 'w') as f:
f.write('# Generated with the following on %s\n#\n' % f.write('# Generated with the following on %s\n#\n' %
datetime.datetime.now().isoformat()) datetime.datetime.now().isoformat())
f.write('# %s\n#\n\n' % ' '.join(self.app.command_options)) f.write('# openstack %s\n#\n\n' %
' '.join(self.app.command_options))
yaml.safe_dump({'parameter_defaults': params}, f, yaml.safe_dump({'parameter_defaults': params}, f,
default_flow_style=False) default_flow_style=False)
@ -285,8 +280,15 @@ class PrepareImageFiles(command.Command):
builder = kolla_builder.KollaImageBuilder([parsed_args.template_file]) builder = kolla_builder.KollaImageBuilder([parsed_args.template_file])
result = builder.container_images_from_template(filter=ffunc, **subs) result = builder.container_images_from_template(filter=ffunc, **subs)
params = {}
for entry in result:
imagename = entry.get('imagename', '')
if 'params' in entry:
for p in entry.pop('params'):
params[p] = imagename
if parsed_args.env_file: if parsed_args.env_file:
self.write_env_file(result, parsed_args.env_file) self.write_env_file(params, parsed_args.env_file)
result_str = yaml.safe_dump({'container_images': result}, result_str = yaml.safe_dump({'container_images': result},
default_flow_style=False) default_flow_style=False)