Skip to content

Commit cdbd31a

Browse files
Jordan-Williams2Jordan-Williams2
authored andcommitted
fix: exclude br-sao
1 parent 3261703 commit cdbd31a

File tree

1 file changed

+15
-5
lines changed

1 file changed

+15
-5
lines changed

tests/pr_test.go

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -428,7 +428,7 @@ func setupOptionsSchematics(t *testing.T, prefix string, dir string) *testschema
428428
return options
429429
}
430430

431-
func setupOptionsVsiExstention(t *testing.T, prefix string, region string, existingTerraformOptions *terraform.Options) *testhelper.TestOptions {
431+
func setupOptionsVsiExtention(t *testing.T, prefix string, region string, existingTerraformOptions *terraform.Options) *testhelper.TestOptions {
432432

433433
sshPublicKey := sshPublicKey(t)
434434
outputVpcJson := terraform.OutputJson(t, existingTerraformOptions, "vpc_data")
@@ -598,7 +598,7 @@ func TestRunVPCPatternSchematics(t *testing.T) {
598598
assert.NoError(t, err, "Schematic Test had unexpected error")
599599
}
600600

601-
func TestRunVsiExstention(t *testing.T) {
601+
func TestRunVsiExtention(t *testing.T) {
602602
t.Parallel()
603603

604604
// ------------------------------------------------------------------------------------
@@ -620,6 +620,11 @@ func TestRunVsiExstention(t *testing.T) {
620620
// Programmatically determine region to use based on availability
621621
region, _ := testhelper.GetBestVpcRegion(val, "../common-dev-assets/common-go-assets/cloudinfo-region-vpc-gen2-prefs.yaml", "eu-de")
622622

623+
// Exclude br-sao due to direct endpoint connectivity issues with CI/CD infrastructure
624+
if region == "br-sao" {
625+
region = "us-south" // Fallback to us-south if br-sao is selected
626+
}
627+
623628
logger.Log(t, "Tempdir: ", tempTerraformDir)
624629
existingTerraformOptions := terraform.WithDefaultRetryableErrors(t, &terraform.Options{
625630
TerraformDir: vpcTerraformDir,
@@ -639,7 +644,7 @@ func TestRunVsiExstention(t *testing.T) {
639644
if existErr != nil {
640645
assert.True(t, existErr == nil, "Init and Apply of temp existing resource failed")
641646
} else {
642-
options := setupOptionsVsiExstention(t, prefix, region, existingTerraformOptions)
647+
options := setupOptionsVsiExtention(t, prefix, region, existingTerraformOptions)
643648
output, err := options.RunTestConsistency()
644649
assert.Nil(t, err, "This should not have errored")
645650
assert.NotNil(t, output, "Expected some output")
@@ -660,7 +665,7 @@ func TestRunVsiExstention(t *testing.T) {
660665
}
661666
}
662667

663-
func TestRunUpgradeVsiExstention(t *testing.T) {
668+
func TestRunUpgradeVsiExtention(t *testing.T) {
664669
// ------------------------------------------------------------------------------------
665670
// Deploy SLZ VPC first since it is needed for the landing-zone extension input
666671
// ------------------------------------------------------------------------------------
@@ -680,6 +685,11 @@ func TestRunUpgradeVsiExstention(t *testing.T) {
680685
// Programmatically determine region to use based on availability
681686
region, _ := testhelper.GetBestVpcRegion(val, "../common-dev-assets/common-go-assets/cloudinfo-region-vpc-gen2-prefs.yaml", "eu-de")
682687

688+
// Exclude br-sao due to direct endpoint connectivity issues with CI/CD infrastructure
689+
if region == "br-sao" {
690+
region = "us-south" // Fallback to us-south if br-sao is selected
691+
}
692+
683693
logger.Log(t, "Tempdir: ", tempTerraformDir)
684694
existingTerraformOptions := terraform.WithDefaultRetryableErrors(t, &terraform.Options{
685695
TerraformDir: vpcTerraformDir,
@@ -698,7 +708,7 @@ func TestRunUpgradeVsiExstention(t *testing.T) {
698708
if existErr != nil {
699709
assert.True(t, existErr == nil, "Init and Apply of temp existing resource failed")
700710
} else {
701-
options := setupOptionsVsiExstention(t, prefix, region, existingTerraformOptions)
711+
options := setupOptionsVsiExtention(t, prefix, region, existingTerraformOptions)
702712
output, err := options.RunTestUpgrade()
703713
if !options.UpgradeTestSkipped {
704714
assert.Nil(t, err, "This should not have errored")

0 commit comments

Comments
 (0)