[arvados] updated: 2.6.0-148-g92e7c012e

git repository hosting git at public.arvados.org
Wed May 10 20:08:50 UTC 2023


Summary of changes:
 tools/salt-install/terraform/aws/vpc/locals.tf     |  6 +----
 .../terraform/aws/vpc/terraform.tfvars             | 26 +++++++++++++++-------
 tools/salt-install/terraform/aws/vpc/variables.tf  | 17 ++++++++++++++
 3 files changed, 36 insertions(+), 13 deletions(-)

       via  92e7c012e250f689e13ef249fc76134fcc5df393 (commit)
      from  34fc414b10c18a1f836daad5520e335647d692eb (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 92e7c012e250f689e13ef249fc76134fcc5df393
Author: Lucas Di Pentima <lucas.dipentima at curii.com>
Date:   Wed May 10 17:05:55 2023 -0300

    20482: Extracts DNS aliases map as configurable variables.
    
    Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas.dipentima at curii.com>

diff --git a/tools/salt-install/terraform/aws/vpc/locals.tf b/tools/salt-install/terraform/aws/vpc/locals.tf
index 83929c14f..7f433950f 100644
--- a/tools/salt-install/terraform/aws/vpc/locals.tf
+++ b/tools/salt-install/terraform/aws/vpc/locals.tf
@@ -30,12 +30,8 @@ locals {
     for k, v in aws_eip.arvados_eip: k => v.public_ip
   }
   private_ip = var.private_ip
-  aliases = {
-    controller: ["ws"]
-    workbench: ["workbench2", "webshell", "keep", "download", "prometheus", "grafana", "*.collections"]
-  }
   cname_by_host = flatten([
-    for host, aliases in local.aliases : [
+    for host, aliases in var.dns_aliases : [
       for alias in aliases : {
         record = alias
         cname = host
diff --git a/tools/salt-install/terraform/aws/vpc/terraform.tfvars b/tools/salt-install/terraform/aws/vpc/terraform.tfvars
index 61e1a37bb..d65b7272f 100644
--- a/tools/salt-install/terraform/aws/vpc/terraform.tfvars
+++ b/tools/salt-install/terraform/aws/vpc/terraform.tfvars
@@ -2,7 +2,8 @@
 #
 # SPDX-License-Identifier: CC-BY-SA-3.0
 
-region_name = "us-east-1"
+# Main cluster configurations. No sensible defaults provided for these:
+# region_name = "us-east-1"
 # cluster_name = "xarv1"
 # domain_name = "xarv1.example.com"
 
@@ -12,10 +13,10 @@ region_name = "us-east-1"
 # Optional networking options. Set existing resources to be used instead of
 # creating new ones.
 # NOTE: We only support fully managed or fully custom networking, not a mix of both.
-# vpc_id = "vpc-"
-# sg_id = "sg-"
-# public_subnet_id = "subnet-"
-# private_subnet_id = "subnet-"
+# vpc_id = "vpc-aaaa"
+# sg_id = "sg-bbbb"
+# public_subnet_id = "subnet-cccc"
+# private_subnet_id = "subnet-dddd"
 
 # Optional custom tags to add to every resource. Default: {}
 # custom_tags = {
@@ -27,8 +28,17 @@ region_name = "us-east-1"
 # Optional cluster service nodes configuration:
 #
 # List of node names which either will be hosting user-facing or internal services
-# user_facing_hosts = [...]
-# internal_service_hosts = [...]
+# user_facing_hosts = ["node1", "node2", ...]
+# internal_service_hosts = ["node3", ...]
 #
 # Map assigning each node name an internal IP address
-# private_ip = {...}
\ No newline at end of file
+# private_ip = {
+#   node1 = "1.2.3.4",
+#   ...
+# }
+#
+# Map assigning DNS aliases for service node names
+# dns_aliases {
+#   node1 = ["alias1", "alias2", ...],
+#   ...
+# }
\ No newline at end of file
diff --git a/tools/salt-install/terraform/aws/vpc/variables.tf b/tools/salt-install/terraform/aws/vpc/variables.tf
index 020c697ab..faba88562 100644
--- a/tools/salt-install/terraform/aws/vpc/variables.tf
+++ b/tools/salt-install/terraform/aws/vpc/variables.tf
@@ -50,6 +50,23 @@ variable "private_ip" {
   }
 }
 
+variable "dns_aliases" {
+  description = "Sets DNS name aliases for every service node"
+  type = map(list(string))
+  default = {
+    controller: ["ws"]
+    workbench: [
+      "workbench2",
+      "webshell",
+      "keep",
+      "download",
+      "prometheus",
+      "grafana",
+      "*.collections"
+    ]
+  }
+}
+
 variable "vpc_id" {
   description = "Use existing VPC instead of creating one for the cluster"
   type = string

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list