Skip to content

Commit

Permalink
Merge pull request Azure#22 from daniv-msft/snehapar/miscellaneous
Browse files Browse the repository at this point in the history
Addressed PR comments
  • Loading branch information
daniv-msft authored May 17, 2023
2 parents ffaf38e + c1e4413 commit d2a892c
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 15 deletions.
14 changes: 2 additions & 12 deletions src/containerapp/azext_containerapp/_params.py
Original file line number Diff line number Diff line change
Expand Up @@ -442,17 +442,7 @@ def load_arguments(self, _):
c.argument('max_nodes', help="The maximum node count for the workload profile")

# Patch
with self.argument_context('containerapp patch list') as c:
with self.argument_context('containerapp patch') as c:
c.argument('resource_group_name', arg_type=resource_group_name_type)
c.argument('managed_env', options_list=['--environment', '-e'], help='Name or resource id of the Container App environment.')
c.argument('show_all', help='Show all patchable and unpatchable container apps')

with self.argument_context('containerapp patch apply') as c:
c.argument('resource_group_name', arg_type=resource_group_name_type)
c.argument('managed_env', validator=validate_managed_env_name_or_id, options_list=['--environment', '-e'], help='Name or resource id of the Container App environment.')
c.argument('show_all', help='Show all patchable and unpatchable container apps')

with self.argument_context('containerapp patch interactive') as c:
c.argument('resource_group_name', arg_type=resource_group_name_type)
c.argument('managed_env', options_list=['--environment', '-e'], help='Name or resource id of the Container App environment.')
c.argument('show_all', help='Show all patchable and unpatchable container apps')
c.argument('show_all', action='store_true', help='Show all patchable and unpatchable container apps')
6 changes: 3 additions & 3 deletions src/containerapp/azext_containerapp/commands.py
Original file line number Diff line number Diff line change
Expand Up @@ -214,6 +214,6 @@ def load_command_table(self, _):
g.custom_command('delete', 'delete_workload_profile')

with self.command_group('containerapp patch', is_preview=True) as g:
g.custom_command('list', 'patch_list', is_preview=True)
g.custom_command('apply', 'patch_apply', is_preview=True)
g.custom_command('interactive', 'patch_interactive', is_preview=True)
g.custom_command('list', 'patch_list')
g.custom_command('apply', 'patch_apply')
g.custom_command('interactive', 'patch_interactive')

0 comments on commit d2a892c

Please sign in to comment.