]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix unnecessary WSGI worker warning at API startup
authorJay S. Bryant <jsbryant@us.ibm.com>
Fri, 19 Sep 2014 17:46:21 +0000 (12:46 -0500)
committerJay S. Bryant <jsbryant@us.ibm.com>
Tue, 23 Sep 2014 21:20:09 +0000 (16:20 -0500)
commit3d301346a7100e52fe0c319c1c1b5b81a3bc1660
tree59373a0a806cec146f67ff62139344b037f05093
parent3120df17457878fc2facbad660baea3a6072b3e0
Fix unnecessary WSGI worker warning at API startup

There was a bug in WSGIService in the way that it was
checking the osapi_volume_workers option.  It was using
getattr() to see if the option was set, if not it was supposed
to set the value to processutils.get_worker_count().  This,
however, never happened because getattr interpreted the default
'None' value to be a value.  So, on any system with no value set
the self.workers < 1 check would be hit and a warning would be
output.

Nova had changed their approach to this option to avoid this
problem.  This patch pulls Nova's approach into Cinder for
consistency.  Cinder will now use processutils.get_worker_count()
if no option is set in /etc/cinder/cinder.conf and when the user sets
osapi_volume_workers to 0.  A negative value will cause an
InvalidInput exception to be thrown.

Unittests have been added for this functionality.

Change-Id: I4ec2fdd0d19195cccffd63cdd1af1b9ca9884c7d
Closes-bug: #1367454
cinder/service.py
cinder/tests/test_service.py