Skip to content

ADR012 Supported Kubernetes Versions#134

Merged
fhennig merged 12 commits into
mainfrom
adr-supported-kubernetes-versions.adoc
Feb 7, 2022
Merged

ADR012 Supported Kubernetes Versions#134
fhennig merged 12 commits into
mainfrom
adr-supported-kubernetes-versions.adoc

Conversation

@fhennig

@fhennig fhennig commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

stackabletech/issues#113

I incoporated Sigis research into the ADR, added a bit of mine own too.

Maybe the reasoning behind the decision could be fleshed out more, I added some pros and cons how I thought about the decision.

I've updated the documentation along with the ADR.

@fhennig fhennig requested a review from lfrancke February 4, 2022 10:12
@fhennig

fhennig commented Feb 4, 2022

Copy link
Copy Markdown
Contributor Author

I want to add that according to my understanding, it would be best that we from now on use the 3rd latest kube API to compile against, so at the moment that would be 1.20. Since all APIs in kube are always supported for at least 3 versions, that would - to my understanding - mean that our binaries are also compatible with the latest three versions. I think it might be as simple as changing the v1_22 in this line to a v1_21

Comment thread modules/adr/pages/ADR012-supported-kubernetes-versions.adoc Outdated
Comment thread modules/adr/pages/ADR012-supported-kubernetes-versions.adoc Outdated
Comment thread modules/adr/pages/ADR012-supported-kubernetes-versions.adoc Outdated
Comment thread modules/adr/pages/ADR012-supported-kubernetes-versions.adoc Outdated
Comment thread modules/adr/pages/ADR012-supported-kubernetes-versions.adoc
@fhennig fhennig requested a review from lfrancke February 7, 2022 10:22
@fhennig fhennig merged commit 6827eb1 into main Feb 7, 2022
@fhennig fhennig deleted the adr-supported-kubernetes-versions.adoc branch February 7, 2022 14:39
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