Skip to content

[Synapse] az synapse: Add for managed private endpoints commands - #19117

Merged
Yu Chen (jsntcy) merged 6 commits into
Azure:devfrom
kevinzz6:synapse-managed-private-endpoint
Aug 18, 2021
Merged

Yu Chen (jsntcy) merged 6 commits into
Azure:devfrom
kevinzz6:synapse-managed-private-endpoint

Conversation

@kevinzz6

@kevinzz6 Weishu Zhao (kevinzz6) commented Aug 4, 2021

Copy link
Copy Markdown
Contributor

Description

Add synapse managed private endpoints commands. Included create, show, list and delete managed-private-endpoints.
Add azure-synapse-managedprivateendpoints =0.3.0
Testing Guide

History Notes

[Synapse] az synapse: Add managed private endpoints feature.


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

@ghost

Deleted user (ghost) commented Aug 4, 2021

Copy link
Copy Markdown

CLA assistant check
All CLA requirements met.

@yonzhan

Copy link
Copy Markdown
Collaborator

Synapse

Comment on lines +489 to +491
g.custom_command('create', 'create_Managed_private_endpoints', supports_no_wait=True)
g.custom_command('list', 'list_Managed_private_endpoints')
g.custom_command('delete', 'delete_Managed_private_endpoints', confirmation=True, supports_no_wait=True)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Both create and delete support no wait but there seems no wait command

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

api doesn't suport wait command, so delete supports_no_wait=True.

Comment thread .gitignore Outdated
@kevinzz6

Copy link
Copy Markdown
Contributor Author

Yishi Wang (@evelyn-ys) Dongwei Wang (@idear1203) please take time to review my new commit, thanks.

@jsntcy
Yu Chen (jsntcy) merged commit 66a344e into Azure:dev Aug 18, 2021
Comment thread src/azure-cli/setup.py
'azure-synapse-accesscontrol~=0.5.0',
'azure-synapse-artifacts~=0.6.0',
'azure-synapse-spark~=0.2.0',
'azure-synapse-managedprivateendpoints~=0.3.0',

@jiasli Jiashuo Li (jiasli) Oct 8, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not sorting dependencies alphabetically can frequently cause merge conflicts (#19560's dependency on azure-synapse-artifacts causes merge conflicts in #19554 (1c5770f)).

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.

6 participants