-
Notifications
You must be signed in to change notification settings - Fork 553
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
doc & ci : modify README, upgrade docs and pr-commentor gh action for v3.11.0 #4528
Conversation
/test ci/centos/k8s-e2e-external-storage/1.27 |
/test ci/centos/k8s-e2e-external-storage/1.29 |
/test ci/centos/mini-e2e-helm/k8s-1.27 |
/test ci/centos/upgrade-tests-cephfs |
/test ci/centos/mini-e2e-helm/k8s-1.29 |
/test ci/centos/k8s-e2e-external-storage/1.28 |
/test ci/centos/mini-e2e/k8s-1.27 |
/test ci/centos/upgrade-tests-rbd |
/test ci/centos/mini-e2e-helm/k8s-1.28 |
/test ci/centos/mini-e2e/k8s-1.29 |
/test ci/centos/mini-e2e/k8s-1.28 |
This commit updates README to deprecate 3.9.x release, add 3.10.1/2 release versions and pin latest release to v3.11.0 Signed-off-by: Rakshith R <rar@redhat.com>
This commit adds steps for upgrading from 3.10 to 3.11. Signed-off-by: Rakshith R <rar@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://github.com/ceph/ceph-csi/blob/e92e4bf365b3894d104f2337919100688dee8195/.mergify.yml
here seems to be we are still referring 1.25 which can be removed
# Ceph-CSI <= 3.10 was released before Kubernetes 1.29 | ||
- k8s: "1.29" | ||
branch: "release-v3.9" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be removed as well.
@@ -18,7 +18,7 @@ jobs: | |||
|
|||
strategy: | |||
matrix: | |||
branch: [release-v3.9, release-v3.10, devel] | |||
branch: [release-v3.10, release-v3.11, devel] | |||
k8s: ["1.25", "1.26", "1.27", "1.28", "1.29"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove 1.25 Ref from here
This commit adds rules for release-v3.11 and removes rules for release-v3.9. Signed-off-by: Rakshith R <rar@redhat.com>
This commit adds support for v3.11 backports and removes support for v3.9 backports. Signed-off-by: Rakshith R <rar@redhat.com>
- "status-success=ci/centos/mini-e2e/k8s-1.26" | ||
- "status-success=ci/centos/mini-e2e/k8s-1.27" | ||
- "status-success=ci/centos/mini-e2e/k8s-1.28" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lines 151 to 205 in 45178c2
- name: update dependencies by dependabot (skip commitlint) | |
conditions: | |
- or: | |
- and: | |
- author=dependabot[bot] | |
- label!=DNM | |
- base~=^(release-.+)$ | |
- "#approved-reviews-by>=2" | |
- "#changes-requested-reviews-by=0" | |
- "approved-reviews-by=@ceph/ceph-csi-contributors" | |
- "approved-reviews-by=@ceph/ceph-csi-maintainers" | |
- "status-success=codespell" | |
- "status-success=multi-arch-build" | |
- "status-success=go-test" | |
- "status-success=golangci-lint" | |
- "status-success=mod-check" | |
- "status-success=lint-extras" | |
- "status-success=ci/centos/k8s-e2e-external-storage/1.26" | |
- "status-success=ci/centos/k8s-e2e-external-storage/1.27" | |
- "status-success=ci/centos/k8s-e2e-external-storage/1.28" | |
- "status-success=ci/centos/mini-e2e-helm/k8s-1.26" | |
- "status-success=ci/centos/mini-e2e-helm/k8s-1.27" | |
- "status-success=ci/centos/mini-e2e-helm/k8s-1.28" | |
- "status-success=ci/centos/mini-e2e/k8s-1.26" | |
- "status-success=ci/centos/mini-e2e/k8s-1.27" | |
- "status-success=ci/centos/mini-e2e/k8s-1.28" | |
- "status-success=ci/centos/upgrade-tests-cephfs" | |
- "status-success=ci/centos/upgrade-tests-rbd" | |
- "status-success=DCO" | |
- and: | |
- author=dependabot[bot] | |
- label!=DNM | |
- base=devel | |
- "#approved-reviews-by>=2" | |
- "#changes-requested-reviews-by=0" | |
- "approved-reviews-by=@ceph/ceph-csi-contributors" | |
- "approved-reviews-by=@ceph/ceph-csi-maintainers" | |
- "status-success=codespell" | |
- "status-success=multi-arch-build" | |
- "status-success=go-test" | |
- "status-success=golangci-lint" | |
- "status-success=mod-check" | |
- "status-success=lint-extras" | |
- "status-success=ci/centos/k8s-e2e-external-storage/1.27" | |
- "status-success=ci/centos/k8s-e2e-external-storage/1.28" | |
- "status-success=ci/centos/k8s-e2e-external-storage/1.29" | |
- "status-success=ci/centos/mini-e2e-helm/k8s-1.27" | |
- "status-success=ci/centos/mini-e2e-helm/k8s-1.28" | |
- "status-success=ci/centos/mini-e2e-helm/k8s-1.29" | |
- "status-success=ci/centos/mini-e2e/k8s-1.27" | |
- "status-success=ci/centos/mini-e2e/k8s-1.28" | |
- "status-success=ci/centos/mini-e2e/k8s-1.29" | |
- "status-success=ci/centos/upgrade-tests-cephfs" | |
- "status-success=ci/centos/upgrade-tests-rbd" | |
- "status-success=DCO" |
is not checking 1.26 for devel branch is that expected?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's how the config is currently, I am not able to find the commit that removed 1.26 from that list.
do you want me to add that in this pr ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will revisit the mergify later, not required for now in that case
@Mergifyio queue |
✅ The pull request has been merged automaticallyThe pull request has been merged automatically at 4aa53c8 |
/test ci/centos/upgrade-tests-cephfs |
/test ci/centos/upgrade-tests-rbd |
/test ci/centos/k8s-e2e-external-storage/1.28 |
/test ci/centos/k8s-e2e-external-storage/1.27 |
/test ci/centos/mini-e2e-helm/k8s-1.27 |
/test ci/centos/mini-e2e-helm/k8s-1.28 |
/test ci/centos/mini-e2e/k8s-1.27 |
/test ci/centos/mini-e2e/k8s-1.28 |
/test ci/centos/k8s-e2e-external-storage/1.29 |
/test ci/centos/mini-e2e-helm/k8s-1.29 |
/test ci/centos/mini-e2e/k8s-1.29 |
Describe what this PR does
Checklist:
guidelines in the developer
guide.
Request
notes
updated with breaking and/or notable changes for the next major release.
Show available bot commands
These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:
/retest ci/centos/<job-name>
: retest the<job-name>
after unrelatedfailure (please report the failure too!)