Comment 2 for bug 1952882

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-ceilometer-agent (master)

Reviewed: https://review.opendev.org/c/openstack/charm-ceilometer-agent/+/820001
Committed: https://opendev.org/openstack/charm-ceilometer-agent/commit/43a23127949e035506b89afd6fb01b5cfec04b63
Submitter: "Zuul (22348)"
Branch: master

commit 43a23127949e035506b89afd6fb01b5cfec04b63
Author: Aurelien Lourot <email address hidden>
Date: Mon Nov 29 15:17:44 2021 +0100

    Remove pause/resume logic

    This is a subordinate charm and since a recent
    commit [1] it shares a list of its services with
    the principal charm nova-compute, which has now
    the responsibility to pause and resume services. [2]

    The ceilometer-agent-compute service has a
    dependency to the nova-compute service anyway, so
    it is impossible for this charm to resume its
    service if its principal charm nova-compute is
    paused. This is what also led to errors in
    ceilometer-agent's post-series-upgrade hook. This
    hook attempted to resume its service although
    the principal service was still paused. Removing
    this logic entirely solves this issue.

    Validated by running openstack-upgrade and
    series-upgrade tests. [3]

    [1]: https://opendev.org/openstack/charm-ceilometer-agent/commit/be45f779
    [2]: https://opendev.org/openstack/charm-nova-compute/commit/8fb37dc0
    [3]: https://github.com/openstack-charmers/charmed-openstack-tester

    Closes-Bug: #1952882
    Change-Id: Ia22b53b52b541250f7f803c6708968d75e64475c