Set a min and max on the connection_debug option

There is a set allowable minimum and maximum to the connection_debug
option of 0 and 100. The config option should enforce this.

Change-Id: Ica610ad8fc0875a8be1042f7daba5d0a0c1aad03
This commit is contained in:
Eric Brown
2016-06-29 17:26:02 -07:00
committed by Roman Podoliaka
parent 58d60ca5b2
commit e0cc306c0a
3 changed files with 9 additions and 1 deletions

View File

@@ -101,6 +101,7 @@ database_opts = [
'SQLAlchemy.'),
cfg.IntOpt('connection_debug',
default=0,
min=0, max=100,
deprecated_opts=[cfg.DeprecatedOpt('sql_connection_debug',
group='DEFAULT')],
help='Verbosity of SQL debugging information: 0=None, '

View File

@@ -93,7 +93,7 @@ def _setup_logging(connection_debug=0):
"""
if connection_debug >= 0:
logger = logging.getLogger('sqlalchemy.engine')
if connection_debug >= 100:
if connection_debug == 100:
logger.setLevel(logging.DEBUG)
elif connection_debug >= 50:
logger.setLevel(logging.INFO)

View File

@@ -0,0 +1,7 @@
---
upgrade:
- The allowed values for the ``connection_debug`` option are now restricted to
the range between 0 and 100 (inclusive). Previously a number lower than 0
or higher than 100 could be given without error. But now, a
``ConfigFileValueError`` will be raised when the option value is outside this
range.