Closed BenTheElder closed 6 days ago
/approve /lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: BenTheElder, dims
The full list of commands accepted by this bot can be found here.
The pull request process is described here
missed a reference to the failing test / dashboard (https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/test-infra/32857/pull-test-infra-unit-test/1806155236738338816), fixed in additional commit https://github.com/kubernetes/test-infra/pull/32857/commits/746a4bd8e5554a2857cb1f34a9d4320a0916ca78
/lgtm
@BenTheElder: Updated the job-config
configmap in namespace default
at cluster test-infra-trusted
using the following files:
kops-periodics-gce.yaml
using file config/jobs/kubernetes/kops/kops-periodics-gce.yaml
gcp-gce.yaml
using file config/jobs/kubernetes/sig-cloud-provider/gcp/gcp-gce.yaml
The one job in this is not passing, doesn't appear to be used.
If we discover anyone does want this job, we should add it back under a different dashboard.
This branch also contains https://github.com/kubernetes/test-infra/pull/32856, because it's smaller (only the first commit) and these PRs are going to conflict on changes to
config/testgrids/google/google.yaml
if done in parallel.I'm hoping #32856 merges first and then this will just be the second and third commit, but merging as one PR would also be fine with me anyhow ...