Skip to content

docs: Presubmit triggering and grid validation in README#Release process#664

Merged
k8s-ci-robot merged 1 commit into
kubernetes:masterfrom
DamianSawicki:grid-in-readme
Dec 18, 2024
Merged

docs: Presubmit triggering and grid validation in README#Release process#664
k8s-ci-robot merged 1 commit into
kubernetes:masterfrom
DamianSawicki:grid-in-readme

Conversation

@DamianSawicki

@DamianSawicki DamianSawicki commented Dec 16, 2024

Copy link
Copy Markdown
Collaborator

This adds to the release instructions in README.md that a presubmit
should be triggered before bumping kubedns or nodecache (cluster/addons/dns/) versions in
kubernetes/kubernetes and the test grid https://testgrid.k8s.io/sig-network-gce#gci-gce-serial-kube-dns (and other related tabs) should be verified afterwards. This should prevent issues like kubernetes/kubernetes#123590 and kubernetes/kubernetes#129230 from happening in the future and/or speed up their detection.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Dec 16, 2024
@k8s-ci-robot k8s-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Dec 16, 2024
@DamianSawicki

Copy link
Copy Markdown
Collaborator Author

/cc @aojea

@k8s-ci-robot k8s-ci-robot requested a review from aojea December 16, 2024 12:07
Comment thread README.md Outdated
@DamianSawicki DamianSawicki force-pushed the grid-in-readme branch 2 times, most recently from 51fc642 to d1a68eb Compare December 16, 2024 12:35
@aojea

aojea commented Dec 16, 2024

Copy link
Copy Markdown
Member

/lgtm
/approve

as a follow up we may want to inrtoduce the optional presubmit so we don't risk to have to revert after merge

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 16, 2024
Comment thread README.md Outdated
This adds to the release instructions in README.md that a presubmit
should be triggered before bumping kubedns or nodecache versions in
kubernetes/kubernetes and the test grid should be verified afterwards.

Signed-off-by: Damian Sawicki <dsawicki@google.com>
@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 16, 2024
@DamianSawicki DamianSawicki requested a review from aojea December 16, 2024 14:11
@DamianSawicki DamianSawicki changed the title docs: Grid validation in README#Release process docs: Presubmit triggering and grid validation in README#Release process Dec 16, 2024
@DamianSawicki

Copy link
Copy Markdown
Collaborator Author

/test

@k8s-ci-robot

Copy link
Copy Markdown
Contributor

@DamianSawicki: The /test command needs one or more targets.
The following commands are available to trigger required jobs:

  • /test pull-kubernetes-dns-test

Use /test all to run all jobs.

Details

In response to this:

/test

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.

@DamianSawicki

Copy link
Copy Markdown
Collaborator Author

/test all

@aojea

aojea commented Dec 17, 2024

Copy link
Copy Markdown
Member

/lgtm
/approve
/assign @bowei

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 17, 2024
@kl52752

kl52752 commented Dec 18, 2024

Copy link
Copy Markdown
Collaborator

/lgtm
/approve

@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aojea, DamianSawicki, kl52752, marqc

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

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 18, 2024
@k8s-ci-robot k8s-ci-robot merged commit af16ca1 into kubernetes:master Dec 18, 2024
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. 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. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants