azure-sdk-for-python icon indicating copy to clipboard operation
azure-sdk-for-python copied to clipboard

Published 20 hours ago • verified publisher icon Azure

Generated SDK weird type: Components1Jq1T4ISchemasManagedserviceidentityPropertiesUserassignedidentitiesAdditionalproperties

Open logachev opened this issue 3 years ago • 12 comments

  • Package Name:
  • Package Version:
  • Operating System:
  • Python Version:

Describe the bug Some SDKs Models have a type with this name: Components1Jq1T4ISchemasManagedserviceidentityPropertiesUserassignedidentitiesAdditionalproperties

https://github.com/Azure/azure-sdk-for-python/blob/2d32ffe99da7194ae721e209137025da220206d4/sdk/cosmos/azure-mgmt-cosmosdb/azure/mgmt/cosmosdb/models/init.py#L34

I guess this is some typo in the REST specs or issue with generator?

To Reproduce Steps to reproduce the behavior: 1.

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

Additional context Add any other context about the problem here.

logachev avatar Mar 10 '21 21:03 logachev

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @southpolesteve, @zfoster

ghost avatar Mar 10 '21 21:03 ghost

Thanks for the feedback, we'll investigate asap. adding @msyyc who could comment further on the mgmt libraries.

yunhaoling avatar Mar 11 '21 11:03 yunhaoling

@lmazuel It seems a known issue. Could you help to confirm it?

msyyc avatar Mar 12 '21 02:03 msyyc

@yunhaoling Please provide more detailed info : image

msyyc avatar Mar 15 '21 03:03 msyyc

@msyyc It is an issue with a bunch of different SDKs. You can search for this string in this repo to get all entries.. This might be the easiest way.

logachev avatar Mar 15 '21 20:03 logachev

Hi @logachev. Thank you for opening this issue and giving us the opportunity to assist. We believe that this has been addressed. If you feel that further discussion is needed, please add a comment with the text “/unresolve” to remove the “issue-addressed” label and continue the conversation.

ghost avatar Dec 15 '21 08:12 ghost

Hi @logachev, since you haven’t asked that we “/unresolve” the issue, we’ll close this out. If you believe further discussion is needed, please add a comment “/unresolve” to reopen the issue.

ghost avatar Dec 22 '21 10:12 ghost

/unresolve

Not sure why it was even marked as resolved. Now there are 2 entries that are clearly some generator/data issue

https://github.com/Azure/azure-sdk-for-python/blob/84a9eb69dc38332434645d6556703716ff549fa7/sdk/cosmos/azure-mgmt-cosmosdb/azure/mgmt/cosmosdb/models/init.py#L45

logachev avatar Dec 23 '21 17:12 logachev

@RAY-316 I still see it in latest azure-mgmt-cosmosdb 7.0.0b5. Please double check.

xiangyan99 avatar May 20 '22 22:05 xiangyan99

Hi @logachev We find the cause is that there is too deep nest in swagger: https://github.com/Azure/azure-rest-api-specs/blob/6c065f8439e6d9cd02bf4f8268fae7f86a5f0fa9/specification/cosmos-db/resource-manager/Microsoft.DocumentDB/preview/2022-05-15-preview/managedCassandra.json#L1083-L1170. We will rename it in swagger and release a new version. Thanks for the patience!

msyyc avatar Aug 11 '22 08:08 msyyc

There is no weird type in latest version: https://pypi.org/project/azure-mgmt-cosmosdb/8.0.0b2/

msyyc avatar Aug 12 '22 01:08 msyyc

Hi @logachev. Thank you for opening this issue and giving us the opportunity to assist. We believe that this has been addressed. If you feel that further discussion is needed, please add a comment with the text “/unresolve” to remove the “issue-addressed” label and continue the conversation.

ghost avatar Aug 12 '22 15:08 ghost

Hi @logachev, since you haven’t asked that we “/unresolve” the issue, we’ll close this out. If you believe further discussion is needed, please add a comment “/unresolve” to reopen the issue.

ghost avatar Aug 19 '22 16:08 ghost