-
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
resource/app_service: Add support for running Containers #1578
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Hey folks, thanks for the great terraform provider! I've just given this new feature a test and I can't seem to get it to work properly. As far as I understand, Web App for Containers only works on Linux but the App Service I end up with seems to be running Windows (I'm not entirely sure about this but I do get a I've created 2 App Services, 1 using the Terraform provider and another just through the Azure console and when I run Here are both of them for reference:
Azure Console provisioned instance
Is this something missing from the provider, or have I missed a configuration for |
hi @jgmchan Linux App Services need to be running within a Reserved App Service Plan as shown here - can you take a look and see if that solves this issue for you? Thanks! |
Thanks for the quick response :) I'll give it a quick test but I'm not sure this is reason because the service plan for the Azure console provisioned instance is not a reserved instance either: Result from
|
I just tested with the reserved instance set to true and it works! Thanks very much for that. It is a little confusing that it doesn't show up when running Thanks very much for your help! |
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 👉 [email protected]. Thanks! |
Fixes: #580
Used the examples provided in #580 to get this working
Test results in the comments below thanks to @tombuildsstuff