metering data lost randomly

Bug #1827768 reported by Shaoman Zhang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Undecided
Shaoman Zhang

Bug Description

In this change:
https://review.opendev.org/#/c/509814/
Thread lock has been added for flush(), while queue may be modified in other places, which has not been protected, so this leads to the random loss of metering data.

Changed in ceilometer:
assignee: nobody → Shaoman Zhang (zhang-shaoman)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

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

Changed in ceilometer:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (stable/stein)

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

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

Reviewed: https://review.opendev.org/657213
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=fd000b8c131b3b5d23d2c45233353107536565e1
Submitter: Zuul
Branch: master

commit fd000b8c131b3b5d23d2c45233353107536565e1
Author: zhang-shaoman <email address hidden>
Date: Sun May 5 16:25:47 2019 +0800

    metering data lost randomly

    Thread lock has been added for flush(), while queue may be modified
    in other places, which has not been protected, so need to add lock in
    such places too.

    Change-Id: I6b46f07690f2fc165fe27bf509ee8926656fd6b9
    Closes-Bug: #1827768

Changed in ceilometer:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (stable/stein)

Reviewed: https://review.opendev.org/657215
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=fc2586e852a6d308719cb668c785147a672341a2
Submitter: Zuul
Branch: stable/stein

commit fc2586e852a6d308719cb668c785147a672341a2
Author: zhang-shaoman <email address hidden>
Date: Sun May 5 16:25:47 2019 +0800

    metering data lost randomly

    Thread lock has been added for flush(), while queue may be modified
    in other places, which has not been protected, so need to add lock in
    such places too.

    Change-Id: I6b46f07690f2fc165fe27bf509ee8926656fd6b9
    Closes-Bug: #1827768

tags: added: in-stable-stein
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ceilometer 13.0.0.0rc1

This issue was fixed in the openstack/ceilometer 13.0.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ceilometer 12.1.0

This issue was fixed in the openstack/ceilometer 12.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.