Allow to change UI image in high level module (outerbounds#17)

* Allow to change UI image in high level module

* Update readme to include ui docker image

Author: ruial

README.md

@@ -71,6 +71,7 @@ pre-commit run --all-files
 | <a name="input_ui_certificate_arn"></a> [ui\_certificate\_arn](#input\_ui\_certificate\_arn) | SSL certificate for UI. If no certificate arn is provided, HTTP will be used. | `string` | `null` | no |
 | <a name="input_ui_cidr_allow_list"></a> [ui\_cidr\_allow\_list](#input\_ui\_cidr\_allow\_list) | A list of CIDRs the UI will be available to | `list(string)` | <pre>[<br>  "0.0.0.0/0"<br>]</pre> | no |
 | <a name="input_ui_external_alb"></a> [ui\_external\_alb](#input\_ui\_external\_alb) | Whether the UI ALB is external or internal. | `bool` | `true` | no |
+| <a name="input_ui_static_container_image"></a> [ui\_static\_container\_image](#input\_ui\_static\_container\_image) | Container image for the UI frontend app | `string` | `""` | no |
 | <a name="input_vpc_cidr_block"></a> [vpc\_cidr\_block](#input\_vpc\_cidr\_block) | The VPC CIDR block that we'll access list on our Metadata Service API to allow all internal communications | `string` | n/a | yes |
 | <a name="input_vpc_id"></a> [vpc\_id](#input\_vpc\_id) | The id of the single VPC we stood up for all Metaflow resources to exist in. | `string` | n/a | yes |
 

locals.tf

@@ -16,4 +16,9 @@ locals {
     module.metaflow-common.default_metadata_service_container_image :
     var.metadata_service_container_image
   )
+  ui_static_container_image = (
+    var.ui_static_container_image == "" ?
+    module.metaflow-common.default_ui_static_container_image :
+    var.ui_static_container_image
+  )
 }

main.tf

@@ -59,6 +59,7 @@ module "metaflow-ui" {
   ui_backend_container_image      = local.metadata_service_container_image
   ui_allow_list                   = var.ui_cidr_allow_list
   ui_external_alb                 = var.ui_external_alb
+  ui_static_container_image       = local.ui_static_container_image
 
   METAFLOW_DATASTORE_SYSROOT_S3      = module.metaflow-datastore.METAFLOW_DATASTORE_SYSROOT_S3
   certificate_arn                    = var.ui_certificate_arn

modules/common/locals.tf

@@ -1,3 +1,4 @@
 locals {
   default_metadata_service_container_image = "netflixoss/metaflow_metadata_service:v2.2.4"
+  default_ui_static_container_image        = "public.ecr.aws/outerbounds/metaflow_ui:v1.0.1"
 }

modules/common/outputs.tf

@@ -1,3 +1,7 @@
 output "default_metadata_service_container_image" {
   value = local.default_metadata_service_container_image
 }
+
+output "default_ui_static_container_image" {
+  value = local.default_ui_static_container_image
+}

modules/ui/README.md

@@ -32,7 +32,7 @@ The services are deployed behind an AWS ALB, and the module will output the ALB
 | <a name="input_ui_allow_list"></a> [ui\_allow\_list](#input\_ui\_allow\_list) | A list of CIDRs the UI will be available to | `list(string)` | <pre>[<br>  "0.0.0.0/0"<br>]</pre> | no |
 | <a name="input_ui_backend_container_image"></a> [ui\_backend\_container\_image](#input\_ui\_backend\_container\_image) | Container image for UI backend | `string` | `""` | no |
 | <a name="input_ui_external_alb"></a> [ui\_external\_alb](#input\_ui\_external\_alb) | Whether the UI ALB is external or internal. | `bool` | `true` | no |
-| <a name="input_ui_static_container_image"></a> [ui\_static\_container\_image](#input\_ui\_static\_container\_image) | Container image for the UI frontend app | `string` | `"public.ecr.aws/outerbounds/metaflow_ui:v1.1.2"` | no |
+| <a name="input_ui_static_container_image"></a> [ui\_static\_container\_image](#input\_ui\_static\_container\_image) | Container image for the UI frontend app | `string` | `""` | no |
 
 ## Outputs
 

modules/ui/variables.tf

@@ -112,7 +112,7 @@ variable "ui_backend_container_image" {
 
 variable "ui_static_container_image" {
   type        = string
-  default     = "public.ecr.aws/outerbounds/metaflow_ui:v1.1.2"
+  default     = ""
   description = "Container image for the UI frontend app"
 }
 

variables.tf

@@ -114,6 +114,12 @@ variable "postgres_engine_version" {
   default     = "11"
 }
 
+variable "ui_static_container_image" {
+  type        = string
+  default     = ""
+  description = "Container image for the UI frontend app"
+}
+
 variable "tags" {
   description = "aws tags"
   type        = map(string)