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

Feature Request: Add sample code for renaming management groups #157

Open
terrymandin opened this issue Feb 29, 2024 · 0 comments
Open

Feature Request: Add sample code for renaming management groups #157

terrymandin opened this issue Feb 29, 2024 · 0 comments
Labels
Area: Terraform Accelerator 👽 Issues / PR's related to Terraform Accelerator IaC: Terraform Infrastructure as Code - Terraform Module: Starter Type: Documentation 📄 Improvements or additions to documentation

Comments

@terrymandin
Copy link

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

Is your feature request related to a problem?

It is a common ask for customers to want to rename management groups. Customers do not typically find the names "Corp" and "Online" as intuitive and prefer to name them something like "Private" and "Public". They also sometimes like to follow our recommended CAF naming standards for the management groups.

Each CSA delivering the ALZ Deployment must search and find the best way to do this. Bilal Amjad has created a repo showing how to do this in TF, but it does not show how to do it in the Accelerator YAML files.

Describe the solution you'd like

Add commented out code in the config-hub-spoke.yaml and config-vwan.yaml files showing how to do this.

Attached is a sample config-hub-spoke.yaml file I created for a customer.

sample.yaml.txt

I'm happy to help with this if you would like me to create the PR.

@jaredfholgate jaredfholgate transferred this issue from Azure/alz-terraform-accelerator Sep 2, 2024
@jaredfholgate jaredfholgate added Needs: Triage 🔍 Needs triaging by the team Type: Documentation 📄 Improvements or additions to documentation Area: Terraform Accelerator 👽 Issues / PR's related to Terraform Accelerator IaC: Terraform Infrastructure as Code - Terraform Module: Starter and removed Needs: Triage 🔍 Needs triaging by the team labels Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Terraform Accelerator 👽 Issues / PR's related to Terraform Accelerator IaC: Terraform Infrastructure as Code - Terraform Module: Starter Type: Documentation 📄 Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants