-
Notifications
You must be signed in to change notification settings - Fork 301
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
Cloud Shell doesn't allow running the TF Azure AD 2.x code due to MSI error #633
Comments
|
Hi @marcosgm, thanks for reporting this. We're aware of some issues around MSI authentication in Cloud Shell, which we'll work to address. As a workaround, Azure CLI authentication does work. Microsoft install a wrapper script at |
wow it works!!
|
@manicminer let us know if you think this is due to an error in Cloud Shell. I can reproduce the issue but I'm not clear what the problem is. |
@edyoung I first thought it was a cloudshell bug Azure/CloudShell#112 But following this tutorial https://kevinhakanson.com/2020-04-22-exploring-the-microsoft-graph-api-from-azure-cloud-shell So maybe it's the terraform azuread code that doesn't work well with that Proxy, see for instance how it's done with curl Thoughts? |
https://github.com/manicminer/hamilton/blob/main/auth/msi.go this is the auth library used by this azuread provider. my 2 cents |
I went ahead and tested the Go library (Hamilton) and filed a bug as they don't support the MSI endpoints inside cloudshell manicminer/hamilton#116 |
@edyoung Thanks for reaching out - this isn't strictly due to an error in Cloud Shell, rather it's arisen from an assumption that MSI endpoints respond similarly in various environments. There is a sanity check at configuration time that tries to get a 200 response from the metadata service, but either there isn't such a service in Cloud Shell or the URL is different. It's not a huge problem, the sanity check adds minimal value and I believe we can remove it. |
This functionality has been released in v2.9.0 of the Terraform Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you! |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
see Azure/CloudShell#112
The text was updated successfully, but these errors were encountered: