nova-manage cell_v2 verify_instance has unnecessary uuid check
Bug #1656479 reported by
Matt Riedemann
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Fix Released
|
Low
|
Matt Riedemann | ||
Newton |
Confirmed
|
Low
|
Unassigned |
Bug Description
The nova-manage cell_v2 verify_instance command has a check to see if the uuid argument is not provided and then fails, breaking it's own rule about not printing anything if --quiet is used. However, it doesn't even need that check because argparse will handle the validation of --uuid not being provided:
stack@ocata:~$ nova-manage cell_v2 verify_instance --quiet
usage: nova-manage cell_v2 verify_instance [-h] --uuid <uuid> [--quiet]
nova-manage cell_v2 verify_instance: error: argument --uuid is required
stack@ocata:~$ echo $?
2
Changed in nova: | |
assignee: | nobody → Matt Riedemann (mriedem) |
To post a comment you must log in.
Fix proposed to branch: master /review. openstack. org/420261
Review: https:/