OSC 'node list' recognizes all provision states
Now OSC 'node list' command with --provision-state argument for
'available' will list all nodes whose provision state is 'available'.
It will also accept all the provision-states for OSC 'node list'
command.
Conflicts:
ironicclient/osc/v1/baremetal_node.py
Change-Id: If0032f652bcb3f6638b59dba32b97cb19176df46
Closes-Bug: #1663202
(cherry picked from commit d55e363904
)
This commit is contained in:
parent
1e5b34dc0f
commit
43ba5c0bd1
|
@ -463,9 +463,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',
|
||||
|
|
|
@ -603,18 +603,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',
|
||||
|
|
|
@ -0,0 +1,4 @@
|
|||
---
|
||||
fixes:
|
||||
- Allows all provision states for OSC node list
|
||||
command with --provision-state argument
|
Loading…
Reference in New Issue