Merge pull request #5388 from sergiordlr/mco-filter-tests-by-platform

MCO-1966: Filter extended tests by platform

Author: openshift-merge-bot[bot]

cmd/machine-config-tests-ext/main.go

@@ -7,6 +7,7 @@ import (
 	"time"
 
 	"github.com/openshift-eng/openshift-tests-extension/pkg/cmd"
+	et "github.com/openshift-eng/openshift-tests-extension/pkg/extension/extensiontests"
 	v "github.com/openshift-eng/openshift-tests-extension/pkg/version"
 	"github.com/openshift/machine-config-operator/pkg/version"
 	exutil "github.com/openshift/machine-config-operator/test/extended-priv/util"
@@ -20,6 +21,37 @@ import (
 	_ "github.com/openshift/machine-config-operator/test/extended-priv"
 )
 
+func applyLabelFilters(specs et.ExtensionTestSpecs) {
+	// Apply Platform label filters: tests with Platform:platformname only run on that platform
+	specs.Walk(func(spec *et.ExtensionTestSpec) {
+		for label := range spec.Labels {
+			if strings.HasPrefix(label, "Platform:") {
+				platformName := strings.TrimPrefix(label, "Platform:")
+				spec.Include(et.PlatformEquals(platformName))
+			}
+		}
+	})
+
+	// Apply NoPlatform label filters: tests with NoPlatform:platformname excluded from that platform
+	specs.Walk(func(spec *et.ExtensionTestSpec) {
+		for label := range spec.Labels {
+			if strings.HasPrefix(label, "NoPlatform:") {
+				platformName := strings.TrimPrefix(label, "NoPlatform:")
+				spec.Exclude(et.PlatformEquals(platformName))
+			}
+		}
+	})
+
+	// Exclude tests with "Exclude:REASON" labels
+	specs.Walk(func(spec *et.ExtensionTestSpec) {
+		for label := range spec.Labels {
+			if strings.HasPrefix(label, "Exclude:") {
+				spec.Exclude("true")
+			}
+		}
+	})
+}
+
 func main() {
 	// Extension registry
 	registry := e.NewRegistry()
@@ -48,6 +80,8 @@ func main() {
 		panic(fmt.Sprintf("couldn't build extension test specs from ginkgo: %+v", err.Error()))
 	}
 
+	applyLabelFilters(specs)
+
 	ext.AddSpecs(specs)
 	registry.Register(ext)
 

test/extended/boot_image_update_azure.go

@@ -18,7 +18,7 @@ import (
 )
 
 // These tests are [Serial] because it modifies the cluster/machineconfigurations.operator.openshift.io object in each test.
-var _ = g.Describe("[sig-mco][Suite:openshift/machine-config-operator/disruptive][Serial][OCPFeatureGate:ManagedBootImagesAzure]", g.Ordered, func() {
+var _ = g.Describe("[sig-mco][Suite:openshift/machine-config-operator/disruptive][Serial][OCPFeatureGate:ManagedBootImagesAzure]", g.Label("Platform:azure"), g.Ordered, func() {
 	defer g.GinkgoRecover()
 	var (
 		AllMachineSetFixture     = filepath.Join("machineconfigurations", "managedbootimages-all.yaml")