yahoo-eng-team team mailing list archive

[OpenStack Infra <1821737@xxxxxxxxxxxxxxxxxx>]

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

commit 8ab3300d5d588f169b40ca7dbc6cd63370226066
Author: Chris Dent <cdent@xxxxxxxxxxxxx>
Date:   Tue Apr 2 15:07:45 2019 +0100

    Don't report 'exiting' when mapping cells
    
    When running 'nova-manage simple_cell_setup...' if there are not hosts
    to map, but there remaining instances to map, an '..., exiting' message
    is produced. This is misleading because "exiting" implies a return of
    control to the user. That doesn't happen if there are many instances
    left to inspect or map.
    
    This change gets around that by getting rid of the exiting message
    in the case where instance mapping can still happen.
    
    Change-Id: I62b20a3676429b5cc756884275138566785b347e
    Closes-Bug: #1821737


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

Title:
  simple_cell_setup reports "exiting" when it is not

Status in OpenStack Compute (nova):
  Fix Released

Bug description:
  When running 'nova-manage simple_cell_setup ...' if hosts are already
  the _map_cell_and_hosts method prints a message of 'All hosts are
  already mapped to cell(s), exiting.' and then proceeds to map
  instances. It does not, in fact, exit.

  This isn't the end of the world, but is somewhat confusing.

  The easiest fix is probably to get rid of ', exiting'. Then in the
  multiple paths to the method, the printed message still makes sense
  and 'exiting' can be implicit.

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