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

Drop support for Kubernetes clusters older than v1.13 #2414

Merged
merged 1 commit into from
May 18, 2023

Conversation

lblackstone
Copy link
Member

Proposed changes

As part of upgrading the client-go dependency, we need to drop support for Kubernetes versions older than v1.13.0 (released December 2018, EOL as of October 2019). client-go has dropped support [1] for the dry-run verifier because all cluster versions v1.13 and greater include this support. As a result, we drop the dry run check from the provider, and enforce a minimum version of v1.13.

Additionally, one of the options for kustomize support moved [2], so update the calling code to match.

[1] kubernetes/kubernetes#114294
[2] kubernetes-sigs/kustomize#4945

Related issues (optional)

Fix #2386

@lblackstone lblackstone requested a review from a team May 16, 2023 21:30
@lblackstone
Copy link
Member Author

Hmm, something about this change broke preview in one of the .NET tests. Working to track that down.

@lblackstone lblackstone force-pushed the lblackstone/upgrade-client branch 3 times, most recently from 1eff379 to 873e0f8 Compare May 17, 2023 20:10
@lblackstone
Copy link
Member Author

@rquitales I'm waiting to see if the tests pass, and then may pull the gvkExists changes out into a separate PR. Upon further inspection, I think the previous check was pretty inefficient, so these changes would be worth shipping ahead of the breaking v4 changes.

@pulumi pulumi deleted a comment from github-actions bot May 18, 2023
@pulumi pulumi deleted a comment from github-actions bot May 18, 2023
@pulumi pulumi deleted a comment from github-actions bot May 18, 2023
@pulumi pulumi deleted a comment from github-actions bot May 18, 2023
@pulumi pulumi deleted a comment from github-actions bot May 18, 2023
@pulumi pulumi deleted a comment from github-actions bot May 18, 2023
@pulumi pulumi deleted a comment from github-actions bot May 18, 2023
@github-actions
Copy link

Does the PR have any schema changes?

Found 2 breaking changes:
Resource "kubernetes:helm.sh/v2:Chart" missing
Type "kubernetes:helm.sh/v2:FetchOpts" missing
No new resources/functions.

@lblackstone lblackstone marked this pull request as draft May 18, 2023 16:47
@lblackstone
Copy link
Member Author

Tests are passing now, but I'm going to pull the gvkExists implementation into a separate PR, and then rebase this PR on that.

As part of upgrading the client-go dependency, we need to drop support for Kubernetes versions older than v1.13.0 (released December 2018, EOL as of October 2019). client-go has dropped support [1] for the dry-run verifier because all cluster versions v1.13 and greater include this support. As a result, we drop the dry run check from the provider, and enforce a minimum version of v1.13.

Additionally, one of the options for kustomize support moved [2], so update the calling code to match.

[1] kubernetes/kubernetes#114294
[2] kubernetes-sigs/kustomize#4945
@lblackstone lblackstone marked this pull request as ready for review May 18, 2023 17:54
@lblackstone lblackstone merged commit 9b779b3 into v4 May 18, 2023
17 checks passed
@lblackstone lblackstone deleted the lblackstone/upgrade-client branch May 18, 2023 18:28
lblackstone added a commit that referenced this pull request Jul 14, 2023
As part of upgrading the client-go dependency, we need to drop support for Kubernetes versions older than v1.13.0 (released December 2018, EOL as of October 2019). client-go has dropped support [1] for the dry-run verifier because all cluster versions v1.13 and greater include this support. As a result, we drop the dry run check from the provider, and enforce a minimum version of v1.13.

Additionally, one of the options for kustomize support moved [2], so update the calling code to match.

[1] kubernetes/kubernetes#114294
[2] kubernetes-sigs/kustomize#4945
lblackstone added a commit that referenced this pull request Jul 17, 2023
As part of upgrading the client-go dependency, we need to drop support for Kubernetes versions older than v1.13.0 (released December 2018, EOL as of October 2019). client-go has dropped support [1] for the dry-run verifier because all cluster versions v1.13 and greater include this support. As a result, we drop the dry run check from the provider, and enforce a minimum version of v1.13.

Additionally, one of the options for kustomize support moved [2], so update the calling code to match.

[1] kubernetes/kubernetes#114294
[2] kubernetes-sigs/kustomize#4945
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.

None yet

3 participants