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: ALZ Terraform Accelerator Scenario Feedback #243

Open
1 task done
FreddyAyala opened this issue Nov 5, 2024 · 1 comment
Open
1 task done

Feature Request: ALZ Terraform Accelerator Scenario Feedback #243

FreddyAyala opened this issue Nov 5, 2024 · 1 comment
Assignees
Labels
Type: Enhancement ✨ New feature or request

Comments

@FreddyAyala
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Infrastructure as Code Type? (Required)

terraform

Starter Module? (Required)

bicep - complete

Use Case (Required)

Hello Team,
I would like to provide some feedback to Layer 0 regarding the accelerator after deliveries I've made and conversations with other CSAs. The accelerator is truly amazing and serves as our main showcase to guide implementation. However, we've noticed some confusion surrounding the multi-region deployment.
The templates available for utilizing tfvars in this deployment can be found here: GitHub Link. These examples demonstrate how to deploy either a multiregional Hub and Spoke or Virtual WAN using tfvars, rather than the .yaml file showcased in the accelerator's wiki.
Additionally, the complete multi-region example does not include any policy deployment, necessitating the integration of policy components from here: Policy Components into the multi-region template: Multi-Region Template. This requires us to combine the old way of using .yaml with the newer tfvars method.
These complexities make it challenging to deliver clear guidance from the VBD side.

Proposed Solution (Required)

We recommend Layer-0 defining layered core scenarios or examples that seamlessly integrate Governance, Management, and Connectivity (such as hub and spoke, virtual WAN, and multi-region by default but optional) and update the wiki to reflect those changes.

Important Factoids (Optional)

No response

References (Optional)

No response

@FreddyAyala FreddyAyala added Needs: Triage 🔍 Needs triaging by the team Type: Enhancement ✨ New feature or request labels Nov 5, 2024
@jaredfholgate jaredfholgate removed the Needs: Triage 🔍 Needs triaging by the team label Nov 6, 2024
@jaredfholgate jaredfholgate self-assigned this Nov 6, 2024
@jaredfholgate
Copy link
Member

Thanks for the feedback. I have been talking to Jamie Pla about a similar subject and looking to make this experience better moving forward.

For clarity, the example tfvars files are currently used to generate the example YAML files. The tfvars are needed for independent testing of the starter module too. They are not currently intended for customer use and as such they are not referenced in our docs.

However, for the sake or usability we want to move towards the tfvars file being the input into the ALZ module and directly used by the starter module. That way there is no confusion and we don't need auto generate YAML examples or transform YAML to json for this part. It will also make future updates in a fire and forget scenarios much easier for the customer.

We will also have the opportunity to provide:

  • More examples
  • More comprehensive examples (showing all options)

I am unsure about the note on the the starter module not deploying policies? It does deploy the policies.

We are in a transition stage as we move towards a full AVM implementation. We will begin to consolidate and make the options simpler as we move forward. We will also improve the VBD docs as we go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement ✨ New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants