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
Fix #802: Update Fabric8 Kubernetes Client to v5.10.1 #1120
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1120 +/- ##
============================================
- Coverage 49.08% 48.94% -0.15%
+ Complexity 3642 3606 -36
============================================
Files 458 456 -2
Lines 20778 20709 -69
Branches 2829 2824 -5
============================================
- Hits 10199 10135 -64
- Misses 9495 9498 +3
+ Partials 1084 1076 -8
Continue to review full report at Codecov.
|
ab63472
to
4b83328
Compare
jkube-kit/common/src/main/java/org/eclipse/jkube/kit/common/util/KubernetesHelper.java
Outdated
Show resolved
Hide resolved
4b83328
to
24f1197
Compare
ee7b69e
to
af66535
Compare
private static boolean isGenericCustomResourceCompatible(Class<?> clazz){ | ||
return clazz.isAssignableFrom(GenericCustomResource.class); | ||
private static boolean isGenericKubernetesResourceCompatible(Class<?> clazz){ | ||
return clazz.isAssignableFrom(GenericKubernetesResource.class); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
jkube-kit/common/src/main/java/org/eclipse/jkube/kit/common/util/KubernetesHelper.java
Outdated
Show resolved
Hide resolved
jkube-kit/config/service/src/main/java/org/eclipse/jkube/kit/config/service/ApplyService.java
Outdated
Show resolved
Hide resolved
...ava/org/eclipse/jkube/kit/config/service/openshift/OpenshiftBuildServiceIntegrationTest.java
Outdated
Show resolved
Hide resolved
0267896
to
391271d
Compare
...vice/src/main/java/org/eclipse/jkube/kit/config/service/kubernetes/KubernetesClientUtil.java
Outdated
Show resolved
Hide resolved
2856e04
to
b32b2f8
Compare
...vice/src/main/java/org/eclipse/jkube/kit/config/service/kubernetes/KubernetesClientUtil.java
Outdated
Show resolved
Hide resolved
...-kit/config/service/src/test/java/org/eclipse/jkube/kit/config/service/ApplyServiceTest.java
Show resolved
Hide resolved
+ Update Fabric8 Kubernetes Client to v5.10.1 + Remove GenericCustomResource from project and replace it with GenericKubernetesResource(provided by Fabric8 Kubernetes Client) + Refactor ApplyService#applyGenericKubernetesResource CustomResourceDefinition discovery logic to use genericKubernetesResources(apiVersion, kind) method instead + Update Framework CustomResourceDefinition manifest in Spring Boot CRD Quickstart to apiextensions.k8s.io/v1 from apiextensions.k8s.io/v1beta1 Signed-off-by: Rohan Kumar <rohaan@redhat.com>
…OpenShiftMockServer Signed-off-by: Rohan Kumar <rohaan@redhat.com>
Signed-off-by: Rohan Kumar <rohaan@redhat.com>
b32b2f8
to
cbdb089
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thx!
SonarCloud Quality Gate failed. |
Description
Fix #802
+Refactor
ApplyService#applyGenericKubernetesResource
CustomResourceDefinition discovery logic to usegenericKubernetesResources(apiVersion, kind)
method insteadapiextensions.k8s.io/v1
fromapiextensions.k8s.io/v1beta1
CQs:
Signed-off-by: Rohan Kumar rohaan@redhat.com
Type of change
test, version modification, documentation, etc.)
Checklist