-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Feature request: MSI support for APP Service #1106
Comments
@pixelicous what's the scenario of using MSI ? Is this for the authentication for Terraform to provision resource ? If you want to use Terraform with MSI on Azure VM, you probably can try this: And MSI support for App Service is supported now: |
@metacpp heads up that this feature request is to support MSI within App Services as we do for other resources, such as Virtual Machines. Support for this is available in the SDK here :) |
@metacpp @tombuildsstuff we provision app services through terraform, today the only reason we need to call an arm template is to set https and add msi, https will be supported on 1.3. 3 but msi still not.. Adding the ability to bind ssl to "custom domain" would be amazing too but currenty referencing certificate in tf is broken anyways Thx guys! |
I'm using a provisioner script to switch on MSI for my app service, but when I do, it kills TF - you can't do anything with the resource, even if you switch MSI off again. Hoping the latest release will fix this as I'm stuck using 1.0.1 - the last release that worked with MSI, but actual support for MSI would be most welcome. |
Any update on when this will get incorporated? Just wondering if I should press forward with doing an ARM template for it or wait (if it is relatively soon). |
@simongh v1.3.3 includes a version of the Azure SDK for Go with the correct schema used for MSI - as such you should be able to upgrade to this now, I believe?
Support for MSI in App Services is being added in #1130 - I missed the notification that changes were pushed, but we'll take another look shortly. Thanks! |
Fixed via #1130 |
@tombuildsstuff great stuff! thanks! Hopefully the dreadful #894 would be gone next release too 😁 I Actually there is another networking matter i should open a ticket for, long ago i tested detaching and reattaching NSGs and it didn't play nice, i need to run a test and send the output, just not too much time on my hands |
Hello @pixelicous, We have just released v1.5.0 of the provider that includes this fix 🙂 |
@katbyte ive been updated! thanks! :) will test soon |
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. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks! |
Hi,
I see that MSI support is slowly being added to resources.
Is it possible to also add it to an app service?
Thank you.
The text was updated successfully, but these errors were encountered: