openstack
/
cinder
Code Issues Proposed changes

Merge "Change 'host' option from HostAddressOpt to StrOpt" into stable/wallaby

This commit is contained in:
Zuul 2021-09-24 18:09:00 +00:00 committed by Gerrit Code Review
parent 80ef8c1046 498608a840
commit 67cb7fbc1c
2 changed files with 13 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
cinder/common/config.py
View File

@ -84,12 +84,12 @@ global_opts = [
cfg.StrOpt('scheduler_manager',
default='cinder.scheduler.manager.SchedulerManager',
help='Full class name for the Manager for scheduler'),
cfg.HostAddressOpt('host',
sample_default='localhost',
default=socket.gethostname(),
help='Name of this node. This can be an opaque '
'identifier. It is not necessarily a host name, '
'FQDN, or IP address.'),
cfg.StrOpt('host',
sample_default='localhost',
default=socket.gethostname(),
help='Name of this node. This can be an opaque '
'identifier. It is not necessarily a host name, '
'FQDN, or IP address.'),
# NOTE(vish): default to nova for compatibility with nova installs
cfg.StrOpt('storage_availability_zone',
default='nova',

7
releasenotes/notes/incorrect-host-config-option-347e60f957458d54_new.yaml Normal file
View File

@ -0,0 +1,7 @@
---
fixes:
- |
`Bug #1941068 <https://bugs.launchpad.net/cinder/+bug/1941068>`_: Fixed
type of the ``host`` configuration option. It was limited to valid FQDN
values when we document that it isn't. This may result in the
``cinder-manage db sync`` command failing.