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

Initial terraform config for azure arm VM #14

Merged
merged 1 commit into from
Nov 10, 2023

Conversation

henrirosten
Copy link
Collaborator

@henrirosten henrirosten commented Nov 10, 2023

  • PoC azure arm VM configuration with terraform
  • Refactor azure-ghaf-infra.tf to sections per VM-specific resources. This is the first step to make it easier to add new VMs.
  • Move the terraform resource group location to 'northeurope' due to the needed arm VM resources not being available in 'swedencentral'
  • Add .direnv directory to .gitignore

- PoC azure arm VM configuration with terraform
- Refactor azure-ghaf-infra.tf in sections per VM-specific resources.
  This is the first step to make it easier to add new VMs.
- Move the terraform resource group location to northeurope due to arm
  resource not being available in swecentral
- Add .direnv to .gitignore

Signed-off-by: Henri Rosten <[email protected]>
@henrirosten henrirosten changed the title Initial terraform config for azarm Initial terraform config for azure arm VM Nov 10, 2023
@henrirosten henrirosten marked this pull request as ready for review November 10, 2023 08:14
@henrirosten henrirosten requested review from karim20230 and a team November 10, 2023 08:14
@karim20230 karim20230 self-requested a review November 10, 2023 09:20
@henrirosten henrirosten requested a review from a team November 10, 2023 10:55
@henrirosten henrirosten merged commit acae699 into main Nov 10, 2023
1 check passed
@henrirosten henrirosten deleted the azure-arm-builder-with-terraform branch November 14, 2023 13:25
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

Successfully merging this pull request may close these issues.

2 participants