Bump consul_connect plugin version and add it as a variable

Skatteetaten · Sep 17, 2020 · b1858d9 · b1858d9
1 parent 20b1036
commit b1858d9
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/conf/nomad/presto.hcl b/conf/nomad/presto.hcl
@@ -205,7 +205,7 @@ job "${nomad_job_name}" {
       artifact {
         # Download custom certificate authenticator plugin
         # TODO: add variable for this and make it optional
-        source = "https://oss.sonatype.org/service/local/repositories/releases/content/io/github/gugalnikov/presto-consul-connect/2.1.0/presto-consul-connect-2.1.0-jar-with-dependencies.jar"
+        source = "https://oss.sonatype.org/service/local/repositories/releases/content/io/github/gugalnikov/presto-consul-connect/${consul_connect_plugin_version}/presto-consul-connect-${consul_connect_plugin_version}-jar-with-dependencies.jar"
         mode = "file"
         destination = "local/presto/plugin/presto-consul-connect.jar"
       }

diff --git a/main.tf b/main.tf
@@ -32,6 +32,9 @@ data "template_file" "template-nomad-job-presto" {
     memory                = var.memory
     consul_http_addr      = var.consul_http_addr
 
+    #Custom plugin for consul connect integration
+    consul_connect_plugin_version = var.consul_connect_plugin_version
+
     #hivemetastore
     hivemetastore_service_name = var.hivemetastore.service_name
     hivemetastore_port         = var.hivemetastore.port

diff --git a/variables.tf b/variables.tf
@@ -87,6 +87,12 @@ variable "docker_image" {
   default     = "prestosql/presto:341"
 }
 
+variable "consul_connect_plugin_version" {
+  type = string
+  description = "Version of the consul connect plugin for presto (on maven central) src here: https://github.com/gugalnikov/presto-consul-connect"
+  default = "2.2.0"
+}
+
 variable "container_environment_variables" {
   type        = list(string)
   description = "Presto environment variables"