Skip to content

Update changelog and test image to release-v1.17.1 #1971

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

Sneha-at
Copy link
Contributor

@Sneha-at Sneha-at commented Mar 5, 2025

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:

/kind api-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:
Updates the testing image to latest tag for new release and adds changelog for the new release release-1.17 branch.
Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

NONE

@k8s-ci-robot k8s-ci-robot added release-note-none Denotes a PR that doesn't merit a release note. kind/documentation Categorizes issue or PR as related to documentation. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Mar 5, 2025
@k8s-ci-robot k8s-ci-robot requested review from leiyiz and mattcary March 5, 2025 18:47
@k8s-ci-robot k8s-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Mar 5, 2025
@Sneha-at Sneha-at changed the base branch from master to release-1.17 March 5, 2025 18:51
@k8s-ci-robot k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Mar 5, 2025
@mattcary
Copy link
Contributor

mattcary commented Mar 5, 2025

This should be done in the master only, and not the release branch.

The change log should have been deleted from the release branch as well, as just the master changelog is kept up.

Also, we only deploy out of the master branch, although I don't think we know the right way to clean up the deploy overlays in the release branches.

@Sneha-at Sneha-at force-pushed the update-release-1.17 branch from 9c5b6ed to 3da315f Compare March 5, 2025 20:35
Copy link

linux-foundation-easycla bot commented Mar 5, 2025

CLA Not Signed

@k8s-ci-robot k8s-ci-robot added cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Mar 5, 2025
@Sneha-at Sneha-at force-pushed the update-release-1.17 branch from 3da315f to f29ec98 Compare March 5, 2025 20:36
@Sneha-at Sneha-at force-pushed the update-release-1.17 branch from f29ec98 to 3b839ab Compare March 5, 2025 20:38
@mattcary
Copy link
Contributor

mattcary commented Mar 5, 2025

/lgtm

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

mattcary commented Mar 5, 2025

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mattcary, Sneha-at

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:

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 Mar 5, 2025
@k8s-ci-robot
Copy link
Contributor

k8s-ci-robot commented Mar 5, 2025

@Sneha-at: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-gcp-compute-persistent-disk-csi-driver-e2e-windows-2019 3b839ab link false /test pull-gcp-compute-persistent-disk-csi-driver-e2e-windows-2019

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

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. I understand the commands that are listed here.

@Sneha-at
Copy link
Contributor Author

Sneha-at commented Mar 5, 2025

/test EasyCLA

@k8s-ci-robot
Copy link
Contributor

@Sneha-at: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

/test pull-gcp-compute-persistent-disk-csi-driver-e2e
/test pull-gcp-compute-persistent-disk-csi-driver-kubernetes-integration
/test pull-gcp-compute-persistent-disk-csi-driver-sanity
/test pull-gcp-compute-persistent-disk-csi-driver-unit
/test pull-gcp-compute-persistent-disk-csi-driver-verify

The following commands are available to trigger optional jobs:

/test pull-gcp-compute-persistent-disk-csi-driver-e2e-windows-2019

Use /test all to run all jobs.

In response to this:

/test EasyCLA

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.

@Sneha-at
Copy link
Contributor Author

Sneha-at commented Mar 5, 2025

/retest-required

@Sneha-at
Copy link
Contributor Author

Sneha-at commented Mar 6, 2025

/easyCLA

@mattcary
Copy link
Contributor

mattcary commented Mar 6, 2025

@Sneha-at has CLA authorization, there's something going wrong with the verification system. Force merging.

@mattcary mattcary merged commit 8a7752f into kubernetes-sigs:release-1.17 Mar 6, 2025
5 of 8 checks passed
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: no Indicates the PR's author has not signed the CNCF CLA. kind/documentation Categorizes issue or PR as related to documentation. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. 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.

3 participants