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

Move warning options to 'GetKubeClientForConfig' #1716

Merged
merged 1 commit into from
Oct 16, 2020

Conversation

nfnt
Copy link
Member

@nfnt nfnt commented Oct 15, 2020

What this PR does / why we need it:
As this function is used by more functions to create Kubernetes clients, more KUDO subcommands will benefit from reduced CRD deprecation warnings when running on Kubernetes >=1.19.

Before:

$ kubectl kudo get instances
W1015 13:21:29.812992    1507 warnings.go:67] apiextensions.k8s.io/v1beta1 CustomResourceDefinition is deprecated in v1.16+, unavailable in v1.22+; use apiextensions.k8s.io/v1 CustomResourceDefinition
W1015 13:21:29.816993    1507 warnings.go:67] apiextensions.k8s.io/v1beta1 CustomResourceDefinition is deprecated in v1.16+, unavailable in v1.22+; use apiextensions.k8s.io/v1 CustomResourceDefinition
W1015 13:21:29.820192    1507 warnings.go:67] apiextensions.k8s.io/v1beta1 CustomResourceDefinition is deprecated in v1.16+, unavailable in v1.22+; use apiextensions.k8s.io/v1 CustomResourceDefinition
List of current installed instances in namespace "default":
.

After:

$ kubectl kudo get instances
Warning: apiextensions.k8s.io/v1beta1 CustomResourceDefinition is deprecated in v1.16+, unavailable in v1.22+; use apiextensions.k8s.io/v1 CustomResourceDefinition
List of current installed operators including versions and instances in namespace "default":
.

@nfnt nfnt self-assigned this Oct 15, 2020
Copy link
Contributor

@alenkacz alenkacz left a comment

Choose a reason for hiding this comment

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

Oh nice! I assumed it's already in that place but I did not check that when reviewing. Nice 👏

As this function is used by more functions to create Kubernetes clients, more KUDO subcommands will benefit from reduced CRD deprecation warnings when running on Kubernetes >=1.19.

Signed-off-by: Jan Schlicht <[email protected]>
@nfnt nfnt force-pushed the nfnt/dedup-warning-handler branch from f8bc09b to fb28846 Compare October 15, 2020 14:35
@nfnt nfnt merged commit bcfaef8 into main Oct 16, 2020
@nfnt nfnt deleted the nfnt/dedup-warning-handler branch October 16, 2020 08:10
kensipe pushed a commit that referenced this pull request Nov 16, 2020
As this function is used by more functions to create Kubernetes clients, more KUDO subcommands will benefit from reduced CRD deprecation warnings when running on Kubernetes >=1.19.

Signed-off-by: Jan Schlicht <[email protected]>
kensipe pushed a commit that referenced this pull request Nov 16, 2020
As this function is used by more functions to create Kubernetes clients, more KUDO subcommands will benefit from reduced CRD deprecation warnings when running on Kubernetes >=1.19.

Signed-off-by: Jan Schlicht <[email protected]>
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.

4 participants