Yuchao Yan

Results 128 comments of Yuchao Yan

Hi @pmatte according to the [swagger definition](https://github.com/Azure/azure-rest-api-specs/blob/f38115ac455af89493b0a0719d9a987404560dda/specification/resources/resource-manager/Microsoft.Resources/stable/2021-01-01/subscriptions.json#L165-L166), `tenants_list` "Gets the tenants for your account", so I think calling the api needs access across different tenant. But service principal only has...

Hi @petrkotas I am only Python SDK owner and I am OK for this change. For other review support, please seek help in [teams channel](https://teams.microsoft.com/l/channel/19%3a0351f5f9404446e4b4fd4eaf2c27448d%40thread.skype/API%2520Spec%2520Review?groupId=3e17dcb0-4257-4a30-b843-77f47f1d4121&tenantId=72f988bf-86f1-41af-91ab-2d7cd011db47)

The ModelValidation error is not caused by this PR

some discussion is in e-mail `RE: Azure Recovery Service Site Recovery CLI PLR`

Hi @danieljurek Please help review: https://github.com/Azure/azure-sdk-for-python/pull/27531

Hi @sima-zhu @danieljurek the following link is broken, could you help us how to fix it? Thanks! ![image](https://user-images.githubusercontent.com/70930885/207266517-02fa95a1-6b43-4c0b-b5a7-eb17596a0fc2.png) (https://learn.microsoft.com/en-us/python/api/overview/azure/traffic-manager?view=azure-python)

Hi @sima-zhu I think the right link shall be https://learn.microsoft.com/en-us/python/api/overview/azure/mgmt-trafficmanager-readme?view=azure-python By the way, I am wondering why different services have different Overview. For example, ![image](https://user-images.githubusercontent.com/70930885/210470552-f705912d-51a4-408c-9d53-dac1deab0b70.png) [Traffic Manager Overview](https://learn.microsoft.com/en-us/python/api/overview/azure/traffic-manager?view=azure-python) ![image](https://user-images.githubusercontent.com/70930885/210470401-bcff0a0f-605f-48a5-b546-8584167bf683.png) [Web...

@sima-zhu Got it and please help merge https://github.com/MicrosoftDocs/azure-docs-sdk-python/pull/1669

@NiveditJain I notice that you rename many operation. Could you run live test and commit your recording files?

Hi @NiveditJain The PR overall looks good. I notice that there is empty file `_polling.py` which I think shall be removed.