Skip to content

Conversation

@tjegbejimba
Copy link
Contributor

@tjegbejimba tjegbejimba commented Apr 14, 2022

Description
This PR enables private link commands to create, update, list, show, and delete resource manager private links and private link associations. Microsoft.Resources is updated to use the latest Python SDK.

Testing Guide

History Notes

[ARM] az resourcemanagement private-link create: Create Resource management private link
[ARM] az resourcemanagement private-link delete: Delete Resource management private link
[ARM] az resourcemanagement private-link show: Get Resource management private link
[ARM] az resourcemanagement private-link list: List Resource management private link
[ARM] az private-link association create: Create private link association
[ARM] az private-link association delete: Delete private link association
[ARM] az private-link association show: Get private link association
[ARM] az private-link association list: List private link association


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

@ghost ghost added the Auto-Assign Auto assign by bot label Apr 14, 2022
@ghost ghost requested review from yonzhan and zhoxing-ms April 14, 2022 12:03
@ghost ghost assigned zhoxing-ms Apr 14, 2022
@ghost ghost added this to the Apr 2022 (2022-04-26) milestone Apr 14, 2022
@ghost ghost added the ARM az resource/group/lock/tag/deployment/policy/managementapp/account management-group label Apr 14, 2022
@yonzhan yonzhan removed the ARM az resource/group/lock/tag/deployment/policy/managementapp/account management-group label Apr 14, 2022
@yonzhan
Copy link
Collaborator

yonzhan commented Apr 14, 2022

private links

@yonzhan yonzhan requested a review from necusjz April 18, 2022 23:04
@tjegbejimba tjegbejimba marked this pull request as ready for review April 20, 2022 10:11
@tjegbejimba tjegbejimba changed the title Arm private links [ARM] az resourcemanagement private link: Add new commands for resourcemanagementprivatelinks; [ARM] az private-link association: Add new commands for privatelinkassociations Apr 20, 2022
@zhoxing-ms
Copy link
Contributor

@tjegbejimba Please address these CI issues

@zhoxing-ms
Copy link
Contributor

In addition, if you want to write multiple descriptions in the history note, please refer to submitting-pull-requests and write it in the History Notes in PR description, such as: #21843

@zhoxing-ms
Copy link
Contributor

zhoxing-ms commented Apr 21, 2022

@tjegbejimba Since this PR was submitted late, it cannot catch up with the release of this sprint. We will plan to release it in the next sprint (05-24), is this time acceptable?

@zhoxing-ms
Copy link
Contributor

It is recommended not to modify the format of these files, which will lead to too many irrelevant changes in this PR

@zhoxing-ms
Copy link
Contributor

zhoxing-ms commented Apr 29, 2022

Sorry, you still need to solve the linter issue (I didn't aware you changed so many code styles to solve the linter issue, I thought it was the automatic formatting from IDE)
And please add the AzureServiceName and Doc URL in src/azure-cli/service_name.json to solve the CLI style issue.

@zhoxing-ms zhoxing-ms changed the title [ARM] az resourcemanagement private-link: Add new commands for resourcemanagementprivatelinks; [ARM] az private-link association: Add new commands for privatelinkassociations [ARM] az resourcemanagement private-link and az private-link association: Add new command groups May 3, 2022
Comment on lines +427 to +431
{
"Command": "az resourcemanagement",
"AzureServiceName": "Azure Resource Manager",
"URL": "https://docs.microsoft.com/en-us/azure/azure-resource-manager/management/create-private-link-access-portal"
},
Copy link
Contributor

Choose a reason for hiding this comment

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

https://docs.microsoft.com/en-us/azure/azure-resource-manager/management/create-private-link-access-portal

Are you sure the document link of az resourcemanagement is create-private-link-access-portal?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Out of curiosity, shouldn't the private-link be just a part of resource management? Shouldn't Azure Resource Manager documentation | Microsoft Docs be a more appropriate doc link for the az resourcemanagement command?

Copy link
Member

Choose a reason for hiding this comment

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

The Private Link work is a subsection of resource management with several specific constraints. As far as helpfulness to the user, the private link specific documentation would be more useful.

Copy link
Contributor

Choose a reason for hiding this comment

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

OK, thanks~

@zhoxing-ms zhoxing-ms merged commit 4223f08 into Azure:dev May 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Auto-Assign Auto assign by bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants