Gnocchi

hide fact delete metric is async

Bug #1584139 reported by gordon chung
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Gnocchi
Fix Released
Low
gordon chung
Gnocchi 2.2.0
2.1
Fix Released
Low
Unassigned
Gnocchi 2.1.1

Bug Description

when deleting a metric, we get a 204 return. if we run it again before expunge, it still allows us to delete. the second call should really return 404

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

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

Changed in gnocchi:
assignee: nobody → gordon chung (chungg)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to gnocchi (master)

Reviewed: https://review.openstack.org/320558
Committed: https://git.openstack.org/cgit/openstack/gnocchi/commit/?id=6aae4578441b29bd78a3f57f8db48de0dfb9970c
Submitter: Jenkins
Branch: master

commit 6aae4578441b29bd78a3f57f8db48de0dfb9970c
Author: gord chung <email address hidden>
Date: Tue May 24 12:09:05 2016 -0400

    raise NoSuchMetric when deleting metric already marked deleted

    we should hide the fact the metric is marked for deletion but not
    yet deleted.

    Change-Id: Ib4a5d581208abed7978c53c169ce3ae0cc5d4ece
    Closes-Bug: #1584139

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

Fix proposed to branch: stable/2.1
Review: https://review.openstack.org/321465

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to gnocchi (stable/2.1)

Reviewed: https://review.openstack.org/321465
Committed: https://git.openstack.org/cgit/openstack/gnocchi/commit/?id=99725fc2e7e3230be3852b20d5199ce1c9a1bc58
Submitter: Jenkins
Branch: stable/2.1

commit 99725fc2e7e3230be3852b20d5199ce1c9a1bc58
Author: gord chung <email address hidden>
Date: Tue May 24 12:09:05 2016 -0400

    raise NoSuchMetric when deleting metric already marked deleted

    we should hide the fact the metric is marked for deletion but not
    yet deleted.

    The Gabbi test has not been backported to avoid depending on a too
    recent version of Gabbi.

    Change-Id: Ib4a5d581208abed7978c53c169ce3ae0cc5d4ece
    Closes-Bug: #1584139
    (cherry picked from commit 6aae4578441b29bd78a3f57f8db48de0dfb9970c)

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/gnocchi 2.2.0

This issue was fixed in the openstack/gnocchi 2.2.0 release.

Julien Danjou (jdanjou)
Changed in gnocchi:
status: Fix Committed → Fix Released
milestone: none → 2.2.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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