Skip to content

Commit acc46c5

Browse files
osfricklermmalchuk
authored andcommitted
Allow hyphen in cinder catalog_info service-type
The official service type for Cinder according to [0] is "block-storage", so it should be allowed in the validation regex. [0] https://specs.openstack.org/openstack/service-types-authority/ Closes-Bug: 2092194 Change-Id: I9a57aa72cc9589cec08b0f8d5797108043747ef6 (cherry picked from commit 2c84000) (cherry picked from commit 137097d)
1 parent 0b1d384 commit acc46c5

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

nova/conf/cinder.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
cinder_opts = [
2525
cfg.StrOpt('catalog_info',
2626
default='volumev3::publicURL',
27-
regex=r'^\w+:\w*:.*$',
27+
regex=r'^[\w-]+:\w*:.*$',
2828
help="""
2929
Info to match when looking for cinder in the service catalog.
3030
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
---
2+
fixes:
3+
- |
4+
Nova now allows to use a hyphen in the ``[cinder]catalog_info``
5+
service-type field, so in particular the official
6+
``block-storage`` type is now valid.
7+
`Bug 2092194 <https://bugs.launchpad.net/nova/+bug/2092194>`_

0 commit comments

Comments
 (0)