Skip to content
This repository has been archived by the owner on Feb 29, 2024. It is now read-only.

Commit

Permalink
Podman 1.4.1 drops json-file in favor of k8s-file
Browse files Browse the repository at this point in the history
While this new name makes more sense than the "json-file" which wasn't a
json, it breaks podman invocations[0] in TripleO.

Also, moved the configuration snipped to the podman dedicated block,
since docker doesn't have that driver[1].

[0] containers/podman#3363
[1] https://docs.docker.com/config/containers/logging/configure/

Depends-On: https://review.opendev.org/666223
Change-Id: I3891da45e665727ab6d767514f9d24cb06da6b50
cjeanner committed Jun 24, 2019
1 parent d48275a commit 55e1eac
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions common/container-puppet.py
Original file line number Diff line number Diff line change
@@ -408,7 +408,6 @@ def mp_puppet_config(*args):
'--env', 'NO_ARCHIVE=%s' % os.environ.get('NO_ARCHIVE', ''),
'--env', 'STEP=%s' % os.environ.get('STEP', '6'),
'--env', 'NET_HOST=%s' % os.environ.get('NET_HOST', 'false'),
'--log-driver', 'json-file',
'--volume', '/etc/localtime:/etc/localtime:ro',
'--volume', '%s:/etc/config.pp:ro' % tmp_man.name,
'--volume', '/etc/puppet/:/tmp/puppet-etc/:ro',
@@ -425,7 +424,8 @@ def mp_puppet_config(*args):

if container_cli == 'podman':
log_path = os.path.join(container_log_stdout_path, uname)
logging = ['--log-opt',
logging = ['--log-driver', 'k8s-file',
'--log-opt',
'path=%s.log' % log_path]
common_dcmd.extend(logging)

0 comments on commit 55e1eac

Please sign in to comment.