Reviewed:  https://review.opendev.org/c/openstack/nova/+/938053
Committed: 
https://opendev.org/openstack/nova/commit/2c8400043d9c5b088815a48f20aef99d58f1488d
Submitter: "Zuul (22348)"
Branch:    master

commit 2c8400043d9c5b088815a48f20aef99d58f1488d
Author: Dr. Jens Harbott <harb...@osism.tech>
Date:   Thu Dec 19 13:42:52 2024 +0100

    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


** Changed in: nova
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/2092194

Title:
  Official service type for cinder doesn't pass catalog_info regex

Status in OpenStack Compute (nova):
  Fix Released

Bug description:
  The official service type for Cinder according to
  https://specs.openstack.org/openstack/service-types-authority/ is
  "block-storage", but getting nova to use that via a config like

  [cinder]
  catalog_info = block-storage::internalURL

  leads to a failure:

  2024-12-04 16:53:58.852 7 CRITICAL nova [None
  req-97d97c2e-6fde-459c-830b-e35ae21cbf8c - - - - - -] Unhandled error:
  oslo_config.cfg.ConfigFileValueError: Value for option catalog_info
  from LocationInfo(location=<Locations.user: (4, True)>,
  detail='/etc/nova/nova.conf') is not valid: Value 'block-
  storage::internalURL' doesn't match regex '^\\w+:\\w*:.*$'

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/2092194/+subscriptions


-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to     : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to