Skip to content

Support policy for new minor versions on older major versions#423

Merged
k8s-ci-robot merged 1 commit intokubernetes-csi:masterfrom
chrishenzie:minor-version-support
Apr 30, 2021
Merged

Support policy for new minor versions on older major versions#423
k8s-ci-robot merged 1 commit intokubernetes-csi:masterfrom
chrishenzie:minor-version-support

Conversation

@chrishenzie
Copy link
Contributor

@chrishenzie chrishenzie commented Apr 4, 2021

See #419 for more context.

Any assistance with clarifying the wording is appreciated.

Does this PR introduce a user-facing change?

NONE

@k8s-ci-robot k8s-ci-robot added the do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. label Apr 4, 2021
@k8s-ci-robot k8s-ci-robot requested review from msau42 and saad-ali April 4, 2021 02:35
@k8s-ci-robot k8s-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Apr 4, 2021
@chrishenzie
Copy link
Contributor Author

@msau42 @xing-yang

@k8s-ci-robot k8s-ci-robot added release-note-none Denotes a PR that doesn't merit a release note. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Apr 4, 2021
To minimize the number of branches we need to support, we do not have a general
policy for releasing new minor versions on older majors. We will make exceptions
for work related to meeting production readiness requirements.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should add a clause that this older major can only be one version lower than the current major and also the time between the two major releases is less than 3-4 months, etc.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. Does the wording above make sense?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i'm not sure how to read this. Are we trying to say that if "X.0.0" and "X+1.0.0" were released about a quarter apart, then we will consider some exception?

But if it was "X.2.0" and "X+1.0.0", then we won't consider any exception?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something like that. We only look at the initial release times of the major releases for minor release exceptions. Otherwise we could end up with too many releases. For example, if X.0.0 was released a year before X+1.0.0, we should move on and not to keep releasing minor releases on X.

Copy link
Contributor Author

@chrishenzie chrishenzie Apr 13, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does the wording in this PR capture this in an understandable way? Or what if we worded it like so?

Only prior major versions released within four months of the current major version will be eligible for this exception.

This wording does broaden it to more than one major release, but maybe that's more intuitive than the current wording.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the wording as is captures it, but I wanted to confirm the intended scenario. Maybe give an example with numbers to make it more clear.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just checking if this captures everything. Do we also want to add a section for RBAC not requiring new majors?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think RBACs still require some more discussion, so let's consider that separately.

policy for releasing new minor versions on older majors. We will make exceptions
for work related to meeting production readiness requirements. Only the previous
major version will be eligible for these exceptions, so long as the time between
the previous major version and the current major version is under four months.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thinking about this some more, given that the Kubernetes release cadence is slowling down, I wonder if 4 months is actually enough enough to capture even a single release. Should we bump it to 6 months? @xing-yang

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

To minimize the number of branches we need to support, we do not have a general
policy for releasing new minor versions on older majors. We will make exceptions
for work related to meeting production readiness requirements.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think RBACs still require some more discussion, so let's consider that separately.

@msau42
Copy link
Collaborator

msau42 commented Apr 30, 2021

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 30, 2021
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chrishenzie, msau42

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 Apr 30, 2021
@k8s-ci-robot k8s-ci-robot merged commit ea454bc into kubernetes-csi:master Apr 30, 2021
@chrishenzie chrishenzie deleted the minor-version-support branch April 30, 2021 04:34
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. release-note-none Denotes a PR that doesn't merit a release note. 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.

4 participants