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

Bump direct dependencies to their latest releases #1685

Merged
merged 12 commits into from
Oct 2, 2020
Merged

Bump direct dependencies to their latest releases #1685

merged 12 commits into from
Oct 2, 2020

Conversation

nfnt
Copy link
Member

@nfnt nfnt commented Sep 16, 2020

What this PR does / why we need it:
This changes the Kubernetes client API to 0.19.2.
Furthermore, controller-tools has been updated. This version creates V1 CRDs by default, but its configuration has been changed to still create v1beta1 CRDs, as some clusters might have components that don't support V1 CRDs yet.
Go 1.15 is used for builds.

This changes the Kubernetes client API to 0.19.1.
Furthermore, 'controller-tools' has been updated to a version that generates V1 CRDs.

Signed-off-by: Jan Schlicht <jan@d2iq.com>
@nfnt nfnt added the dependencies Pull requests that update a dependency file label Sep 16, 2020
@nfnt nfnt self-assigned this Sep 16, 2020
Jan Schlicht added 3 commits September 16, 2020 11:16
Signed-off-by: Jan Schlicht <jan@d2iq.com>
Signed-off-by: Jan Schlicht <jan@d2iq.com>
Signed-off-by: Jan Schlicht <jan@d2iq.com>
@zen-dog
Copy link
Contributor

zen-dog commented Sep 16, 2020

Furthermore, controller-tools has been updated to a version that generates V1 CRDs

I don't think we can drop support for the v1beta1 today. See for example this recent commit.

@nfnt
Copy link
Member Author

nfnt commented Sep 16, 2020

Yes, switching to V1 CRDs would drop support for Kubernetes < 1.16 which would be in line with the proposed API version policy.

@nfnt
Copy link
Member Author

nfnt commented Sep 16, 2020

Updating controller-tools is blocked by kudobuilder/kuttl#210.

Jan Schlicht added 4 commits September 17, 2020 09:16
Signed-off-by: Jan Schlicht <jan@d2iq.com>
Signed-off-by: Jan Schlicht <jan@d2iq.com>
Signed-off-by: Jan Schlicht <jan@d2iq.com>
Not all tooling is able to handle v1 CRDs yet, let's keep generating v1beta1 CRDs.

Signed-off-by: Jan Schlicht <jan@d2iq.com>
@nfnt
Copy link
Member Author

nfnt commented Oct 1, 2020

I'll keep the CRDs at v1beta1 for now following the input from @zen-dog.
Need to revert some changes in the code, afterwards the CI will be green again.

Jan Schlicht added 4 commits October 2, 2020 12:13
This reverts commit faece87.

Signed-off-by: Jan Schlicht <jan@d2iq.com>
Signed-off-by: Jan Schlicht <jan@d2iq.com>
Signed-off-by: Jan Schlicht <jan@d2iq.com>
Signed-off-by: Jan Schlicht <jan@d2iq.com>
@nfnt nfnt merged commit 2f7c6c2 into main Oct 2, 2020
@nfnt nfnt deleted the nfnt/bump-deps branch October 2, 2020 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants