refactor(e2e): enhance build skipping logic for operator images

- Updated the environment variable for skipping operator image builds in the GitHub Actions workflow.
- Introduced a new function to determine if only the operator image build should be skipped, improving clarity and maintainability.
- Modified the operator deployment logic to utilize the new skipping function, allowing for more flexible build configurations.

Author: ZiyangLinScality

.github/workflows/tests.yml

@@ -74,5 +74,5 @@ jobs:
       - name: Run E2E tests
         run: make test-e2e
         env:
-          E2E_SKIP_BUILD: "true"
+          E2E_SKIP_OPERATOR_BUILD: "true"
           E2E_OPERATOR_IMAGE: "ghcr.io/${{ github.repository }}:${{ github.sha }}"

test/e2e/framework/config.go

@@ -44,11 +44,20 @@ func GetProjectRoot() string {
 	return "../.."
 }
 
+// SkipBuild returns true if all local builds should be skipped.
+// This affects both operator and mock server builds.
 func SkipBuild() bool {
 	val := strings.ToLower(os.Getenv("E2E_SKIP_BUILD"))
 	return val == "true" || val == "1" || val == "yes"
 }
 
+// SkipOperatorBuild returns true if only the operator image build should be skipped.
+// Use this when pulling a pre-built operator image from registry but still need to build mock server locally.
+func SkipOperatorBuild() bool {
+	val := strings.ToLower(os.Getenv("E2E_SKIP_OPERATOR_BUILD"))
+	return val == "true" || val == "1" || val == "yes"
+}
+
 func SkipOperatorDeploy() bool {
 	val := strings.ToLower(os.Getenv("E2E_SKIP_OPERATOR"))
 	return val == "true" || val == "1" || val == "yes"

test/e2e/framework/operator.go

@@ -195,8 +195,8 @@ func DeployOperatorSetup(clusterName string) func(ctx context.Context, cfg *envc
 
 		image := GetOperatorImage()
 
-		if SkipBuild() {
-			fmt.Printf("Skipping image build (E2E_SKIP_BUILD=true), using existing image: %s\n", image)
+		if SkipBuild() || SkipOperatorBuild() {
+			fmt.Printf("Skipping operator image build, using existing image: %s\n", image)
 		} else {
 			fmt.Printf("Building operator image: %s\n", image)
 			if err := BuildOperatorImage(projectRoot, image); err != nil {