Skip to content

Commit 90d86ef

Browse files
committed
Enable "openstack server unpause" command to take multiple servers.
Current "openstack server unpause" command could only unpause one server. Improve it to be able to handle more than one servers. Also improve the doc to reflect the new feature. Change-Id: I069ebdd6dcd121f6e55c2bf40d42197f93830e0c Implements: blueprint cmd-with-multi-servers
1 parent 6611f37 commit 90d86ef

3 files changed

Lines changed: 12 additions & 10 deletions

File tree

doc/source/command-objects/server.rst

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -658,17 +658,17 @@ Unlock server
658658
server unpause
659659
--------------
660660
661-
Unpause server
661+
Unpause server(s)
662662
663663
.. program:: server unpause
664664
.. code:: bash
665665
666666
os server unpause
667-
<server>
667+
<server> [<server> ...]
668668
669669
.. describe:: <server>
670670
671-
Server (name or ID)
671+
Server(s) to unpause (name or ID)
672672
673673
server unrescue
674674
---------------

doc/source/commands.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,7 @@ Those actions with an opposite action are noted in parens if applicable.
187187
* ``stop`` (``start``) - stop one or more servers
188188
* ``suspend`` (``resume``) - stop a server and save to disk freeing memory
189189
* ``unlock`` (``lock``) - unlock a server
190-
* ``unpause`` (``pause``) - return a paused server to running state
190+
* ``unpause`` (``pause``) - return one or more paused servers to running state
191191
* ``unrescue`` (``rescue``) - return a server to normal boot mode
192192
* ``unset`` (``set``) - remove an attribute of the object
193193

openstackclient/compute/v2/server.py

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1583,7 +1583,7 @@ def take_action(self, parsed_args):
15831583

15841584

15851585
class UnpauseServer(command.Command):
1586-
"""Unpause server"""
1586+
"""Unpause server(s)"""
15871587

15881588
log = logging.getLogger(__name__ + '.UnpauseServer')
15891589

@@ -1592,18 +1592,20 @@ def get_parser(self, prog_name):
15921592
parser.add_argument(
15931593
'server',
15941594
metavar='<server>',
1595-
help=_('Server (name or ID)'),
1595+
nargs='+',
1596+
help=_('Server(s) to unpause (name or ID)'),
15961597
)
15971598
return parser
15981599

15991600
@utils.log_method(log)
16001601
def take_action(self, parsed_args):
16011602

16021603
compute_client = self.app.client_manager.compute
1603-
utils.find_resource(
1604-
compute_client.servers,
1605-
parsed_args.server,
1606-
).unpause()
1604+
for server in parsed_args.server:
1605+
utils.find_resource(
1606+
compute_client.servers,
1607+
server,
1608+
).unpause()
16071609

16081610

16091611
class UnrescueServer(command.Command):

0 commit comments

Comments
 (0)