Skip to content

Use of Azure Verified Modules #93

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

Open
rick-pb opened this issue Aug 22, 2024 · 3 comments
Open

Use of Azure Verified Modules #93

rick-pb opened this issue Aug 22, 2024 · 3 comments

Comments

@rick-pb
Copy link

rick-pb commented Aug 22, 2024

Please provide us with the following information:

This issue is for a: (mark with an x)

- [ ] bug report -> please search issues before submitting
- [x] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

n/a

Any log messages given by the failure

n/a

Expected/desired behavior

use of Azure Verified Modules in the Terraform IaC

OS and Version?

n/a

Versions

n/a

Mention any other details that might be useful

I would have expected the Terraform code here to utilize the Azure Verified Modules to help wth alignment to CAF, consistency with other microsoft initiatives etc. Was there any reason these were not adopted, and if not, can we expect these to be referenced in future?


Thanks! We'll be in touch soon.

@dtzar
Copy link
Collaborator

dtzar commented Aug 23, 2024

This repo is using Azure/terraform-azurerm-aks see code reference that is referenced in the "terraform verified modules" directory. It seems there is only this one avm-ptn-aks-production 0.1.0 module from April release. FWIW, @nellyk and @zioproto are the ones who created that module, and they also created this.

This repo wants to align where the long-term official support resides for production Terraform AKS module.

@rick-pb
Copy link
Author

rick-pb commented Aug 25, 2024

you could be using Azure Verified modules (AVM) for resources like your user assigned identity could use the following module.

@dtzar
Copy link
Collaborator

dtzar commented Aug 26, 2024

Makes sense - PRs welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants