Horizon container logs path chown'ing is not idempotent

Bug #1819626 reported by Bogdan Dobrelya
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
Medium
Bogdan Dobrelya

Bug Description

Whenever /var/log/horizon/horizon.log recreated as a root,
the touch command fails, and the chown commands got skipped.
This results in idempotency issues when the logs path become
accessed under mismatching user.

Changed in tripleo:
importance: Undecided → Medium
milestone: none → stein-3
tags: added: idempotency
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (master)

Fix proposed to branch: master
Review: https://review.openstack.org/642717

Changed in tripleo:
assignee: nobody → Bogdan Dobrelya (bogdando)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/642718

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (master)

Reviewed: https://review.openstack.org/642717
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=95e142d84eeaf82b23323bb3a8eab1e4f6a9e005
Submitter: Zuul
Branch: master

commit 95e142d84eeaf82b23323bb3a8eab1e4f6a9e005
Author: Bogdan Dobrelya <email address hidden>
Date: Tue Mar 12 11:38:51 2019 +0100

    Fix idempotency for horizon container logs

    Whenever /var/log/horizon/horizon.log recreated as a root,
    the touch command fails, and the chown commands got skipped.
    This results in idempotency issues when the logs path become
    accessed under mismatching user.

    Fix this by making the chown command always applying disregard
    of the touch command results.

    Closes-Bug: #1819626

    Change-Id: Ied538dbc7dbd311361108747a1e89d1f5740eedd
    Signed-off-by: Bogdan Dobrelya <email address hidden>

Changed in tripleo:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.openstack.org/642979

tags: added: queens-backport-potential rocky-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (stable/rocky)

Reviewed: https://review.openstack.org/642979
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=0fc5b0efc8506efd7826e340292cb09beb7b2244
Submitter: Zuul
Branch: stable/rocky

commit 0fc5b0efc8506efd7826e340292cb09beb7b2244
Author: Bogdan Dobrelya <email address hidden>
Date: Tue Mar 12 11:38:51 2019 +0100

    Fix idempotency for horizon container logs

    Whenever /var/log/horizon/horizon.log recreated as a root,
    the touch command fails, and the chown commands got skipped.
    This results in idempotency issues when the logs path become
    accessed under mismatching user.

    Fix this by making the chown command always applying disregard
    of the touch command results.

    Closes-Bug: #1819626

    Change-Id: Ied538dbc7dbd311361108747a1e89d1f5740eedd
    Signed-off-by: Bogdan Dobrelya <email address hidden>
    (cherry picked from commit 95e142d84eeaf82b23323bb3a8eab1e4f6a9e005)

tags: added: in-stable-rocky
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (stable/queens)

Reviewed: https://review.openstack.org/642718
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=7761ae38d487bd074a3d5e2bab50924588772243
Submitter: Zuul
Branch: stable/queens

commit 7761ae38d487bd074a3d5e2bab50924588772243
Author: Bogdan Dobrelya <email address hidden>
Date: Tue Mar 12 11:38:51 2019 +0100

    Fix idempotency for horizon container logs

    Whenever /var/log/horizon/horizon.log recreated as a root,
    the touch command fails, and the chown commands got skipped.
    This results in idempotency issues when the logs path become
    accessed under mismatching user.

    Fix this by making the chown command always applying disregard
    of the touch command results.

    Closes-Bug: #1819626

    Change-Id: Ied538dbc7dbd311361108747a1e89d1f5740eedd
    Signed-off-by: Bogdan Dobrelya <email address hidden>
    (cherry picked from commit 95e142d84eeaf82b23323bb3a8eab1e4f6a9e005)

tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 10.5.0

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 9.4.0

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 8.4.0

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

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.