Skip to content

Remove periodic robustness test from presubmits#34848

Merged
k8s-ci-robot merged 1 commit intokubernetes:masterfrom
ahrtr:robust_test_20250524
May 25, 2025
Merged

Remove periodic robustness test from presubmits#34848
k8s-ci-robot merged 1 commit intokubernetes:masterfrom
ahrtr:robust_test_20250524

Conversation

@ahrtr
Copy link
Member

@ahrtr ahrtr commented May 24, 2025

Revert #34834

/hold

Let me hold cancel this PR once etcd-io/etcd#20019 gets merged.

cc @ivanvc @jmhbnz @serathius

Signed-off-by: Benjamin Wang <benjamin.ahrtr@gmail.com>
@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. labels May 24, 2025
@k8s-ci-robot k8s-ci-robot requested review from ivanvc and wenjiaswe May 24, 2025 20:47
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. area/config Issues or PRs related to code in /config area/jobs sig/etcd Categorizes an issue or PR as relevant to SIG Etcd. sig/testing Categorizes an issue or PR as relevant to SIG Testing. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels May 24, 2025
@ahrtr
Copy link
Member Author

ahrtr commented May 25, 2025

/hold cancel

The PR etcd-io/etcd#20019 has been merged, we can merge this as well. cc. @ivanvc @jmhbnz @serathius

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 25, 2025
@ahrtr
Copy link
Member Author

ahrtr commented May 25, 2025

/assign @ivanvc @jmhbnz

@dims
Copy link
Member

dims commented May 25, 2025

/approve
/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 25, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ahrtr, dims

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:

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

@dims
Copy link
Member

dims commented May 25, 2025

@ahrtr please remove hold when ready

@ahrtr
Copy link
Member Author

ahrtr commented May 25, 2025

@ahrtr please remove hold when ready

the /hold already removed 6 hours ago, thx

@k8s-ci-robot k8s-ci-robot merged commit 1f761ee into kubernetes:master May 25, 2025
7 checks passed
@k8s-ci-robot
Copy link
Contributor

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

  • key etcd-presubmits.yaml using file config/jobs/etcd/etcd-presubmits.yaml
Details

In response to this:

Revert #34834

/hold

Let me hold cancel this PR once etcd-io/etcd#20019 gets merged.

cc @ivanvc @jmhbnz @serathius

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.

@ahrtr ahrtr deleted the robust_test_20250524 branch May 25, 2025 14:41
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/jobs cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/etcd Categorizes an issue or PR as relevant to SIG Etcd. sig/testing Categorizes an issue or PR as relevant to SIG Testing. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants