containerized swift still logs to /var/log/swift

[Martin André]

The containerized swift services still log to /var/log/swift due to rsyslog's magic in the rpm packaging, https://github.com/rdo-packages/swift-distgit/blob/110f4a22d45bf0ea2da85c67e04bb73f44fd49fc/openstack-swift.rsyslog.

This is a little bit confusing, because containers are supposed to log to /var/log/containers, and swift, while having the bind mounts in the containerized service templates doesn't log to it but to syslog, then the rsyslog config comes from the swift package installed on the host picks it up and write it to /var/log/swift.

We should at least make a symlink between /var/log/swift and /var/log/containers/swift on the host for consistency with other containerized services.

[Bogdan Dobrelya (bogdando)]

A symlink between /var/log/swift and /var/log/containers/swift would complicate upgrades. Users will have to first backup existing logs, only then /var/log/swift can be removed and recreated as a symlink

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/519577
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=84e9425edab83578d09fa612a1e772b188a9c3db
Submitter: Zuul
Branch: master

commit 84e9425edab83578d09fa612a1e772b188a9c3db
Author: Juan Antonio Osorio Robles <email address hidden>
Date: Tue Nov 14 08:11:20 2017 +0000

    Polish swift containers logging configuration

    The logging directories are not used by swift, which in turn logs
    to syslog. So the unnecessary bind-mounts were removed.

    Furthermore, as part of the swift package which is included in the
    overcloud image, there is an rsyslog rule which gets the swift logs
    and persists them in /var/log/swift. Due to this and to be consistent
    with the rest of the containers. A symlink was created to have the
    swift logs available from /var/log/containers/swift as well. This makes
    the readme indicating the change of place of the logs unnecessary, so
    that was removed as well.

    Closes-Bug: #1732107
    Change-Id: I61610d3cb187235ac316e1c5de0a344be3ebb1e2

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/pike
Review: https://review.openstack.org/521501

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/521501
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=ee46c9564402d02657a86a47e161d8a660209264
Submitter: Zuul
Branch: stable/pike

commit ee46c9564402d02657a86a47e161d8a660209264
Author: Juan Antonio Osorio Robles <email address hidden>
Date: Tue Nov 14 08:11:20 2017 +0000

    Polish swift containers logging configuration

    The logging directories are not used by swift, which in turn logs
    to syslog. So the unnecessary bind-mounts were removed.

    Furthermore, as part of the swift package which is included in the
    overcloud image, there is an rsyslog rule which gets the swift logs
    and persists them in /var/log/swift. Due to this and to be consistent
    with the rest of the containers. A symlink was created to have the
    swift logs available from /var/log/containers/swift as well. This makes
    the readme indicating the change of place of the logs unnecessary, so
    that was removed as well.

    Closes-Bug: #1732107
    Change-Id: I61610d3cb187235ac316e1c5de0a344be3ebb1e2
    (cherry picked from commit 84e9425edab83578d09fa612a1e772b188a9c3db)

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/tripleo-heat-templates 8.0.0.0b2 development milestone.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/tripleo-heat-templates 7.0.6 release.