-
Notifications
You must be signed in to change notification settings - Fork 271
Fix support dates #906
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
base: master
Are you sure you want to change the base?
Fix support dates #906
Conversation
365 days and 90 days are not *quite* the same.
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: thatcode The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Welcome @thatcode! |
Hi @thatcode. Thanks for your PR. I'm waiting for a kubernetes-csi member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
Pull Request Test Coverage Report for Build 14759356772Details
💛 - Coveralls |
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.
can you make the change in https://github.com/kubernetes-csi/csi-release-tools/blob/master/contrib/get_supported_version_csi-sidecar.py? thanks.
I did wonder why this file was mastered here! Thanks for pointing me to the canonical location - I'll do as you suggest. Once/if I get this changed there, what's the process for changing it here? Is it automatic? Or would this PR still have use for that? I'm probably happy to close this PR and then re-open if it's useful later? What do you suggest? Edit: Now done, in kubernetes-csi/csi-release-tools#276. |
after the PR in csi-release-tools is merged, you could run following command to update the csi-release-tools in this repo: git subtree pull --squash --prefix=release-tools https://github.com/kubernetes-csi/csi-release-tools.git master |
365 days and 90 days are not quite the same.
What type of PR is this?
/kind bug
What this PR does / why we need it:
This file references the support timelines as detailed in https://kubernetes-csi.github.io/docs/project-policies.html#support. This says:
The code in this file uses 365 days as 1 year, and 90 days as 3 months, which is close but not quite the same. Using
relativedelta
enables us to get the dates exactly correct, even on leap years and more often for the 3 month cases.This is quite minor (and probably a bit nit-picking!) but I also know of some automated checkers that don't like having references to 1 year and 365 days on the same line - as they're not the same! I'm having to patch this locally due to this, but thought I'd contribute it back upstream as it's easy.
Key decisions made that should be considered during review:
Which issue(s) this PR fixes:
No current issues are fixed by this PR. Would it help if I raised one? I thought not at this point.
Special notes for your reviewer:
I have tested this with no changes in supported releases. Testing this file is rather specific to the current date! I have used faketime to test this on a variety of dates.
v0.12.1 drops support on 2025-05-25 because v0.12.0 was released on 2024-05-24, so 2025-05-24 is the last supported day.
Does this PR introduce a user-facing change?: