devstack logs do not contain pid information for log messages with context
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
devstack |
Fix Released
|
Undecided
|
Ihar Hrachyshka | ||
neutron |
Fix Released
|
Medium
|
Ihar Hrachyshka |
Bug Description
Compare:
2015-03-18 15:00:15.990 INFO neutron.wsgi [req-412094f3-
2015-03-18 15:00:16.001 15709 INFO neutron.wsgi [-] (15709) accepted ('127.0.0.1', 60381)
This is because in devstack, we override the default log format string with the one that misses the info. Note that to make it work, it is not enough to fall back to default string, since it uses user_identity context field that is missing in neutron context object. That is because neutron.
The proper fix would be:
- make neutron context reuse oslo_context.
- make devstack not overwrite the default log format string
Also note that log colorizer from devstack also rewrites the default format string value. In that case, we just need to update the string to include pid information.
Also note that the issue may be more far reaching, since devstack rewrites the string for other services too (nova, ironic, among others).
Changed in neutron: | |
status: | New → In Progress |
Changed in neutron: | |
milestone: | none → liberty-1 |
status: | Fix Committed → Fix Released |
Changed in neutron: | |
milestone: | liberty-1 → 7.0.0 |
no longer affects: | oslo.log |
The bug is especially interesting now that we run multiple workers for neutron-server by default.