-
Notifications
You must be signed in to change notification settings - Fork 78
Fix support dates for CSI sidecars #276
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?
Conversation
- Supported releases should probably be done by day, rather than changing on the time of day. - Use 1 year and 3 months as stated in https://kubernetes-csi.github.io/docs/project-policies.html#support, rather than 365 days and 90 days, which 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. |
/ok-to-test |
I'm really sorry if I've misread the guidelines, but step 1 of the guidance says: It feels a bit presumptuous to assign a reviewer, but I think that's what it's telling me to do... Sorry if I am incorrect. /assign @gnufied |
As per step 2 of the guidance: @gnufied Is it possible to have an ETA on a review of this PR? Or some feedback on this change? Thanks! |
Trying next reviewer. Can I please have feedback on this PR, or an estimate of when that might be possible? I have seen this post on the K8s Slack so understand it might not be a good time for this PR, but it'd be good to know this is why the review is being delayed, rather than the silence I'm currently getting! Thanks :) /assign @pohly |
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?: