Skip to content

Commit

Permalink
update to use non-depracated modules and to use python3
Browse files Browse the repository at this point in the history
  • Loading branch information
kentikethan committed Feb 15, 2024
1 parent 1c92842 commit e113395
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 5 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ override.tf
override.tf.json
*_override.tf
*_override.tf.json
*_override.tfvars

# Include override files you do wish to add to version control using negated pattern
#
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
# Azure
subscription_id = ""
location = ""
resource_group_names = []
storage_account_names = []
resource_group_names = [""]
storage_account_names = [""]

# Kentik
email= ""
Expand Down
2 changes: 1 addition & 1 deletion cloud_Azure/terraform/module/network_watcher.tf
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ data "azurerm_network_watcher" "network_watcher" {
# "ResourceGroupName2" -> "NetworkSercurityGroupId3,NetworkSecurityGroupId4"
# }
data "external" "nsg_data_source" {
program = ["python", "${path.module}/get_nsg.py"]
program = ["python3", "${path.module}/get_nsg.py"]
query = {
resource_group_names = join(",", var.resource_group_names)
}
Expand Down
4 changes: 2 additions & 2 deletions cloud_Azure/terraform/module/service_principal.tf
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
data "azuread_client_config" "current" {}

data "azuread_service_principals" "existing_nsg_flow_exporter" {
application_ids = [var.flow_exporter_application_id]
client_ids = [var.flow_exporter_application_id]
ignore_missing = true
}

Expand All @@ -14,7 +14,7 @@ locals {
resource "azuread_service_principal" "new_nsg_flow_exporter" {
count = local.nsg_flow_exporter_already_exists ? 0 : 1

application_id = var.flow_exporter_application_id
client_id = var.flow_exporter_application_id
app_role_assignment_required = false
owners = [data.azuread_client_config.current.object_id]

Expand Down

0 comments on commit e113395

Please sign in to comment.