Skip to content

Commit 52e710a

Browse files
committed
Adjust criteria for use of never-stable, add techpreview variants
- I haven't found being more liberal with adding jobs to the never-stable list to be particularly helpful, especially with so many failing. I think each should be investigated, and a bug filed -- and then added to the never-stable list. - Adds a new techpreview variant, for TPNU jobs
1 parent 9e232a6 commit 52e710a

1 file changed

Lines changed: 28 additions & 38 deletions

File tree

pkg/testgridanalysis/testidentification/ocp_variants.go

Lines changed: 28 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ var (
3232
rtRegex = regexp.MustCompile(`(?i)-rt`)
3333
s390xRegex = regexp.MustCompile(`(?i)-s390x`)
3434
serialRegex = regexp.MustCompile(`(?i)-serial`)
35+
techpreview = regexp.MustCompile(`(?i)-techpreview`)
3536
upgradeRegex = regexp.MustCompile(`(?i)-upgrade`)
3637
// some vsphere jobs do not have a trailing -version segment
3738
vsphereRegex = regexp.MustCompile(`(?i)-vsphere`)
@@ -58,64 +59,46 @@ var (
5859
"realtime",
5960
"s390x",
6061
"serial",
62+
"techpreview",
6163
"upgrade",
6264
"vsphere-ipi",
6365
"vsphere-upi",
6466
"single-node",
6567
)
6668

67-
// openshiftJobsNeverStableForVariants is a list of jobs that are
68-
// below 40% passing rate. As we phase these jobs in, they should be
69-
// excluded from "normal" variants and once they are passing above 40%
70-
// can "graduated" from never-stable.
69+
// openshiftJobsNeverStableForVariants is a list of unproven new jobs or
70+
// jobs that are near permafail (i.e. < 40%) for an extended period of time.
71+
// They are excluded from "normal" variants and once they are passing above
72+
// 40% can "graduated" from never-stable.
73+
//
74+
// Jobs should have a linked BZ before being added to this list.
7175
//
7276
// These jobs are still listed as jobs in total and when individual
7377
// tests fail, they will still be listed with these jobs as causes.
7478
openshiftJobsNeverStableForVariants = sets.NewString(
79+
// Experimental jobs that are under active development
7580
"periodic-ci-openshift-release-master-ci-4.9-e2e-aws-hypershift",
76-
"periodic-ci-openshift-release-master-ci-4.9-e2e-aws-ovn",
77-
"periodic-ci-openshift-release-master-ci-4.9-e2e-aws-upgrade-single-node",
78-
"periodic-ci-openshift-release-master-ci-4.9-e2e-gcp-ovn",
79-
"periodic-ci-openshift-release-master-ci-4.9-e2e-openstack-ovn",
80-
"periodic-ci-openshift-release-master-ci-4.9-upgrade-from-stable-4.8-e2e-aws-ovn-upgrade",
81-
"periodic-ci-openshift-release-master-ci-4.9-upgrade-from-stable-4.8-e2e-aws-ovn-upgrade-rollback",
82-
"periodic-ci-openshift-release-master-ci-4.9-upgrade-from-stable-4.8-e2e-aws-upgrade",
83-
"periodic-ci-openshift-release-master-ci-4.9-upgrade-from-stable-4.8-e2e-aws-upgrade-rollback",
84-
"periodic-ci-openshift-release-master-ci-4.9-upgrade-from-stable-4.8-e2e-azure-ovn-upgrade",
85-
"periodic-ci-openshift-release-master-ci-4.9-upgrade-from-stable-4.8-e2e-gcp-ovn-upgrade",
86-
"periodic-ci-openshift-release-master-ci-4.9-upgrade-from-stable-4.8-e2e-openstack-upgrade",
87-
"periodic-ci-openshift-release-master-ci-4.9-upgrade-from-stable-4.8-from-stable-4.7-e2e-aws-upgrade",
88-
"periodic-ci-openshift-release-master-nightly-4.9-e2e-aws-arm64",
89-
"periodic-ci-openshift-release-master-nightly-4.9-e2e-aws-csi-migration",
90-
"periodic-ci-openshift-release-master-nightly-4.9-e2e-aws-proxy",
91-
"periodic-ci-openshift-release-master-nightly-4.9-e2e-aws-workers-rhel7",
92-
"periodic-ci-openshift-release-master-nightly-4.9-e2e-metal-ipi-ovn-dualstack",
93-
"periodic-ci-openshift-release-master-nightly-4.9-e2e-metal-ipi-ovn-dualstack-local-gateway",
94-
"periodic-ci-openshift-release-master-nightly-4.9-e2e-metal-ipi-ovn-ipv6",
95-
"periodic-ci-openshift-release-master-nightly-4.9-e2e-remote-libvirt-ppc64le",
96-
"periodic-ci-openshift-release-master-nightly-4.9-e2e-remote-libvirt-s390x",
97-
"periodic-ci-openshift-release-master-nightly-4.9-openshift-ipi-azure-arcconformance",
98-
"periodic-ci-openshift-release-master-nightly-4.9-upgrade-from-stable-4.7-e2e-aws-upgrade-paused",
99-
"periodic-ci-openshift-release-master-nightly-4.9-upgrade-from-stable-4.8-e2e-aws-upgrade",
100-
"release-openshift-ocp-installer-e2e-azure-ovn-4.9",
101-
"release-openshift-ocp-installer-e2e-gcp-ovn-4.9",
81+
82+
// https://issues.redhat.com/browse/OSD-8071
10283
"release-openshift-ocp-osd-aws-nightly-4.9",
10384
"release-openshift-ocp-osd-gcp-nightly-4.9",
104-
"release-openshift-origin-installer-e2e-aws-disruptive-4.9",
105-
"release-openshift-origin-installer-e2e-aws-sdn-network-stress-4.9",
106-
"release-openshift-origin-installer-e2e-aws-upgrade-4.6-to-4.7-to-4.8-to-4.9-ci",
107-
"release-openshift-origin-installer-old-rhcos-e2e-aws-4.9",
10885

109-
// Compact jobs alerting on etcdGRPCRequestsSlow
110-
// BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1986119
86+
// https://bugzilla.redhat.com/show_bug.cgi?id=1979966
87+
"periodic-ci-openshift-release-master-nightly-4.9-e2e-aws-workers-rhel7",
88+
89+
// https://bugzilla.redhat.com/show_bug.cgi?id=1997345
90+
"periodic-ci-openshift-multiarch-master-nightly-4.9-ocp-e2e-remote-libvirt-ppc64le",
91+
"periodic-ci-openshift-multiarch-master-nightly-4.9-ocp-e2e-compact-remote-libvirt-ppc64le",
92+
93+
// https://bugzilla.redhat.com/show_bug.cgi?id=1979962
11194
"periodic-ci-openshift-release-master-ci-4.9-e2e-aws-compact",
11295
"periodic-ci-openshift-release-master-ci-4.9-e2e-aws-compact-upgrade",
11396
"periodic-ci-openshift-release-master-ci-4.9-e2e-azure-compact",
11497
"periodic-ci-openshift-release-master-ci-4.9-e2e-azure-compact-upgrade",
11598
"periodic-ci-openshift-release-master-ci-4.9-e2e-gcp-compact",
11699
"periodic-ci-openshift-release-master-ci-4.9-e2e-gcp-compact-upgrade",
117-
"periodic-ci-openshift-release-master-nightly-4.9-e2e-compact-remote-libvirt-ppc64le",
118-
"periodic-ci-openshift-release-master-nightly-4.9-e2e-compact-remote-libvirt-s390x",
100+
"release-openshift-ocp-installer-e2e-metal-compact-4.9",
101+
"release-openshift-origin-installer-e2e-aws-sdn-network-stress-4.9",
119102
)
120103
)
121104

@@ -140,10 +123,16 @@ func (v openshiftVariants) IdentifyVariants(jobName string) []string {
140123
}
141124
}()
142125

126+
// If a job is in never-stable, it is excluded from other possible variant aggregations
143127
if v.IsJobNeverStable(jobName) {
144128
return []string{"never-stable"}
145129
}
146130

131+
// Tech preview jobs are excluded from other possible variant aggregations
132+
if techpreview.MatchString(jobName) {
133+
return []string{"techpreview"}
134+
}
135+
147136
// if it's a promotion job, it can't be a part of any other variant aggregation
148137
if promoteRegex.MatchString(jobName) {
149138
variants = append(variants, "promote")
@@ -156,6 +145,7 @@ func (v openshiftVariants) IdentifyVariants(jobName string) []string {
156145
if azureRegex.MatchString(jobName) {
157146
variants = append(variants, "azure")
158147
}
148+
159149
if compactRegex.MatchString(jobName) {
160150
variants = append(variants, "compact")
161151
}

0 commit comments

Comments
 (0)