Skip to content

[WIP] KeyVault Track 2 SDK adoption #10128

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

Closed
wants to merge 5 commits into from

Conversation

bim-msft
Copy link

@bim-msft bim-msft commented Mar 6, 2020

WIP: DO NOT MERGE

@bim-msft bim-msft requested a review from yungezz March 6, 2020 08:00
@bim-msft bim-msft requested a review from zikalino as a code owner March 6, 2020 08:00
@bim-msft bim-msft self-assigned this Mar 6, 2020
@adxsdk6
Copy link

adxsdk6 commented Mar 6, 2020

Can one of the admins verify this patch?

@yungezz
Copy link
Member

yungezz commented Mar 6, 2020

this PR is experiment for Track 2 SDK generation, pls do NOT review or merge. thanks.

@bim-msft bim-msft changed the title [WIP] KeyVault Track 2 adoption [WIP] KeyVault Track 2 SDK adoption Mar 23, 2020
@bim-msft bim-msft force-pushed the bim_kv_track2 branch 2 times, most recently from 19c5284 to 5921a05 Compare March 23, 2020 07:13
# license information.
# --------------------------------------------------------------------------

VERSION = "0.1.0"
Copy link
Member

Choose a reason for hiding this comment

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

3.0.0b1 ?

Copy link
Author

Choose a reason for hiding this comment

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

3.0.0b1 ?

Thanks, I will bump the version here.

@iscai-msft
Copy link
Contributor

hey @bim-msft can you regenerate using https://github.com/Azure/autorest.python/releases/tag/v5.0.0-dev.20200326.1?

This will remove the "azure-" prefix from the user agent name to fulfill library guidelines (also includes other fixes, not sure how / if they apply here)

Thanks so much!

@yungezz
Copy link
Member

yungezz commented Mar 28, 2020

hey @bim-msft can you regenerate using https://github.com/Azure/autorest.python/releases/tag/v5.0.0-dev.20200326.1?

This will remove the "azure-" prefix from the user agent name to fulfill library guidelines (also includes other fixes, not sure how / if they apply here)

Thanks so much!

Thanks @iscai-msft for the info. Just a question about userAgent in general, should azure- prefix been removed from all languages' SDKs? thanks!

@bim-msft
Copy link
Author

hey @bim-msft can you regenerate using https://github.com/Azure/autorest.python/releases/tag/v5.0.0-dev.20200326.1?

This will remove the "azure-" prefix from the user agent name to fulfill library guidelines (also includes other fixes, not sure how / if they apply here)

Thanks so much!

@iscai-msft Thanks, I will have a try.

@@ -119,16 +120,21 @@ def __init__(self, client, config, serializer, deserializer) -> None:
:param private_endpoint_connection_name: Name of the private endpoint connection associated
with the key vault.
:type private_endpoint_connection_name: str
:param properties: The intended state of private endpoint connection.
:type properties: ~azure.mgmt.keyvault.v2018_02_14.models.PrivateEndpointConnection
:param private_endpoint: Properties of the private endpoint object.
Copy link
Contributor

@iscai-msft iscai-msft Apr 2, 2020

Choose a reason for hiding this comment

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

Just want to make sure: when you regenerated this time, did you specify a payload flattening threshold? Did your command change in any ways?

Copy link
Author

@bim-msft bim-msft Apr 3, 2020

Choose a reason for hiding this comment

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

Just want to make sure: when you regenerated this time, did you specify a payload flattening threshold? Did your command change in any ways?

@iscai-msft No, I didn't change my regen command.

autorest --python --use=/mnt/c/repos/autorest.python /mnt/c/repos/azure-rest-api-specs/specification/keyvault/resource-manager/readme.md --multiapi --python-sdks-folder=/mnt/c/repos/azure-sdk-for-python/sdk --debug

Regened again, nothing changed.

@bim-msft
Copy link
Author

Closing it due to we have a larger coverage now: #11484

@bim-msft bim-msft closed this Jun 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants