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

Azure/bastion #129

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Azure/bastion #129

wants to merge 2 commits into from

Conversation

nellyk
Copy link
Member

@nellyk nellyk commented Oct 7, 2024

example with Bastion

closes #117

<!-- BEGIN_TF_DOCS -->
# Default example

This deploys the module with Azure Bastion.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

neat: double space "module with"


```hcl
terraform {
required_version = ">= 1.3.0"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

by AVM template this should be:

required_version = ">= 1.9, < 2.0"

required_providers {
azurerm = {
source = "hashicorp/azurerm"
version = ">= 3.7.0, < 4.0.0"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here:

version = ">= 4, <5"


## Section to provide a random Azure region for the resource group
# This allows us to randomize the region for the resource group.
module "regions" {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use

module "regions" {
  source  = "Azure/avm-utl-regions/azurerm"
  version = "0.3.0"
}


# This is required for resource modules
resource "azurerm_resource_group" "this" {
location = "East US 2" # Hardcoded because we have to test in a region without availability zones
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this true for this example ?

# with a data source.
module "test" {
source = "../../"
kubernetes_version = "1.28"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Too old AKS version


module "avm_res_network_virtualnetwork" {
source = "Azure/avm-res-network-virtualnetwork/azurerm"
version = "0.2.3"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use version 0.8.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs: Triage 🔍 Maintainers need to triage still
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[AVM Question/Feedback]: Private cluster example with bastion
2 participants