Remove baremetal-branch flag from the integration test CLI

aws · Nov 2, 2024 · c15e109 · c15e109
1 parent fd108f5
commit c15e109
Show file tree

Hide file tree

Showing 9 changed files with 21 additions and 39 deletions.
diff --git a/cmd/integration_test/build/buildspecs/cloudstack-test-eks-a-cli.yml b/cmd/integration_test/build/buildspecs/cloudstack-test-eks-a-cli.yml
@@ -108,7 +108,6 @@ phases:
         --cleanup-resources=true
         --test-report-folder=reports
         --branch-name=${BRANCH_NAME}
-        --baremetal-branch=${BAREMETAL_BRANCH}
   post_build:
     commands:
       - unset AWS_SDK_LOAD_CONFIG AWS_PROFILE

diff --git a/cmd/integration_test/build/buildspecs/docker-test-eks-a-cli.yml b/cmd/integration_test/build/buildspecs/docker-test-eks-a-cli.yml
@@ -57,7 +57,6 @@ phases:
         --bundles-override=${BUNDLES_OVERRIDE}
         --test-report-folder=reports
         --branch-name=${BRANCH_NAME}
-        --baremetal-branch=${BAREMETAL_BRANCH}
 reports:
   e2e-reports:
     files:

diff --git a/cmd/integration_test/build/buildspecs/nutanix-test-eks-a-cli.yml b/cmd/integration_test/build/buildspecs/nutanix-test-eks-a-cli.yml
@@ -91,7 +91,6 @@ phases:
         --cleanup-resources=true
         --test-report-folder=reports
         --branch-name=${BRANCH_NAME}
-        --baremetal-branch=${BAREMETAL_BRANCH}
   post_build:
     commands:
       - >

diff --git a/cmd/integration_test/build/buildspecs/quick-test-eks-a-cli.yml b/cmd/integration_test/build/buildspecs/quick-test-eks-a-cli.yml
@@ -174,7 +174,6 @@ env:
     T_TINKERBELL_CP_NETWORK_CIDR: "tinkerbell_ci:cp_network_cidr"
     T_TINKERBELL_S3_INVENTORY_CSV_KEY: "tinkerbell_ci:s3_inventory_csv"
     T_TINKERBELL_S3_AG_INVENTORY_CSV_KEY: "tinkerbell_ci:s3_ag_inventory_csv"
-    BAREMETAL_BRANCH: "tinkerbell_ci:baremetal_branch"
     TEST_RUNNER_GOVC_USERNAME: "tinkerbell_ci:govc_username"
     TEST_RUNNER_GOVC_PASSWORD: "tinkerbell_ci:govc_password"
     TEST_RUNNER_GOVC_URL: "tinkerbell_ci:govc_url"
@@ -237,7 +236,6 @@ phases:
         --cleanup-resources=true
         --test-report-folder=reports
         --branch-name=${BRANCH_NAME}
-        --baremetal-branch=${BAREMETAL_BRANCH}
   post_build:
     commands:
       - unset AWS_SDK_LOAD_CONFIG AWS_PROFILE

diff --git a/cmd/integration_test/build/buildspecs/snow-test-eks-a-cli.yml b/cmd/integration_test/build/buildspecs/snow-test-eks-a-cli.yml
@@ -58,7 +58,6 @@ phases:
         --cleanup-resources=true
         --test-report-folder=reports
         --branch-name=${BRANCH_NAME}
-        --baremetal-branch=${BAREMETAL_BRANCH}
 reports:
   e2e-reports:
     files:

diff --git a/cmd/integration_test/build/buildspecs/tinkerbell-test-eks-a-cli.yml b/cmd/integration_test/build/buildspecs/tinkerbell-test-eks-a-cli.yml
@@ -56,7 +56,6 @@ env:
     T_TINKERBELL_CP_NETWORK_CIDR: "tinkerbell_ci:cp_network_cidr"
     T_TINKERBELL_S3_INVENTORY_CSV_KEY: "tinkerbell_ci:s3_inventory_csv"
     T_TINKERBELL_S3_AG_INVENTORY_CSV_KEY: "tinkerbell_ci:s3_ag_inventory_csv"
