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

* Allow to change UI image in high level module

* Update readme to include ui docker image

Author: ruial

README.md

@@ -113,6 +113,7 @@ You can find a more complete example that uses this module but also includes set
 | <a name="input_subnet2_id"></a> [subnet2\_id](#input\_subnet2\_id) | Second subnet used for availability zone redundancy | `string` | n/a | yes |
 | <a name="input_tags"></a> [tags](#input\_tags) | aws tags | `map(string)` | n/a | yes |
 | <a name="input_ui_certificate_arn"></a> [ui\_certificate\_arn](#input\_ui\_certificate\_arn) | SSL certificate for UI. If set to empty string, UI is disabled. | `string` | `""` | 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

@@ -54,6 +54,7 @@ module "metaflow-ui" {
   subnet1_id                      = var.subnet1_id
   subnet2_id                      = var.subnet2_id
   ui_backend_container_image      = local.metadata_service_container_image
+  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.2"
+  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

@@ -30,7 +30,7 @@ The services are deployed behind an AWS ALB, and the module will output the ALB
 | <a name="input_subnet2_id"></a> [subnet2\_id](#input\_subnet2\_id) | Second private subnet used for availability zone redundancy | `string` | n/a | yes |
 | <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_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.0.1"` | 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

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

variables.tf

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