Skip to content
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

Allow the certification of old versions of Kubernetes when the provider has a current registration #414

Closed
wants to merge 1 commit into from

Conversation

WilliamDenniss
Copy link
Member

Per email thread, updated terms for review.

@@ -64,7 +64,7 @@ _Conformance Time Period; Later Versions of Kubernetes_. A Participant’s permi
* 12 months after the release date of Kubernetes x.y, or
* 9 months after the release date of the next minor release (e.g., Kubernetes x.y+1).

However, notwithstanding the time limits above, the Participant may also continue to use the Certified Kubernetes Marks and Participant Kubernetes Combinations for any prior version of Kubernetes for which the offering was a Qualifying Offering, for so long as (1) the offering continues to be a Qualifying Offering for each subsequent version of Kubernetes; (2) the Qualifying Offering with the subsequent version of Kubernetes is made generally available to users of the prior version; and (3) the Participant otherwise continues to abide by the terms of the Conformance Program.
However, notwithstanding the time limits above, the Participant may also continue to certify and use the Certified Kubernetes Marks and Participant Kubernetes Combinations for any prior version of Kubernetes for which the offering was a Qualifying Offering, for so long as (1) the offering continues to be a Qualifying Offering for each subsequent version of Kubernetes; (2) the Qualifying Offering with the subsequent version of Kubernetes is made generally available to users of the prior version; and (3) the Participant otherwise continues to abide by the terms of the Conformance Program.
Copy link

Choose a reason for hiding this comment

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

Just FYI - this diff not showing what changed is why some groups try to stick with 80 columns :-)
Is the only diff the new words "certify and" in the first line?

@dankohn
Copy link
Contributor

dankohn commented Dec 18, 2018

@WilliamDenniss Could you please add a little more context about why this PR is necessary if #413 is approved?

Today, a provider can certify K8s versions 1.13 and 1.12. If #413 is approved, they would also have the option of certifying 1.11.

If this PR is approved, a provider who first certifies 1.12 or 1.13 would then have the option of certifying versions 1.0 through 1.10. However, security releases are only available for the last 3 versions, so 1.11 and higher. (See https://github.com/kubernetes/community/blob/master/contributors/design-proposals/release/versioning.md )

I'm an explicit -1 on the idea of allowing new certification of a K8s version that is not getting security releases. I understand that some vendors are backporting fixes to older releases, but I'm unconvinced that we want to use the signaling mechanism of certification to support new vendors doing security releases for older versions.

So, unless I'm misunderstanding some of the versioning here, I think support of #413 largely eliminates the need for this PR.

Cc @swinslow

@WilliamDenniss
Copy link
Member Author

Per the discussion on the call today I'm closing this for now, if anyone has a need for this please make your case.

Here's the rationale for the record:

Currently a provider who certified 1.10 can remain certified and continue to offer Certified Kubernetes 1.10 (provided they also have a more current version on offer), however a new participant can not do this. The idea behind this PR was to equalize this so that that all participants can offer the same versions, regardless of when they join or certify (provided they continue to offer a current release, per the terms).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants