Ceilometer containers are not restarted when polling.yaml is changed

Bug #1860651 reported by Pierre Riteau
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kolla-ansible
Invalid
Undecided
Unassigned
Stein
Fix Committed
Medium
Unassigned

Bug Description

When a custom polling.yaml file is changed and a deploy/reconfigure is run, polling.yaml is updated on hosts, but Ceilometer containers are not restarted.

This only affects stable/stein and earlier.

Pierre Riteau (priteau)
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kolla-ansible (stable/stein)

Fix proposed to branch: stable/stein
Review: https://review.opendev.org/703985

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla-ansible (stable/stein)

Reviewed: https://review.opendev.org/703985
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=6997f8f711ab11ad119c9808ba4ff2c4c073b374
Submitter: Zuul
Branch: stable/stein

commit 6997f8f711ab11ad119c9808ba4ff2c4c073b374
Author: Pierre Riteau <email address hidden>
Date: Thu Jan 23 13:39:57 2020 +0100

    Fix restart handlers for polling.yaml changes

    Changes to polling.yaml were not triggering restart handlers due to a
    missing check on polling_overwriting.changed. This is committed straight
    to stable/stein, as stable/train and later have removed this code.

    The polling_overwriting variable is also removed from the
    ceilometer-notification handler as it doesn't use this file.

    Change-Id: I1e665c7b276d9c885881d96315daac4fa78682ce
    Closes-Bug: #1860651

tags: added: in-stable-stein
Mark Goddard (mgoddard)
Changed in kolla-ansible:
status: New → Invalid
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kolla-ansible (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.opendev.org/704118

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla-ansible (stable/rocky)

Reviewed: https://review.opendev.org/704118
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=5477464c2140c2233b9e49c0ba99e2d761ad1337
Submitter: Zuul
Branch: stable/rocky

commit 5477464c2140c2233b9e49c0ba99e2d761ad1337
Author: Pierre Riteau <email address hidden>
Date: Thu Jan 23 13:39:57 2020 +0100

    Fix restart handlers for polling.yaml changes

    Changes to polling.yaml were not triggering restart handlers due to a
    missing check on polling_overwriting.changed. This is committed straight
    to stable/stein, as stable/train and later have removed this code.

    The polling_overwriting variable is also removed from the
    ceilometer-notification handler as it doesn't use this file.

    Conflicts:
     ansible/roles/ceilometer/handlers/main.yml

    NOTE(priteau): Conflicts due to the ceilometer-ipmi handler not being
    present in stable/rocky, see I4cda336dedb3d807b80d13bcc219268a8d667b4d.

    Change-Id: I1e665c7b276d9c885881d96315daac4fa78682ce
    Closes-Bug: #1860651
    (cherry picked from commit 6997f8f711ab11ad119c9808ba4ff2c4c073b374)

tags: added: in-stable-rocky
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/kolla-ansible 7.2.0

This issue was fixed in the openstack/kolla-ansible 7.2.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/kolla-ansible 8.1.0

This issue was fixed in the openstack/kolla-ansible 8.1.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.