Skip to content

Commit

Permalink
#169 Don't validate helm_charts_version
Browse files Browse the repository at this point in the history
  • Loading branch information
pbobov committed Jan 31, 2025
1 parent 2fe9cf9 commit 7119e42
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 16 deletions.
7 changes: 1 addition & 6 deletions aws/arcgis-enterprise-k8s/organization/variables.tf
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# Copyright 2024 Esri
# Copyright 2024-2025 Esri
#
# Licensed under the Apache License Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -43,11 +43,6 @@ variable "helm_charts_version" {
description = "Helm Charts for ArcGIS Enterprise on Kubernetes version"
type = string
default = "1.4.0"

validation {
condition = contains(["1.2.0", "1.2.1", "1.2.2", "1.2.3", "1.3.0", "1.3.1", "1.3.2", "1.3.3", "1.3.4", "1.4.0"], var.helm_charts_version)
error_message = "Valid values for helm_charts_version variable are 1.2.0, 1.2.1, 1.2.2, 1.2.3, 1.3.0, 1.3.1, 1.3.2, 1.3.3, 1.3.4 and 1.4.0."
}
}

variable "upgrade_token" {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
* * Creates cloud config JSON file for the object store.
*/

# Copyright 2024 Esri
# Copyright 2024-2025 Esri
#
# Licensed under the Apache License Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -65,7 +65,7 @@ resource "azurerm_storage_account" "deployment_storage" {
# Create blob container for the organization object store if cloud_config_json_file_path is not specified.
resource "azurerm_storage_container" "object_store" {
name = "object-store"
storage_account_name = azurerm_storage_account.deployment_storage.name
storage_account_id = azurerm_storage_account.deployment_storage.id
container_access_type = "private"
}

Expand Down Expand Up @@ -128,9 +128,7 @@ resource "local_sensitive_file" "cloud_config_json_file" {
usage = "DEFAULT"
connection = {
containerName = azurerm_storage_container.object_store.name
# regionEndpointUrl = "https://${module.site_core_info.storage_account_name}.blob.core.windows.net"
accountEndpointUrl = trimsuffix(azurerm_storage_account.deployment_storage.primary_blob_endpoint, "/")
# rootDir = var.deployment_id
}
category = "storage"
}]
Expand Down
7 changes: 1 addition & 6 deletions azure/arcgis-enterprise-k8s/organization/variables.tf
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# Copyright 2024 Esri
# Copyright 2024-2025 Esri
#
# Licensed under the Apache License Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -43,11 +43,6 @@ variable "helm_charts_version" {
description = "Helm Charts for ArcGIS Enterprise on Kubernetes version"
type = string
default = "1.4.0"

validation {
condition = contains(["1.2.0", "1.2.1", "1.2.2", "1.2.3", "1.3.0", "1.3.1", "1.3.2", "1.3.3", "1.3.4", "1.4.0"], var.helm_charts_version)
error_message = "Valid values for helm_charts_version variable are 1.2.0, 1.2.1, 1.2.2, 1.2.3, 1.3.0, 1.3.1, 1.3.2, 1.3.3, 1.3.4 and 1.4.0."
}
}

variable "upgrade_token" {
Expand Down

0 comments on commit 7119e42

Please sign in to comment.