yahoo-eng-team team mailing list archive

[OpenStack Infra <1751692@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.openstack.org/558089
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=b3c3f2fda883750736bb16d3bb34fd446cc0190a
Submitter: Zuul
Branch:    master

commit b3c3f2fda883750736bb16d3bb34fd446cc0190a
Author: imacdonn <iain.macdonnell@xxxxxxxxxx>
Date:   Tue Apr 17 18:57:24 2018 +0000

    Remove placement config check
    
    Removes a check that prevented compute from starting when config option
    placement.[os_]region_name was not set. The check was not appropriate
    since that option is not (otherwise) required. The check was originally
    added to catch upgrades to Ocata, where placement (newly) needed to be
    configured. Since we're well past that point now, the check is no longer
    necessary.
    
    Change-Id: I973180d6a384b32838ab61d4e6aaf73c255fd116
    Closes-Bug: 1751692


** 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/1751692

Title:
  os_region_name an unnecessary required option for placement

Status in OpenStack Compute (nova):
  Fix Released

Bug description:
  When configuring placement service for nova-computes it is required to
  put in the region name for the placement services.

  When talking to other services like neutron or cinder specifying a
  region name isn't required and if you just have 1 region (possibly the
  most common type of installation?) then it will pick that.

  Would be nice if we didn't have to specify the region for placement
  too.


  Code is in nova/compute/manager.py

          # NOTE(sbauza): We want the compute node to hard fail if it won't be                                                                                                                                
          # able to provide its resources to the placement API, or it will not                                                                                                                                
          # be able to be eligible as a destination.                                                                                                                                                          
          if CONF.placement.os_region_name is None:                                                                                                                                                           
              raise exception.PlacementNotConfigured()

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