User ID for Notifications is Incorrect

Bug #1235145 reported by Justin Hopper
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack DBaaS (Trove)
Fix Released
Undecided
Justin Hopper

Bug Description

The User ID value for Notifications is actually using the User Name for create/delete events. We should align with Nova and send the actual User ID. In order to do this, we need to pass the User ID across on the Context which will require enhancing the TroveContext object. Once this is done, the value will be available to the Task Manager to populate the notification

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

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

Changed in trove:
assignee: nobody → Justin Hopper (justin-hopper)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to trove (master)

Reviewed: https://review.openstack.org/49855
Committed: http://github.com/openstack/trove/commit/6ba72ad35904c2907c6a20c58fc21173e2a237ed
Submitter: Jenkins
Branch: master

commit 6ba72ad35904c2907c6a20c58fc21173e2a237ed
Author: justin-hopper <email address hidden>
Date: Fri Oct 4 15:43:49 2013 -0700

    Fix User ID in Context and Notifications

        Notifications should contain the User ID not the User Name
        This change replaces the deprecated X-User header with
        X-User-Id. In most cases, the User ID is preferred over the
        name.

    Fixes: Bug 1235145
    Change-Id: I3cce6b35fc40f4ff4cc45b04714067956fb1864c

Changed in trove:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in trove:
milestone: none → icehouse-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in trove:
milestone: icehouse-1 → 2014.1
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.