Claim ID not required to delete claimed messages

Bug #1215484 reported by Kurt Griffiths
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
Fix Released
High
Zhihao Yuan

Bug Description

As discussed in meeting #21 [1], we need to modify the storage drivers to require a claim ID in order to delete messages, IFF the messages in question have been claimed (i.e., have a non-expired claim associated with them).

If a producer wishes to "recall" a message, it may still do so up to the point that it is claimed by a worker.

[1]: http://eavesdrop.openstack.org/meetings/marconi/2013/marconi.2013-08-19-16.03.html

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

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

Changed in marconi:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to marconi (master)

Reviewed: https://review.openstack.org/43339
Committed: http://github.com/stackforge/marconi/commit/3376b920ed1abdc0822cbd7d97cef4b2e6a3b410
Submitter: Jenkins
Branch: master

commit 3376b920ed1abdc0822cbd7d97cef4b2e6a3b410
Author: Zhihao Yuan <email address hidden>
Date: Thu Aug 22 15:32:10 2013 -0400

    fix: claimed message require claim_id to delete

    But bulk deletion can still delete both unclaimed and claimed
    messages. However, the major purpose to support bulk deletion
    is to make it easy for a user to issue a DELETE on the Location
    header returned from a POST; if a user intend to use claim, then
    this URI does not help.

    Change-Id: Ieed92fc0a07d2e873b729242fb2fb14536462780
    Closes-Bug: #1215484

Changed in marconi:
status: In Progress → Fix Committed
Changed in marconi:
status: Fix Committed → Fix Released
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.