ha containers hardcode container's log-driver

[Michele Baldessari]

Currently in puppet-tripleo for the HA container we hardcode the following:
 options => "--user=root --log-driver=journald -e KOLLA_CONFIG_STRATEGY=COPY_ALWAYS${tls_priorities_real}",

Since at least podman had some changes in tmers of supported driver backends (and bugs) it's best if we make this configurable. While we're at it we should also switch to k8s-file as a driver which is what all other containers are using

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/695578

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/695578
Committed: https://git.openstack.org/cgit/openstack/puppet-tripleo/commit/?id=bad716070a70b258c110d4ef1feca422b14a9a98
Submitter: Zuul
Branch: master

commit bad716070a70b258c110d4ef1feca422b14a9a98
Author: Michele Baldessari <email address hidden>
Date: Thu Nov 21 22:02:13 2019 +0100

    Switch HA containers to k8s-file log-driver and make it a parameter

    Currently in puppet-tripleo for the HA container we hardcode the following:
     options => "--user=root --log-driver=journald -e KOLLA_CONFIG_STRATEGY=COPY_ALWAYS${tls_priorities_real}",

    Since at least podman had some changes in terms of supported driver
    backends (and bugs) it's best if we make this configurable. While we're
    at it we should also switch to k8s-file as a driver when podman is being
    used which is what all other containers are using. When docker is the
    default container_cli we will stick to journald as usual.

    Tested this on a Train environment and successfully verified that
    we still see the correct logs in /var/log/containers/.../...

    Change-Id: I5b1483826f816d11a064a937d59f9a8f468315a5
    Closes-Bug: #1853517

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/train
Review: https://review.opendev.org/695734

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/695734
Committed: https://git.openstack.org/cgit/openstack/puppet-tripleo/commit/?id=bb277689eaa7d61d88be5c360a0428b466bd5a7a
Submitter: Zuul
Branch: stable/train

commit bb277689eaa7d61d88be5c360a0428b466bd5a7a
Author: Michele Baldessari <email address hidden>
Date: Thu Nov 21 22:02:13 2019 +0100

    Switch HA containers to k8s-file log-driver and make it a parameter

    Currently in puppet-tripleo for the HA container we hardcode the following:
     options => "--user=root --log-driver=journald -e KOLLA_CONFIG_STRATEGY=COPY_ALWAYS${tls_priorities_real}",

    Since at least podman had some changes in terms of supported driver
    backends (and bugs) it's best if we make this configurable. While we're
    at it we should also switch to k8s-file as a driver when podman is being
    used which is what all other containers are using. When docker is the
    default container_cli we will stick to journald as usual.

    Tested this on a Train environment and successfully verified that
    we still see the correct logs in /var/log/containers/.../...

    Change-Id: I5b1483826f816d11a064a937d59f9a8f468315a5
    Closes-Bug: #1853517
    (cherry picked from commit bad716070a70b258c110d4ef1feca422b14a9a98)

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/puppet-tripleo 12.0.0 release.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/puppet-tripleo 11.4.0 release.