address feedback

Author: whoffler

.catalog-onboard-pipeline.yaml

@@ -12,3 +12,11 @@ offerings:
         scc:
           instance_id: 1c7d5f78-9262-44c3-b779-b28fe4d88c37
           region: us-south
+          scope_resource_group_var_name: existing_resource_group_name
+      - name: security-enforced
+        mark_ready: true
+        install_type: fullstack
+        scc:
+          instance_id: 1c7d5f78-9262-44c3-b779-b28fe4d88c37
+          region: us-south
+          scope_resource_group_var_name: existing_resource_group_name

cra-config.yaml

@@ -1,10 +1,10 @@
 # More info about this file at https://github.com/terraform-ibm-modules/common-pipeline-assets/blob/main/.github/workflows/terraform-test-pipeline.md#cra-config-yaml
 version: "v1"
 CRA_TARGETS:
-  - CRA_TARGET: "solutions/fully-configurable"
-    CRA_IGNORE_RULES_FILE: "cra-tf-validate-ignore-rules.json"
-    PROFILE_ID: "fe96bd4d-9b37-40f2-b39f-a62760e326a3"  # SCC profile ID (currently set to 'IBM Cloud Framework for Financial Services' '1.7.0' profile).
-    CRA_ENVIRONMENT_VARIABLES:
+  - CRA_TARGET: "solutions/fully-configurable" # Target directory for CRA scan. If not provided, the CRA Scan will not be run.
+    CRA_IGNORE_RULES_FILE: "cra-tf-validate-ignore-rules.json" # CRA Ignore file to use. If not provided, it checks the repo root directory for `cra-tf-validate-ignore-rules.json`
+    PROFILE_ID: "fe96bd4d-9b37-40f2-b39f-a62760e326a3"         # SCC profile ID (currently set to 'IBM Cloud Framework for Financial Services' '1.7.0' profile).
+    CRA_ENVIRONMENT_VARIABLES:  # An optional map of environment variables for CRA, where the key is the variable name and value is the value. Useful for providing TF_VARs.
       TF_VAR_existing_kms_instance_crn: "crn:v1:bluemix:public:hs-crypto:us-south:a/abac0df06b644a9cabc6e44f55b3880e:e6dce284-e80f-46e1-a3c1-830f7adff7a9::"
       TF_VAR_existing_resource_group_name: "geretain-test-redis"
       TF_VAR_kms_encryption_enabled: true

ibm_catalog.json

@@ -514,9 +514,6 @@
                 }
               ]
             },
-            {
-              "key": "plan"
-            },
             {
               "key": "mongodb_version",
               "required": true,
@@ -531,6 +528,9 @@
                 }
               ]
             },
+            {
+              "key": "plan"
+            },
             {
               "key": "name"
             },

solutions/fully-configurable/catalogValidationValues.json.template

@@ -3,7 +3,7 @@
   "region": "us-south",
   "resource_tags": $TAGS,
   "name": $PREFIX,
-  "existing_resource_group_name": $PREFIX,
+  "existing_resource_group_name": "geretain-test-mongo",
   "existing_kms_instance_crn": $HPCS_US_SOUTH_CRN,
   "kms_encryption_enabled": true
 }

solutions/fully-configurable/variables.tf

@@ -122,7 +122,7 @@ variable "member_cpu_count" {
 variable "member_disk_mb" {
   type        = number
   description = "The disk that is allocated per member. [Learn more](https://cloud.ibm.com/docs/databases-for-mongodb?topic=databases-for-mongodb-resources-scaling)."
-  default     = 10240
+  default     = 5120
 }
 
 variable "member_host_flavor" {

solutions/security-enforced/catalogValidationValues.json.template

@@ -3,6 +3,6 @@
   "region": "us-south",
   "resource_tags": $TAGS,
   "name": $PREFIX,
-  "existing_resource_group_name": $PREFIX,
+  "existing_resource_group_name": "geretain-test-mongo",
   "existing_kms_instance_crn": $HPCS_US_SOUTH_CRN
 }