]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix 'Inheritance-based rule deprecated' log warning
authorRushi Agrawal <rushi.agr@gmail.com>
Mon, 27 May 2013 14:18:12 +0000 (19:48 +0530)
committerRushi Agrawal <rushi.agr@gmail.com>
Mon, 27 May 2013 14:59:50 +0000 (20:29 +0530)
commit0f7865bfb0cf8d328df395e583d3889e5dcf39d1
treeaed3f5e9198ae3e30d3e83b1e793bc2f5c5e5660
parent05ce36aa24bce801546de53ac74942a788b3baef
Fix 'Inheritance-based rule deprecated' log warning

A warning message appears in Cinder logs which state that
inheritance-based rules are deprecated and use default
Brain instead of HttpBrain. It looks like HttpBrain class
doesnt have any difference from Brain class, so changing
all the instances of HttpBrain to Brain shouldn't affect
anything and removes this warning from logs.

Digging further, I found out that the policy engine has
gone through a major rework in Oslo and Nova, and I think
at some point of time in future, we will also be moving
to use the latest policy engine code. In that way of
thinking, I guess this fix is acceptable for now.
Reference: https://review.openstack.org/#/c/14122

Changed the brain from HttpBrain -> Brain, which makes the
warnings in the log disapper. A more comprehensive work will be
to pull the latest policy change code from Oslo.

Bug #1156608

Change-Id: I9f63ec4c41025042725db9b2e7c8ffa3d91e0596
cinder/policy.py
cinder/tests/test_policy.py
cinder/tests/test_volume.py