]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fixed logging for oslo versioned objects
authorSzymon Borkowski <szymon.borkowski@intel.com>
Tue, 12 Jan 2016 10:32:22 +0000 (11:32 +0100)
committerMichal Dulko <michal.dulko@intel.com>
Tue, 1 Mar 2016 18:21:31 +0000 (18:21 +0000)
commitfedd45446992efc848178f4851b0405cfa7d2fdd
tree57d9e486c4b9311483f59777a86e67ff8514d373
parent3f2fe6266ec48327b60c9ed73d2a30ae976beb83
Fixed logging for oslo versioned objects

Earlier, when we wanted to log a message with an object as a parameter,
obj_attr_is_set method used to check name parameter, which was a wrong
approach. The reason was, that the oslo logger, when received a
versioned object as a resource parameter, is trying to get the 'name'
parameter, which is a property, so the obj_attr_is_set method will
return False and the logger will try to get the 'type' parameter, which
does not exist in some versioned objects (please take a look at
oslo.logging code [1]).

Now, when the parameter's name is present in obj_extra_fields
dict, we avoid calling obj_attr_is_set and simply get() it.

The other cause was a difference between fields names: size (in volume
object) and volume_size (in snapshot object), and inproper condition
statement in lvm driver.

[1] goo.gl/YffLcK

Change-Id: Id92d58b4ccced907cc6e3e59d9e71650a459b4a8
Closes-Bug: 1501521
cinder/objects/base.py
cinder/tests/unit/objects/test_base.py
cinder/volume/drivers/lvm.py