-    BAREMETAL_BRANCH: "tinkerbell_ci:baremetal_branch"
     TEST_RUNNER_GOVC_USERNAME: "tinkerbell_ci:govc_username"
     TEST_RUNNER_GOVC_PASSWORD: "tinkerbell_ci:govc_password"
     TEST_RUNNER_GOVC_URL: "tinkerbell_ci:govc_url"
@@ -106,7 +105,6 @@ phases:
         --cleanup-resources=true
         --test-report-folder=reports
         --branch-name=${BRANCH_NAME}
-        --baremetal-branch=${BAREMETAL_BRANCH}
 reports:
   e2e-reports:
     files:

diff --git a/cmd/integration_test/build/buildspecs/vsphere-test-eks-a-cli.yml b/cmd/integration_test/build/buildspecs/vsphere-test-eks-a-cli.yml
@@ -126,7 +126,6 @@ phases:
         --cleanup-resources=true
         --test-report-folder=reports
         --branch-name=${BRANCH_NAME}
-        --baremetal-branch=${BAREMETAL_BRANCH}
   post_build:
     commands:
       - unset AWS_SDK_LOAD_CONFIG AWS_PROFILE

diff --git a/cmd/integration_test/cmd/run.go b/cmd/integration_test/cmd/run.go
@@ -25,7 +25,6 @@ const (
 	testReportFolderFlagName   = "test-report-folder"
 	branchNameFlagName         = "branch-name"
 	instanceConfigFlagName     = "instance-config"
-	baremetalBranchFlagName    = "baremetal-branch"
 )
 
 var runE2ECmd = &cobra.Command{
@@ -67,7 +66,6 @@ func init() {
 	runE2ECmd.Flags().Bool(cleanupResourcesFlagName, false, "Flag to indicate if test resources should be cleaned up automatically as tests complete")
 	runE2ECmd.Flags().String(testReportFolderFlagName, "", "Folder destination for JUnit tests reports")
 	runE2ECmd.Flags().String(branchNameFlagName, "main", "EKS-A origin branch from where the tests are being run")
-	runE2ECmd.Flags().String(baremetalBranchFlagName, "main", "Branch for baremetal tests to run on")
 
 	for _, flag := range requiredFlags {
 		if err := runE2ECmd.MarkFlagRequired(flag); err != nil {
@@ -88,7 +86,6 @@ func runE2E(ctx context.Context) error {
 	cleanupResources := viper.GetBool(cleanupResourcesFlagName)
 	testReportFolder := viper.GetString(testReportFolderFlagName)
 	branchName := viper.GetString(branchNameFlagName)
-	baremetalBranchName := viper.GetString(baremetalBranchFlagName)
 
 	runConf := e2e.ParallelRunConf{
 		MaxConcurrentTests:     maxConcurrentTests,
@@ -102,7 +99,6 @@ func runE2E(ctx context.Context) error {
 		TestReportFolder:       testReportFolder,
 		BranchName:             branchName,
 		TestInstanceConfigFile: instanceConfigFile,
-		BaremetalBranchName:    baremetalBranchName,
 		Logger:                 logger.Get(),
 	}
 

diff --git a/internal/test/e2e/run.go b/internal/test/e2e/run.go
@@ -50,7 +50,6 @@ type ParallelRunConf struct {
 	CleanupResources       bool
 	TestReportFolder       string
 	BranchName             string
-	BaremetalBranchName    string
 	Logger                 logr.Logger
 }
 
@@ -89,21 +88,19 @@ func RunTestsInParallel(conf ParallelRunConf) error {
 
 	// For Tinkerbell tests, get hardware inventory pool
 	var invCatalogue map[string]*hardwareCatalogue
-	if strings.EqualFold(conf.BranchName, conf.BaremetalBranchName) {
-		nonAirgappedHardwareInv, err := getNonAirgappedHardwarePool(conf.StorageBucket)
-		if err != nil {
-			return fmt.Errorf("failed to get non-airgapped hardware inventory for Tinkerbell Tests: %v", err)
-		}
-		nonAirgappedInvCatalogue := newHardwareCatalogue(nonAirgappedHardwareInv)
-		airgappedHardwareInv, err := getAirgappedHardwarePool(conf.StorageBucket)
-		if err != nil {
-			return fmt.Errorf("failed to get airgapped hardware inventory for Tinkerbell Tests: %v", err)
-		}
-		airgappedInvCatalogue := newHardwareCatalogue(airgappedHardwareInv)
-		invCatalogue = map[string]*hardwareCatalogue{
-			nonAirgappedHardware: nonAirgappedInvCatalogue,
-			airgappedHardware:    airgappedInvCatalogue,
-		}
+	nonAirgappedHardwareInv, err := getNonAirgappedHardwarePool(conf.StorageBucket)
+	if err != nil {
+		return fmt.Errorf("failed to get non-airgapped hardware inventory for Tinkerbell Tests: %v", err)
+	}
+	nonAirgappedInvCatalogue := newHardwareCatalogue(nonAirgappedHardwareInv)
+	airgappedHardwareInv, err := getAirgappedHardwarePool(conf.StorageBucket)
+	if err != nil {
+		return fmt.Errorf("failed to get airgapped hardware inventory for Tinkerbell Tests: %v", err)
+	}
+	airgappedInvCatalogue := newHardwareCatalogue(airgappedHardwareInv)
+	invCatalogue = map[string]*hardwareCatalogue{
+		nonAirgappedHardware: nonAirgappedInvCatalogue,
+		airgappedHardware:    airgappedInvCatalogue,
 	}
 
 	work := make(chan instanceRunConf)
@@ -411,17 +408,15 @@ func splitTests(testsList []string, conf ParallelRunConf) ([]instanceRunConf, er
 		runConfs = append(runConfs, newInstanceRunConf(awsSession, conf, len(runConfs), testName, ips, []*api.Hardware{}, 0, false, Ec2TestRunnerType, testRunnerConfig))
 	}
 
-	if strings.EqualFold(conf.BranchName, conf.BaremetalBranchName) {
-		tinkerbellIPManager := newE2EIPManager(conf.Logger, os.Getenv(tinkerbellControlPlaneNetworkCidrEnvVar))
-		runConfs, err = appendNonAirgappedTinkerbellRunConfs(awsSession, testsList, conf, testRunnerConfig, runConfs, tinkerbellIPManager)
-		if err != nil {
-			return nil, fmt.Errorf("failed to split Tinkerbell tests: %v", err)
-		}
+	tinkerbellIPManager := newE2EIPManager(conf.Logger, os.Getenv(tinkerbellControlPlaneNetworkCidrEnvVar))
+	runConfs, err = appendNonAirgappedTinkerbellRunConfs(awsSession, testsList, conf, testRunnerConfig, runConfs, tinkerbellIPManager)
+	if err != nil {
+		return nil, fmt.Errorf("failed to split Tinkerbell tests: %v", err)
+	}
 
-		runConfs, err = appendAirgappedTinkerbellRunConfs(awsSession, testsList, conf, testRunnerConfig, runConfs, tinkerbellIPManager)
-		if err != nil {
-			return nil, fmt.Errorf("failed to run airgapped Tinkerbell tests: %v", err)
-		}
+	runConfs, err = appendAirgappedTinkerbellRunConfs(awsSession, testsList, conf, testRunnerConfig, runConfs, tinkerbellIPManager)
+	if err != nil {
+		return nil, fmt.Errorf("failed to run airgapped Tinkerbell tests: %v", err)
 	}
 
 	return runConfs, nil