zuul/zuul
Code Issues Proposed changes

Remove --id option for autohold_delete/autohold_info

Browse Source 
Since the hold request ID is the only argument, simplify things by
removing the need for using the --id option.

Change-Id: I0c176dca3cd7e3007348b5aafae812ebb55556d2
This commit is contained in:
David Shrewsbury
2019-10-24 13:48:14 -04:00
parent e85fb93d1d
commit e16810d178
2 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
zuul/cmd/client.py
View File

@@ -204,16 +204,14 @@ class Client(zuul.cmd.ZuulApp):
cmd_autohold_delete = subparsers.add_parser(
'autohold-delete', help='delete autohold request')
cmd_autohold_delete.set_defaults(func=self.autohold_delete)
cmd_autohold_delete.add_argument('--id',
help='request ID',
required=True)
cmd_autohold_delete.add_argument('id', metavar='REQUEST_ID',
help='the hold request ID')
cmd_autohold_info = subparsers.add_parser(
'autohold-info', help='retrieve autohold request detailed info')
cmd_autohold_info.set_defaults(func=self.autohold_info)
cmd_autohold_info.add_argument('--id',
help='request ID',
required=True)
cmd_autohold_info.add_argument('id', metavar='REQUEST_ID',
help='the hold request ID')
cmd_autohold_list = subparsers.add_parser(
'autohold-list', help='list autohold requests')