Skip to content
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

Deploying Azure App service plans into App service environments #712

Closed
hashibot opened this issue Jan 18, 2018 · 4 comments
Closed

Deploying Azure App service plans into App service environments #712

hashibot opened this issue Jan 18, 2018 · 4 comments

Comments

@hashibot
Copy link

This issue was originally opened by @JoMoore88 as hashicorp/terraform#17124. It was migrated here as a result of the provider split. The original body of the issue is below.


Hey everyone,
I currently have a need to deploy an ASE with some underlying ASP'S that will host a handful of web apps. Given that i do not see an associated ASE resource for terraform i'm using the azurerm_template_deployment to deploy the ASE, which is fine.

Ideally i would like to deploy the ASPS and Web apps using native terraform resources and just deploy the ASE using the ARM deployment resource. In ARM the deployment for an ASP which resides within an ASE has a property for the ASE resource id this looks like :

       "apiVersion": "2015-08-01",
        "name": "[parameters('appServicePlanName')]",
        "type": "Microsoft.Web/serverfarms",
        "location": "[parameters('existingAseLocation')]",
        "properties": {
            "name": "[parameters('appServicePlanName')]",
            **"hostingEnvironmentProfile": {
                 "id": "[resourceId('Microsoft.Web/hostingEnvironments',parameters('appServiceEnvironmentName'))]"
            }**
        },

Trouble is, i cannot find an argument in the terraform ASP resource that represents the ASE resource ID (hostingEnvironmentProfile in the above).

Assuming i have not overlooked anything and this is not currently a feature in terraform today - could it be added?

Also, if there is a known workaround for this situation, outside of putting the ASPS in ARM, could someone please let me know?

@tombuildsstuff
Copy link
Contributor

Hey @JoMoore88

Thanks for opening this issue

This isn't currently possible natively in Terraform (on the azurerm_app_service_plan resource) - but we should expose this field. I've marked this as an enhancement and we'll add support for it in the near future.

Thanks!

@tombuildsstuff tombuildsstuff added this to the 1.0.3 milestone Jan 23, 2018
@ghost
Copy link

ghost commented Jan 23, 2018

Sounds good, thank you!

@tombuildsstuff
Copy link
Contributor

Fixed via #850

@ghost
Copy link

ghost commented Mar 31, 2020

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!

@ghost ghost locked and limited conversation to collaborators Mar 31, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants