-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[Containerapp] az containerapp env http-route-config/premium-ingress: Remove GA commands which exist in azure-cli version 2.79.0
#9265
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 4 commits
f9ea847
b08c7d2
9a2db95
c7898ce
42d100c
f6091dc
1bb1796
959f14f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -39,13 +39,6 @@ def load_command_table(self, args): | |
| g.custom_command('delete', 'delete_managed_environment', supports_no_wait=True, confirmation=True, exception_handler=ex_handler_factory()) | ||
| g.custom_command('update', 'update_managed_environment', supports_no_wait=True, exception_handler=ex_handler_factory()) | ||
|
|
||
| with self.command_group('containerapp env http-route-config', is_preview=True) as g: | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. To avoid breaking change, we can remove the
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. what if we just wait to merge this until after the azure-cli release?
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @Tratcher It depends on if you mind customer will see the
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. But yes we can just wait to merge until after the azure-cli release. |
||
| g.custom_show_command('show', 'show_http_route_config') | ||
| g.custom_command('list', 'list_http_route_configs') | ||
| g.custom_command('create', 'create_http_route_config', exception_handler=ex_handler_factory()) | ||
| g.custom_command('update', 'update_http_route_config', exception_handler=ex_handler_factory()) | ||
| g.custom_command('delete', 'delete_http_route_config', confirmation=True, exception_handler=ex_handler_factory()) | ||
|
|
||
| with self.command_group('containerapp job') as g: | ||
| g.custom_show_command('show', 'show_containerappsjob') | ||
| g.custom_command('list', 'list_containerappsjob') | ||
|
|
@@ -294,9 +287,3 @@ def load_command_table(self, args): | |
| with self.command_group('containerapp revision label') as g: | ||
| g.custom_command('add', 'add_revision_label') | ||
| g.custom_command('remove', 'remove_revision_label') | ||
|
|
||
| with self.command_group('containerapp env premium-ingress', is_preview=True) as g: | ||
| g.custom_show_command('show', 'show_environment_premium_ingress') | ||
| g.custom_command('add', 'add_environment_premium_ingress') | ||
| g.custom_command('update', 'update_environment_premium_ingress') | ||
| g.custom_command('remove', 'remove_environment_premium_ingress', confirmation=True) | ||

Uh oh!
There was an error while loading. Please reload this page.