Skip to content

remove legacy prow cluster configuration#36655

Open
BenTheElder wants to merge 2 commits intokubernetes:masterfrom
BenTheElder:remove-defunct-prow
Open

remove legacy prow cluster configuration#36655
BenTheElder wants to merge 2 commits intokubernetes:masterfrom
BenTheElder:remove-defunct-prow

Conversation

@BenTheElder
Copy link
Copy Markdown
Member

this was used by the google operated prow cluster, current config is in github.com/kubernetes/k8s.io

/hold
/cc @kubernetes/sig-k8s-infra-leads

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

@BenTheElder: GitHub didn't allow me to request PR reviews from the following users: kubernetes/sig-k8s-infra-leads.

Note that only kubernetes members and repo collaborators can review this PR, and authors cannot review their own PRs.

Details

In response to this:

this was used by the google operated prow cluster, current config is in github.com/kubernetes/k8s.io

/hold
/cc @kubernetes/sig-k8s-infra-leads

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@k8s-ci-robot k8s-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. approved Indicates a PR has been approved by an approver from all required OWNERS files. area/config Issues or PRs related to code in /config area/prow/bump Updates to the k8s prow cluster sig/testing Categorizes an issue or PR as relevant to SIG Testing. labels Mar 13, 2026
@@ -1,47 +0,0 @@
---
gitHubLogin: "k8s-infra-ci-robot"
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, we might still need this one?

@BenTheElder BenTheElder force-pushed the remove-defunct-prow branch from 56857f5 to 1200389 Compare March 13, 2026 22:08
this was used by the google operated prow cluster, current config is in github.com/kubernetes/k8s.io
@BenTheElder BenTheElder force-pushed the remove-defunct-prow branch from 1200389 to e1669cc Compare March 13, 2026 22:09
Copy link
Copy Markdown
Contributor

@michelle192837 michelle192837 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is fine, but there are references to it in Prow documentation unfortunately (https://github.com/search?q=repo%3Akubernetes-sigs%2Fprow+test-infra%2Fblob%2Fmaster%2Fconfig%2Fprow%2Fcluster&type=code).

I don't think that needs to block this but we should follow up with a doc update. (Might be a good help-wanted or first issue? 🤔 )

/hold

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 16, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: BenTheElder, michelle192837

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

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [BenTheElder,michelle192837]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@BenTheElder
Copy link
Copy Markdown
Member Author

BenTheElder commented Mar 17, 2026

Oh, yeah.

I mean these are definitely out of date, if we want to point at Kubernetes's we can point at the files in github.com/kubernetes/k8s.io

I wonder if we should be pointing at the starter copies in the prow repo though.

cc @petr-muller -- FYI

@ameukam
Copy link
Copy Markdown
Member

ameukam commented Mar 17, 2026

+1 for removing the terraform code.

For the rest, not tombstone files? are we not breaking downstream ?

@BenTheElder
Copy link
Copy Markdown
Member Author

I mean we should update the docs, but these are defunct, they haven't been updated in ages.

starter configs were already migrated IIRC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/config Issues or PRs related to code in /config area/prow/bump Updates to the k8s prow cluster cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/testing Categorizes an issue or PR as relevant to SIG Testing. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants