search

Azure / azure-sdk-for-python

This repository is for active development of the Azure SDK for Python. For consumers of the SDK we recommend visiting our public developer docs at https://learn.microsoft.com/python/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-python.
MIT License
4.64k stars 2.84k forks source link

Managed identity is wrongly recognized as User identity (credential passthrough) #34277

Open bupt-wenxiaole opened 9 months ago

bupt-wenxiaole commented 9 months ago

Describe the bug Set a node (for example, a command node) in a pipeline job with managed identity configuration, but the job was recognized as user identity configuration, also, the raw json of the job showed that the credential passthrough as true. To Reproduce Steps to reproduce the behavior:

  1. submit with job with managed identity configuration image 2.check the job raw json. image
xiangyan99 commented 9 months ago

When you provide the client_id, we treat it as user assigned managed identity.

Do you want to use system assigned managed identity?

github-actions[bot] commented 9 months ago

Hi @bupt-wenxiaole. Thank you for opening this issue and giving us the opportunity to assist. To help our team better understand your issue and the details of your scenario please provide a response to the question asked above or the information requested above. This will help us more accurately address your issue.

bupt-wenxiaole commented 9 months ago

Thanks @xiangyan99 , actually I am not trying to use system assigned managed identity, my point is when I using the user assigned managed identity, the credential passthrough enabled during runtime, this is a bug, suppose the credential passthrough only enabled when prepare_data.identity = UserIdentityConfiguration()

github-actions[bot] commented 9 months ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @Azure/azure-ml-sdk @azureml-github @diondrapeck @kdestin @luigiw @MilesHolland @needuv @ninghu @paulshealy1 @singankit @YusakuNo1.

github-actions[bot] commented 9 months ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @Azure/azure-ml-sdk @azureml-github @diondrapeck @kdestin @luigiw @MilesHolland @needuv @ninghu @paulshealy1 @singankit @YusakuNo1.

banibrata-de commented 9 months ago

@eniac871 could you please help looking into this

bupt-wenxiaole commented 9 months ago

I have a PR to fix this issue.