@@ -114,7 +114,6 @@ func TestRunUpgradeInstances(t *testing.T) {
114114// A test to pass existing resources to the SCC instances DA
115115func TestRunExistingResourcesInstances (t * testing.T ) {
116116 t .Parallel ()
117-
118117 // ------------------------------------------------------------------------------------
119118 // Provision COS, Sysdig and EN first
120119 // ------------------------------------------------------------------------------------
@@ -151,28 +150,26 @@ func TestRunExistingResourcesInstances(t *testing.T) {
151150 } else {
152151
153152 // ------------------------------------------------------------------------------------
154- // Deploy SCC instances DA passing in existing COS instance, bucket, Sysdig and EN details
153+ // Deploy SCC instances DA passing in existing COS instance (not bucket), KMS key and Sysdig
155154 // ------------------------------------------------------------------------------------
156155
157156 options := testhelper .TestOptionsDefault (& testhelper.TestOptions {
158157 Testing : t ,
159158 TerraformDir : instanceFlavorDir ,
160159 // Do not hard fail the test if the implicit destroy steps fail to allow a full destroy of resource to occur
161160 ImplicitRequired : false ,
162- Region : region ,
163161 TerraformVars : map [string ]interface {}{
164162 "prefix" : prefix ,
165163 "cos_region" : region ,
166164 "scc_region" : region ,
167165 "resource_group_name" : terraform .Output (t , existingTerraformOptions , "resource_group_name" ),
168166 "use_existing_resource_group" : true ,
169167 "existing_monitoring_crn" : terraform .Output (t , existingTerraformOptions , "monitoring_crn" ),
170- "existing_scc_cos_bucket_name" : terraform .Output (t , existingTerraformOptions , "bucket_name" ),
168+ "existing_kms_instance_crn" : permanentResources ["hpcs_south_crn" ],
169+ "kms_endpoint_type" : "public" ,
170+ "provider_visibility" : "public" ,
171171 "existing_cos_instance_crn" : terraform .Output (t , existingTerraformOptions , "cos_crn" ),
172172 "management_endpoint_type_for_bucket" : "public" ,
173- "provider_visibility" : "public" ,
174- "existing_en_crn" : terraform .Output (t , existingTerraformOptions , "en_crn" ),
175- "en_source_name" : prefix , // This name must be unique per SCC instance that is integrated with the Event Notifications instance.
176173 },
177174 })
178175
@@ -181,26 +178,28 @@ func TestRunExistingResourcesInstances(t *testing.T) {
181178 assert .NotNil (t , output , "Expected some output" )
182179
183180 // ------------------------------------------------------------------------------------
184- // Deploy SCC instances DA passing in existing COS instance (not bucket), KMS key and Sysdig
181+ // Deploy SCC instances DA passing in existing COS instance, bucket, Sysdig and EN details
185182 // ------------------------------------------------------------------------------------
186183
187184 options2 := testhelper .TestOptionsDefault (& testhelper.TestOptions {
188185 Testing : t ,
189186 TerraformDir : instanceFlavorDir ,
190187 // Do not hard fail the test if the implicit destroy steps fail to allow a full destroy of resource to occur
191188 ImplicitRequired : false ,
189+ Region : region ,
192190 TerraformVars : map [string ]interface {}{
193191 "prefix" : prefix ,
194192 "cos_region" : region ,
195193 "scc_region" : region ,
196194 "resource_group_name" : terraform .Output (t , existingTerraformOptions , "resource_group_name" ),
197195 "use_existing_resource_group" : true ,
198196 "existing_monitoring_crn" : terraform .Output (t , existingTerraformOptions , "monitoring_crn" ),
199- "existing_kms_instance_crn" : permanentResources ["hpcs_south_crn" ],
200- "kms_endpoint_type" : "public" ,
201- "provider_visibility" : "public" ,
197+ "existing_scc_cos_bucket_name" : terraform .Output (t , existingTerraformOptions , "bucket_name" ),
202198 "existing_cos_instance_crn" : terraform .Output (t , existingTerraformOptions , "cos_crn" ),
203199 "management_endpoint_type_for_bucket" : "public" ,
200+ "provider_visibility" : "public" ,
201+ "existing_en_crn" : terraform .Output (t , existingTerraformOptions , "en_crn" ),
202+ "en_source_name" : prefix , // This name must be unique per SCC instance that is integrated with the Event Notifications instance.
204203 },
205204 })
206205
0 commit comments