logs in greenthreads don't log request_id
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Fix Released
|
High
|
Vish Ishaya |
Bug Description
When I'm tracking stuff through the volume logs, sometimes the request_id, user_id and project_id are missing:
2012-02-28 14:33:14 DEBUG nova.rpc.common [-] received {u'_context_roles': [u'Admin', u'Admin', u'admin'], u'_context_
023-4f0f4636ec25', u'_context_
rategy': u'keystone', u'_context_
: u'0747b73bf9404
dist-packages/
2012-02-28 14:33:14 DEBUG nova.rpc.common [req-38167a3a-
u'req-38167a3a-
8T19:33:14.361743', 'is_admin': True, 'auth_token': u'999888777666', 'project_id': u'12345', 'remote_address': u'10.6.60.211', 'read_deleted': u'no', 'strate
gy': u'keystone'} from (pid=3934) unpack_context /usr/lib/
2012-02-28 14:33:14 INFO nova.volume.manager [-] volume volume-0000001a: creating
2012-02-28 14:33:14 DEBUG nova.volume.manager [-] volume volume-0000001a: creating lv of size 10G from (pid=3934) create_volume /usr/lib/
ges/nova/
2012-02-28 14:33:14 DEBUG nova.volume.manager [-] volume volume-0000001a: creating export from (pid=3934) create_volume /usr/lib/
/volume/
2012-02-28 14:33:14 DEBUG nova.volume.manager [-] volume volume-0000001a: created successfully from (pid=3934) create_volume /usr/lib/
/nova/volume/
You can see the [req_id user_id project_id] in the rpc.common "unpacked context" line - but the subsequent "nova.volume.
This makes it really hard to troubleshoot when lots of stuff is going on...
Changed in nova: | |
status: | New → Incomplete |
Changed in nova: | |
status: | New → In Progress |
importance: | Undecided → High |
milestone: | none → essex-rc1 |
assignee: | nobody → Vish Ishaya (vishvananda) |
summary: |
- nova-volume doesn't log request_id + greenthrads don't log request_id |
summary: |
- greenthrads don't log request_id + logs in greenthreads don't log request_id |
Changed in nova: | |
status: | Fix Committed → Fix Released |
Changed in nova: | |
milestone: | essex-rc1 → 2012.1 |
is this fairly current code? There was an issue where elevated() was overwriting the context that we fixed a while ago.