Group Based Policy

Multiple consumer removed notification received by node driver

Bug #1590340 reported by Ashutosh Mishra
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Group Based Policy
Fix Released
Medium
Ashutosh Mishra
Group Based Policy next

Bug Description

For deletion of consumer ptg from chain,
multiple consumer received notifications(update_chains_consumer_removed) calls received by node driver.
While deleting policy target group, grouppolicy plugin is updating group first before making ptg delete post_commit call.
Issue is, initially plugin_context gets created with original ptg(i.e. with prs attached), and same plugin_context is used in post_commit call, but in between group gets updated, to have no prs.
So we need to modify plugin context to have this change.

See original description
Ashutosh Mishra (mca-ashu4)
description: updated
Ashutosh Mishra (mca-ashu4)
summary: - Multiple consumer removed notificartion received by node driver
+ Multiple consumer removed notification received by node driver
Sumit Naiksatam (snaiksat)
Changed in group-based-policy:
status: New → Confirmed
importance: Undecided → Medium
assignee: nobody → ashu (mca-ashu4)
milestone: none → next
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to group-based-policy (master)

Reviewed: https://review.openstack.org/326953
Committed: https://git.openstack.org/cgit/openstack/group-based-policy/commit/?id=daf773f2c038e7bd244270c65809ca4084f08407
Submitter: Jenkins
Branch: master

commit daf773f2c038e7bd244270c65809ca4084f08407
Author: Ashutosh Mishra <email address hidden>
Date: Wed Jun 8 14:52:08 2016 +0530

    Fixes for multiple notifications recevied by node drivers.

    While deleting group, Group based policy plugin creates context
    before precommit call, updates group(to unset all prs) and then
    makes post commit call with same context. Since group gets
    updated in between, we need to modify context to have those changes.

    Change-Id: I3a9606b66026e00ce8a7ea9dc590f1ba8594af21
    Closes-Bug: 1590340

Changed in group-based-policy:
status: Confirmed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to group-based-policy (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/348125

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to group-based-policy (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/348729

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to group-based-policy (stable/mitaka)

Reviewed: https://review.openstack.org/348125
Committed: https://git.openstack.org/cgit/openstack/group-based-policy/commit/?id=ff267cecc24e1d34328e4e8d80fd8f70a53d2f75
Submitter: Jenkins
Branch: stable/mitaka

commit ff267cecc24e1d34328e4e8d80fd8f70a53d2f75
Author: Ashutosh Mishra <email address hidden>
Date: Wed Jun 8 14:52:08 2016 +0530

    Fixes for multiple notifications recevied by node drivers.

    While deleting group, Group based policy plugin creates context
    before precommit call, updates group(to unset all prs) and then
    makes post commit call with same context. Since group gets
    updated in between, we need to modify context to have those changes.

    Change-Id: I3a9606b66026e00ce8a7ea9dc590f1ba8594af21
    Closes-Bug: 1590340
    (cherry picked from commit daf773f2c038e7bd244270c65809ca4084f08407)

tags: added: in-stable-mitaka
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to group-based-policy (stable/liberty)

Reviewed: https://review.openstack.org/348729
Committed: https://git.openstack.org/cgit/openstack/group-based-policy/commit/?id=614688633210ae3d0cb3bb11ceb2e609ce910bf9
Submitter: Jenkins
Branch: stable/liberty

commit 614688633210ae3d0cb3bb11ceb2e609ce910bf9
Author: Ashutosh Mishra <email address hidden>
Date: Wed Jun 8 14:52:08 2016 +0530

    Fixes for multiple notifications in node drivers

    While deleting group, Group based policy plugin
    creates context before precommit call, updates
    group(to unset all prs) and then makes post
    commit call with same context. Since group gets
    updated in between, we need to modify context
    to have those changes

    Change-Id: I3a9606b66026e00ce8a7ea9dc590f1ba8594af21
    Closes-Bug: 1590340
    (cherry picked from commit daf773f2c038e7bd244270c65809ca4084f08407)

tags: added: in-stable-liberty
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.