]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Add resource tag to logging in volume.manager.py
authorJohn Griffith <john.griffith8@gmail.com>
Sat, 18 Apr 2015 00:19:37 +0000 (00:19 +0000)
committerJohn Griffith <john.griffith@solidfire.com>
Thu, 23 Apr 2015 14:50:04 +0000 (08:50 -0600)
commit386e2858a4e76fafc1d78737154859a3c1618b20
tree69fdda3dd94752cea7bf304594edacc50deacefd
parentcbcbc90cf64d91a33b7bcfb826c59d4b69697486
Add resource tag to logging in volume.manager.py

We now have resource tag support in oslo logging,
and our logging is pretty inconsistent and down right
ugly in places.  Let's clean things up based on the
standard logging guidelines and use the fancy new
resource tag.

To use set the following in cinder.conf:
logging_context_format_string = \
%(asctime)s.%(msecs)03d %(levelname)s %(name)s [%(request_id)s \
%(project_name)s] %(resource)s%(message)s

This change hits the majority of the code in manager and
should be used as an example/guide.  There are some exceptions
around migration and replication where things are kinda ugly,
those should be fixed up as follow up work.

Change-Id: I1315cc16116601fea8c023b47f99b2eaad01d6fb
cinder/tests/unit/test_volume.py
cinder/volume/manager.py