Skip to content

Conversation

@mmyyrroonn
Copy link
Contributor

Description

Fix #13018

image

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


This checklist is used to make sure that common guidelines for a pull request are followed.

@mmyyrroonn mmyyrroonn requested a review from haroldrandom as a code owner May 15, 2020 03:00
@yonzhan yonzhan requested a review from jsntcy May 15, 2020 05:00
@yonzhan yonzhan added this to the S170 milestone May 15, 2020
@yonzhan
Copy link
Collaborator

yonzhan commented May 15, 2020

Network

c.argument('virtual_network_name', help='The virtual network (VNet) associated with the subnet (Omit if supplying a subnet id).', metavar='', id_part=None)
c.argument('private_connection_resource_id', help='The resource id of which private enpoint connect to')
c.argument('group_ids', nargs='+', help='The ID(s) of the group(s) obtained from the remote resource that this private endpoint should connect to. You can use "az keyvault(storage/etc) private-link-resource list" to obtain the list of group ids.')
c.argument('group_ids', nargs='+', options_list=[c.deprecate(target='--group-ids', redirect='--group-id'), '--group-id'], help='The ID of the group obtained from the remote resource that this private endpoint should connect to. You can use "az network private-link-resource list" to obtain the list of group ids.')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

c.deprecate(target='--group-ids', redirect='--group-id') inside options_list ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mmyyrroonn mmyyrroonn merged commit 530cd07 into Azure:dev May 18, 2020
@mmyyrroonn mmyyrroonn deleted the fix-13018-remove-group-ids branch May 18, 2020 02:10
derekbekoe added a commit to derekbekoe/azure-cli that referenced this pull request May 18, 2020
* 'dev' of https://github.com/Azure/azure-cli: (85 commits)
  {Monitor} az monitor autoscale profile show: bug fix (Azure#13515)
  [Monitor] az monitor diagnostic-settings create: support --export-to-specific-resource argument (Azure#13544)
  [aro]Add tests for generate_random_id function (Azure#13522)
  [ARO]Change CLIError to correct flag for --worker-vm-disk-size-gb (Azure#13439)
  [Packaging] Add Ubuntu Focal Package (Azure#13491)
  Update parse_zone_file.py (Azure#13377)
  {Release} Upgrade to Azure CLI 2.6.0 (Azure#13542)
  fix (Azure#13511)
  {Resource} Add example to tell how to pass array to --parameters (Azure#13510)
  [NetAppFiles] Anf 5207 Bugfix - add missing snapshot restore function (Azure#13481)
  {Network} az network nic create: Refine help message (Azure#13513)
  [Storage] az storage blob sync: Fix the incorrect error message when azcopy cannot find the installation location (Azure#9576)  (Azure#13104)
  Support --connect-string for az storage blob sync (Azure#13135)
  Modify the deprecate information for deployment operation (Azure#13390)
  [AppService] Onboard local context for app service (Azure#12984)
  {Monitor} az monitor metrics alert: refine severity explanation (Azure#13512)
  [IoT Hub] Support for 2020-03-01 API and Network Isolation commands (Azure#13467)
  [Compute] az vm list-skus: Update --zone behavior, return all type skus now (Azure#13470)
  Fix that cli doe not fail when user only specifies Windows password (Azure#13418)
  {ACR}: expose --expiration from token commands (Azure#13451)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

--group-ids should just support one group id in private endpoint create

3 participants