Skip to content

Commit

Permalink
Make BMO e2e prow job required
Browse files Browse the repository at this point in the history
This removes the required status context for BMO e2e tests and instead
makes the prow job required.

It also removes the report template because this is already covered by
the report template set for plank. This config is currently causing
double messages when a job fails (one for jenkins and one for plank).
  • Loading branch information
lentzi90 committed Jan 18, 2024
1 parent dc181ad commit 421c765
Showing 1 changed file with 1 addition and 11 deletions.
12 changes: 1 addition & 11 deletions prow/manifests/overlays/metal3/config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -138,11 +138,7 @@ branch-protection:
[
"test-centos-e2e-integration-main",
"test-ubuntu-integration-main",
"metal3-bmo-e2e-test",
]
release-0.5:
required_status_checks:
contexts: ["metal3-bmo-e2e-test"]
cluster-api-provider-metal3:
branches:
main:
Expand Down Expand Up @@ -245,11 +241,6 @@ jenkins_operators:
- max_concurrency: 150
max_goroutines: 20
job_url_template: 'https://jenkins.nordix.org/view/Metal3/job/{{.Spec.Job}}/{{.Status.JenkinsBuildID}}/'
report_templates:
"*": >-
[Full PR test history](https://prow.apps.test.metal3.io/pr-history?org={{.Spec.Refs.Org}}&repo={{.Spec.Refs.Repo}}&pr={{with index .Spec.Refs.Pulls 0}}{{.Number}}{{end}}).
[Your PR dashboard](https://prow.apps.test.metal3.io/pr?query=is:pr+state:open+author:{{with
index .Spec.Refs.Pulls 0}}{{.Author}}{{end}}).

periodics:
- name: periodic-stale
Expand Down Expand Up @@ -542,8 +533,7 @@ presubmits:
agent: jenkins
# Don't run unless triggered to avoid wasting resources
always_run: false
# Until we have checked that it works, keep it optional
optional: true
optional: false

metal3-io/cluster-api-provider-metal3:
- name: gomod
Expand Down

0 comments on commit 421c765

Please sign in to comment.