openstack
/
ceilometermiddleware
Code Issues Proposed changes

Convert nonblocking_notify value from string to boolean 

Otherwise just setting it to "False" which is a string and therefore, is
True, enables it!

(cherry picked from commit 87bd49b62e)

Change-Id: I1155e1a16ccd34ffb0cd1e966e0bef91f6257b6d
This commit is contained in:
Julien Danjou 2016-11-07 18:36:18 +01:00
parent bc25de190d
commit a170716d3f
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ceilometermiddleware/swift.py
View File

@ -50,6 +50,7 @@ import logging
from oslo_config import cfg
import oslo_messaging
from oslo_utils import strutils
from oslo_utils import timeutils
from pycadf import event as cadf_event
from pycadf.helper import api
@ -142,7 +143,8 @@ class Swift(object):
# For backward compatibility we default to False and therefore wait for
# sending to complete. This causes swift proxy to hang if the
# destination is unavailable.
self.nonblocking_notify = conf.get('nonblocking_notify', False)
self.nonblocking_notify = strutils.bool_from_string(
conf.get('nonblocking_notify', False))
# Initialize the sending queue and thread, but only once
if self.nonblocking_notify and Swift.event_queue is None: