delete subnet from admin tenant fails for PLUMgrid Plugin

Bug #1404688 reported by Fawad Khaliq
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Fawad Khaliq

Bug Description

If a non-admin tenant creates a network and a subnet and delete operation is performed by the admin tenant, PLUMgrid plugin ends up passing incorrect tenant_id to the backend and this fails the calls.

Fix would be to get correct tenant UUID in delete subnet operation.

Changed in neutron:
assignee: nobody → Fawad Khaliq (fawadkhaliq)
Changed in neutron:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

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

Reviewed: https://review.openstack.org/143315
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=d5e9f06cf34096cd4c68c168086b6f9cc48072c3
Submitter: Jenkins
Branch: master

commit d5e9f06cf34096cd4c68c168086b6f9cc48072c3
Author: Fawad Khaliq <email address hidden>
Date: Sun Dec 21 12:57:27 2014 -0800

    PLUMgrid plugin: Fix for delete subnet with admin context

    When delete call using admin for a subnet created from a
    non-admin project is made, the tenant_id passed to backend
    happened to be of admin project. This commit fixes the issues
    by getting the correct tenant_id.

    Closes-Bug: 1404688
    Change-Id: Id21c38610ed73defb937d971a7aade57713541c0

Changed in neutron:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
milestone: none → kilo-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: kilo-2 → 2015.1.0
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.