]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Sync 'service' module from oslo-incubator
authorJay S. Bryant <jsbryant@us.ibm.com>
Fri, 20 Feb 2015 22:29:10 +0000 (16:29 -0600)
committerJay S. Bryant <jsbryant@us.ibm.com>
Fri, 20 Feb 2015 22:29:10 +0000 (16:29 -0600)
The service module hasn't had a sync to Cinder since November
of 2014.  There have been a number of changes, including updates
to get it using the appropriate namespace for oslo and support
for moving away from using oslo.log.  So, we need to update the
module.

Current HEAD in OSLO:
---------------------
commit e589dde0721a0a67e4030813e582afec6e70d042
Date:  Wed Feb 18 03:08:12 2015 +0000
Merge "Have a little fun with release notes"

Changes merged with this patch:
---------------------
2aacb111 - Change oslo.config to oslo_config
2fbf5065 - Remove oslo.log code and clean up versionutils API
dbca53e9 - Merge "Added graceful argument on Service.stop method"
442fc223 - Added graceful argument on Service.stop method
cf429e50 - Remove extra white space in log message

Change-Id: If78596d8779b8dd5b2fc22891d7ca21ccdabc9d6

cinder/openstack/common/service.py

index e2c57d7d09895da8a20bc9288460b75f35ce8566..910d2bbbcc1b9f0b8599b9b5cc6bedef68ad206c 100644 (file)
@@ -18,7 +18,7 @@
 """Generic Node base class for all workers that run on hosts."""
 
 import errno
-import logging as std_logging
+import logging
 import os
 import random
 import signal
@@ -39,7 +39,6 @@ from oslo.config import cfg
 
 from cinder.openstack.common import eventlet_backdoor
 from cinder.openstack.common._i18n import _LE, _LI, _LW
-from cinder.openstack.common import log as logging
 from cinder.openstack.common import systemd
 from cinder.openstack.common import threadgroup
 
@@ -163,7 +162,7 @@ class ServiceLauncher(Launcher):
         signo = 0
 
         LOG.debug('Full set of CONF:')
-        CONF.log_opt_values(LOG, std_logging.DEBUG)
+        CONF.log_opt_values(LOG, logging.DEBUG)
 
         try:
             if ready_callback:
@@ -377,7 +376,7 @@ class ProcessLauncher(object):
 
         systemd.notify_once()
         LOG.debug('Full set of CONF:')
-        CONF.log_opt_values(LOG, std_logging.DEBUG)
+        CONF.log_opt_values(LOG, logging.DEBUG)
 
         try:
             while True:
@@ -397,7 +396,7 @@ class ProcessLauncher(object):
                 self.running = True
                 self.sigcaught = None
         except eventlet.greenlet.GreenletExit:
-            LOG.info(_LI("Wait called after thread killed.  Cleaning up."))
+            LOG.info(_LI("Wait called after thread killed. Cleaning up."))
 
         self.stop()
 
@@ -434,8 +433,8 @@ class Service(object):
     def start(self):
         pass
 
-    def stop(self):
-        self.tg.stop()
+    def stop(self, graceful=False):
+        self.tg.stop(graceful)
         self.tg.wait()
         # Signal that service cleanup is done:
         if not self._done.ready():