From 341ff28bfa65cf00ae5a44bc14996bd54a3c7adf Mon Sep 17 00:00:00 2001 From: Stephen Benjamin Date: Thu, 1 Oct 2020 10:50:19 -0400 Subject: [PATCH] Mandate e2e-metal-ipi on cluster-network-operator and ovn-kubernetes Multiple times now, pull requests have merged with e2e-metal-ipi failing with explicitly clear messages in the CI summary that the failure was CNO and/or OVN related. We are working on renaming these jobs starting with 4.7 to indicate they are ovn-ipv6, but it's been proven now that optional jobs are meaningless. This sets e2e-metal-ipi to be required to pass on both CNO and OVN master. These jobs are now as stable as any other platform, so there's no reason to leave them as optional. It's also the only IPv6 coverage that exists in OpenShift today. --- .../openshift-cluster-network-operator-master-presubmits.yaml | 1 - .../openshift-ovn-kubernetes-master-presubmits.yaml | 1 - 2 files changed, 2 deletions(-) diff --git a/ci-operator/jobs/openshift/cluster-network-operator/openshift-cluster-network-operator-master-presubmits.yaml b/ci-operator/jobs/openshift/cluster-network-operator/openshift-cluster-network-operator-master-presubmits.yaml index 8f207ecc112c1..e423e1e3a40e0 100644 --- a/ci-operator/jobs/openshift/cluster-network-operator/openshift-cluster-network-operator-master-presubmits.yaml +++ b/ci-operator/jobs/openshift/cluster-network-operator/openshift-cluster-network-operator-master-presubmits.yaml @@ -625,7 +625,6 @@ presubmits: ci-operator.openshift.io/prowgen-controlled: "true" pj-rehearse.openshift.io/can-be-rehearsed: "true" name: pull-ci-openshift-cluster-network-operator-master-e2e-metal-ipi - optional: true rerun_command: /test e2e-metal-ipi spec: containers: diff --git a/ci-operator/jobs/openshift/ovn-kubernetes/openshift-ovn-kubernetes-master-presubmits.yaml b/ci-operator/jobs/openshift/ovn-kubernetes/openshift-ovn-kubernetes-master-presubmits.yaml index 840d4ec30a943..862a31aa0a656 100644 --- a/ci-operator/jobs/openshift/ovn-kubernetes/openshift-ovn-kubernetes-master-presubmits.yaml +++ b/ci-operator/jobs/openshift/ovn-kubernetes/openshift-ovn-kubernetes-master-presubmits.yaml @@ -334,7 +334,6 @@ presubmits: ci-operator.openshift.io/prowgen-controlled: "true" pj-rehearse.openshift.io/can-be-rehearsed: "true" name: pull-ci-openshift-ovn-kubernetes-master-e2e-metal-ipi - optional: true rerun_command: /test e2e-metal-ipi spec: containers: