Merge "OSC 'node list' recognizes all provision states"

This commit is contained in:
Jenkins 2017-04-18 16:51:52 +00:00 committed by Gerrit Code Review
commit 3d2389ed1a
3 changed files with 5 additions and 15 deletions

View File

@ -565,9 +565,7 @@ class ListBaremetalNode(command.Lister):
'--provision-state',
dest='provision_state',
metavar='<provision state>',
choices=self.PROVISION_STATES,
help=_("Limit list to nodes in <provision state>. One of %s.") %
(", ".join(self.PROVISION_STATES)))
help=_("List nodes in specified provision state."))
parser.add_argument(
'--resource-class',
dest='resource_class',

View File

@ -779,18 +779,6 @@ class TestBaremetalList(TestBaremetal):
**kwargs
)
def test_baremetal_list_invalid_provision_state(self):
arglist = [
'--provision-state', 'invalid',
]
verifylist = [
('provision_state', 'invalid'),
]
self.assertRaises(oscutils.ParserException,
self.check_parser,
self.cmd, arglist, verifylist)
def test_baremetal_list_resource_class(self):
arglist = [
'--resource-class', 'foo',

View File

@ -0,0 +1,4 @@
---
fixes:
- Allows all provision states for OSC node list
command with --provision-state argument