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

[Module Proposal]: avm-res-cache-redis-enterprise (tf) #1772

Open
2 tasks done
kewalaka opened this issue Dec 23, 2024 · 4 comments
Open
2 tasks done

[Module Proposal]: avm-res-cache-redis-enterprise (tf) #1772

kewalaka opened this issue Dec 23, 2024 · 4 comments
Labels
Language: Terraform 🌐 This is related to the Terraform IaC language Needs: Module Owner 📣 This module needs an owner to develop or maintain it Needs: Triage 🔍 Maintainers need to triage still Type: New Module Proposal 💡 A new module for AVM is being proposed

Comments

@kewalaka
Copy link
Contributor

kewalaka commented Dec 23, 2024

Check for previous/existing GitHub issues/module proposals

  • I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

  • I have checked for that this module doesn't already exist in the module indexes.

Bicep or Terraform?

Terraform

Module Classification?

Resource Module

Module Name

avm-res-cache-redis-enterprise

Module Details

This would be the Terraform equivalent of the bicep proposal in #1731

There already is a redis_cache module, but this is in a different resource provider, as also noted by the bicep proposal, thus I think it is appropriate to have this as a separate AVM resource module.

Do you want to be the owner of this module?

No

Module Owner's GitHub Username (handle)

No response

(Optional) Secondary Module Owner's GitHub Username (handle)

kewalaka

@kewalaka kewalaka added Needs: Triage 🔍 Maintainers need to triage still Type: New Module Proposal 💡 A new module for AVM is being proposed labels Dec 23, 2024

Important

The "Needs: Triage 🔍" label must be removed once the triage process is complete!

Tip

For additional guidance on how to triage this issue/PR, see the AVM Issue Triage documentation.

@microsoft-github-policy-service microsoft-github-policy-service bot added Language: Terraform 🌐 This is related to the Terraform IaC language Needs: Module Owner 📣 This module needs an owner to develop or maintain it labels Dec 23, 2024

@kewalaka, thanks for submitting this module proposal!

Important

The AVM core team will review it and will try to find a module owner.

@kewalaka
Copy link
Contributor Author

I have an AVM aligned version that passes tfplan & lints here;

https://github.com/kewalaka/terraform-azurerm-avm-res-cache-redis-enterprise

E2E testing to be done.

@kewalaka
Copy link
Contributor Author

@jchancellor-ms I see you are the owner of the redis_cache AVM - would you be interested in being owner of this too? I'm happy to be co-owner and have a start made as noted above.

This is assuming there is general agreement with the thinking from the Bicep module that the enterprise version should be a separate resource.

@kewalaka kewalaka changed the title [Module Proposal]: avm-res-cache-redis-enterprise [Module Proposal]: avm-res-cache-redis-enterprise (tf) Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language: Terraform 🌐 This is related to the Terraform IaC language Needs: Module Owner 📣 This module needs an owner to develop or maintain it Needs: Triage 🔍 Maintainers need to triage still Type: New Module Proposal 💡 A new module for AVM is being proposed
Projects
Status: Proposed
Development

No branches or pull requests

1 participant