fix: added validation for the managed_domain_mappings input (#138)

Author: Ak-sky

modules/app/README.md

@@ -58,7 +58,7 @@ No modules.
 | <a name="input_image_port"></a> [image\_port](#input\_image\_port) | The port which is used to connect to the port that is exposed by the container image. | `number` | `8080` | no |
 | <a name="input_image_reference"></a> [image\_reference](#input\_image\_reference) | The name of the image that is used for the app. | `string` | n/a | yes |
 | <a name="input_image_secret"></a> [image\_secret](#input\_image\_secret) | The name of the image registry access secret. | `string` | `null` | no |
-| <a name="input_managed_domain_mappings"></a> [managed\_domain\_mappings](#input\_managed\_domain\_mappings) | Define which of the system managed domain mappings will be setup for the application. Valid values are 'local\_public', 'local\_private' and 'local'. | `string` | `null` | no |
+| <a name="input_managed_domain_mappings"></a> [managed\_domain\_mappings](#input\_managed\_domain\_mappings) | Define which of the following values for the system-managed domain mappings to set up for the application: `local_public`, `local_private`, and `local`. See https://cloud.ibm.com/docs/codeengine?topic=codeengine-application-workloads#optionsvisibility | `string` | `null` | no |
 | <a name="input_name"></a> [name](#input\_name) | The name of the app. | `string` | n/a | yes |
 | <a name="input_project_id"></a> [project\_id](#input\_project\_id) | The ID of the project where app will be created. | `string` | n/a | yes |
 | <a name="input_run_arguments"></a> [run\_arguments](#input\_run\_arguments) | Arguments for the app that are passed to start the container. | `list(string)` | `[]` | no |

modules/app/variables.tf

@@ -54,9 +54,13 @@ variable "image_port" {
 }
 
 variable "managed_domain_mappings" {
-  description = "Define which of the system managed domain mappings will be setup for the application. Valid values are 'local_public', 'local_private' and 'local'."
+  description = "Define which of the following values for the system-managed domain mappings to set up for the application: `local_public`, `local_private`, and `local`. See https://cloud.ibm.com/docs/codeengine?topic=codeengine-application-workloads#optionsvisibility"
   type        = string
   default     = null
+  validation {
+    condition     = var.managed_domain_mappings == null || can(regex("local_public|local_private|local", var.managed_domain_mappings))
+    error_message = "Valid values are 'local_public', 'local_private', or 'local'."
+  }
 }
 
 variable "run_arguments" {

solutions/apps/variables.tf

@@ -95,9 +95,13 @@ variable "image_port" {
 }
 
 variable "managed_domain_mappings" {
-  description = "Specify which of the following values for the system-managed domain mappings to set up for the application: `local_public`, `local_private`, and `local`."
+  description = "Define which of the following values for the system-managed domain mappings to set up for the application: `local_public`, `local_private`, and `local`. See https://cloud.ibm.com/docs/codeengine?topic=codeengine-application-workloads#optionsvisibility"
   type        = string
   default     = null
+  validation {
+    condition     = var.managed_domain_mappings == null || can(regex("local_public|local_private|local", var.managed_domain_mappings))
+    error_message = "Valid values are 'local_public', 'local_private', or 'local'."
+  }
 }
 
 variable "run_arguments" {