-
Notifications
You must be signed in to change notification settings - Fork 67
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
Add support for Azure Container Apps #322
Comments
Hey @SandraAhlgrimm,
What do you think? |
Yes, please assign it to me. I’d love to get through the „nubesgen add a
new feature“ process first before thinking about the refactoring you
suggested.
Christopher MANEU ***@***.***> schrieb am Mi. 16. Nov. 2022
um 17:09:
… Hey @SandraAhlgrimm <https://github.com/SandraAhlgrimm>,
Great catch! Would you like to work on this task? If so, this is were I
would start:
- Test the mentioned sample
- Review the existing TF modules (I don't remember your level of
fluency about Terraform)
- Having the proper
- Remove Dapr-specific things. Dapr support is interesting, but not
all ACA users are DAPR users. Let's ship a simple version first
- Look at our existing Application Insights template. I think these
ones are outdated. It would make sense to create a new Azure Monitor / AI
TF module, and leverage it in this ACA module.
What do you think?
—
Reply to this email directly, view it on GitHub
<#322 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABHZDW5OZDGQCSN627SYI23WIUBMFANCNFSM5YCFKPRQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Yes @SandraAhlgrimm this is clearly the best approach! And ACA support would help a ton of people. |
HI @cmaneu, let's wait until ACA is in the official azurerm terraform project hashicorp/terraform-provider-azurerm#18008 It should be there soon, and it will ease adding this feature. Otherwise, we'll have to update it anyhow as soon as it gets released. |
Waiting for Terraform support of ACA
The text was updated successfully, but these errors were encountered: