Skip to content

Commit d642504

Browse files
committed
Fixing some breaking changes in Azure terraform configs
Signed-off-by: Mark Rossetti <[email protected]>
1 parent 2754d6f commit d642504

File tree

3 files changed

+12
-9
lines changed

3 files changed

+12
-9
lines changed

infra/azure/terraform/capz/container-registry/main.tf

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,7 @@ resource "azurerm_container_registry" "capzci_registry" {
2929
sku = "Premium"
3030
anonymous_pull_enabled = true
3131

32-
retention_policy {
33-
days = 7
34-
enabled = true
35-
}
32+
retention_policy_in_days = 7
3633

3734
tags = {
3835
RetentionPolicy = "7days"
@@ -87,10 +84,7 @@ resource "azurerm_container_registry" "e2eprivate_registry" {
8784
resource_group_name = var.resource_group_name
8885
sku = "Premium"
8986

90-
retention_policy {
91-
days = 7
92-
enabled = true
93-
}
87+
retention_policy_in_days = 7
9488

9589
tags = {
9690
RetentionPolicy = "7days"

infra/azure/terraform/capz/main.tf

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ limitations under the License.
1616

1717
provider "azurerm" {
1818
features {}
19+
subscription_id = var.subscription_id
1920
}
2021

2122
terraform {
@@ -131,7 +132,9 @@ module "cluster_api_gallery" {
131132
source = "./cluster-api-gallery"
132133
resource_group_name = var.resource_group_name
133134
location = var.location
134-
depends_on = module.role_assignments
135+
depends_on = [
136+
module.role_assignments
137+
]
135138
}
136139

137140
# Import CAPZ monitoring module

infra/azure/terraform/capz/variables.tf

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,3 +31,9 @@ variable "storage_account_name" {
3131
default = "k8sprowstoragecomm"
3232
description = "Name of the storage account."
3333
}
34+
35+
variable "subscription_id" {
36+
type = string
37+
default = "46678f10-4bbb-447e-98e8-d2829589f2d8"
38+
description = "Azure Subscription ID to use for the azurerm provider."
39+
}

0 commit comments

Comments
 (0)