kubernetes / test-infra

Test infrastructure for the Kubernetes project.
Apache License 2.0
3.81k stars 2.61k forks source link

use boskos for cadvisor CI jobs #32826

Closed dims closed 1 week ago

dims commented 1 week ago

cadvisor-e2e is not under community infra AFAIK, let's use boskos instead. (note we tried to move these jobs from default cluster to community cluster and landed with this problem)

2024/06/24 10:36:08 process.go:155: Step 'gcloud config set project cadvisor-e2e' finished in 1.153281317s
2024/06/24 10:36:08 main.go:795: Checking existing of GCP ssh keys...
2024/06/24 10:36:08 main.go:805: Checking presence of public key in cadvisor-e2e
2024/06/24 10:36:08 process.go:153: Running: gcloud compute --project=cadvisor-e2e project-info describe
ERROR: (gcloud.compute.project-info.describe) Could not fetch resource:
 - Required 'compute.projects.get' permission for 'projects/cadvisor-e2e'

2024/06/24 10:36:10 process.go:155: Step 'gcloud compute --project=cadvisor-e2e project-info describe' finished in 1.560560106s
k8s-ci-robot commented 1 week ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dims

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[config/jobs/cadvisor/OWNERS](https://github.com/kubernetes/test-infra/blob/master/config/jobs/cadvisor/OWNERS)~~ [dims] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
iwankgb commented 1 week ago

/lgtm

k8s-ci-robot commented 1 week ago

@iwankgb: changing LGTM is restricted to collaborators

In response to [this](https://github.com/kubernetes/test-infra/pull/32826#issuecomment-2186889777): >/lgtm Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.
akhilerm commented 1 week ago

/lgtm

k8s-triage-robot commented 1 week ago

The Kubernetes project has merge-blocking tests that are currently too flaky to consistently pass.

This bot retests PRs for certain kubernetes repos according to the following rules:

You can:

/retest

k8s-ci-robot commented 1 week ago

@dims: Updated the job-config configmap in namespace default at cluster test-infra-trusted using the following files:

In response to [this](https://github.com/kubernetes/test-infra/pull/32826): >`cadvisor-e2e` is not under community infra AFAIK, let's use boskos instead. (note we tried to move these jobs from `default` cluster to community cluster and landed with this problem) > >``` >2024/06/24 10:36:08 process.go:155: Step 'gcloud config set project cadvisor-e2e' finished in 1.153281317s >2024/06/24 10:36:08 main.go:795: Checking existing of GCP ssh keys... >2024/06/24 10:36:08 main.go:805: Checking presence of public key in cadvisor-e2e >2024/06/24 10:36:08 process.go:153: Running: gcloud compute --project=cadvisor-e2e project-info describe >ERROR: (gcloud.compute.project-info.describe) Could not fetch resource: > - Required 'compute.projects.get' permission for 'projects/cadvisor-e2e' > >2024/06/24 10:36:10 process.go:155: Step 'gcloud compute --project=cadvisor-e2e project-info describe' finished in 1.560560106s >``` Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes-sigs/prow](https://github.com/kubernetes-sigs/prow/issues/new?title=Prow%20issue:) repository.
iwankgb commented 1 week ago

@bobbypage @SergeyKanzhelev community is asking about this PR as it blocks merging PRs to cAdvisor. Review will be greatly appreciated 🙇

dims commented 1 week ago

@iwankgb both the presubmit and CI jobs are green now

What exactly are you asking about?

iwankgb commented 1 week ago

I have not realized it is merged. Apologies!