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

deps: Bump Kubernetes|OpenShift client to 5.10.1 (or latest) #802

Closed
4 tasks done
manusa opened this issue Jul 30, 2021 · 13 comments · Fixed by #1120
Closed
4 tasks done

deps: Bump Kubernetes|OpenShift client to 5.10.1 (or latest) #802

manusa opened this issue Jul 30, 2021 · 13 comments · Fixed by #1120
Assignees
Labels
dependencies Pull requests or issues related to a dependency update
Milestone

Comments

@manusa
Copy link
Member

manusa commented Jul 30, 2021

Description

  • Bump the KubernetesClient version to latest release (---> 5.10.1 or latest).
  • Replace usage of GenericCustomResource with GenericKubernetesResource
  • Use the new getApiGroups and getApiResources instead of CustomResourceDefinitions (which may not be available or might be blocked by Cluster Roles) ---> Maybe this was partially addressed when using the resource() DSL method, so might not be necessary to directly access the apiGroups and apiResources endpoints
  • Create corresponding CQs
@manusa manusa added this to Backlog in Sprint #205 + #206 Jul 30, 2021
@manusa manusa added this to the 1.5.0 milestone Jul 30, 2021
@manusa manusa changed the title deps: Bump Kubernetes|OpenShift client to 5.6.0 deps: Bump Kubernetes|OpenShift client to 5.6.0 (or latest) Jul 30, 2021
@manusa manusa added the dependencies Pull requests or issues related to a dependency update label Jul 30, 2021
@manusa manusa moved this from Backlog to Planned in Sprint #205 + #206 Aug 12, 2021
@manusa manusa changed the title deps: Bump Kubernetes|OpenShift client to 5.6.0 (or latest) deps: Bump Kubernetes|OpenShift client to 5.7.0 (or latest) Aug 12, 2021
@chambora1312
Copy link
Contributor

I would like to try this task, if I'm allowed to have several assignments :) if so, I need some further explanation: is there a trick to find out in which pom.xml the soon-to-be changing dependency is? what is to change with "GenericKubernetesResource? And last but not least, what is a cq? :)

@rohanKanojia
Copy link
Member

rohanKanojia commented Aug 16, 2021

It's here
https://github.com/eclipse/jkube/blob/5a1828c0ece30567fdcf89ce7668bb8c2003211d/jkube-kit/parent/pom.xml#L57

In order to handle CustomResources in a generic way, we introduced GenericCustomResource. But in latest releases of Fabric8 Kubernetes Java Client, GenericKubernetesResource got introduced. We need to delete our own version of GenricCustomResource and use GenericKubernetesResource coming from KubernetesClient here

https://github.com/eclipse/jkube/blob/5a1828c0ece30567fdcf89ce7668bb8c2003211d/jkube-kit/config/service/src/main/java/org/eclipse/jkube/kit/config/service/ApplyService.java#L486-L499

You don't need to worry about CQ. It's something that project maintainers have to create whenever they add/upgrade new dependency.

@chambora1312
Copy link
Contributor

Then please assign me :)

@chambora1312
Copy link
Contributor

Am I supposed to change only in the given code citation? or throughout ApplyService and ApplyServiceTest.java?

@rohanKanojia
Copy link
Member

Am I supposed to change only in the given code citation? or throughout ApplyService and ApplyServiceTest.java?

You're right. You're supposed to replace usages of GenericCustomResource across project. Idea is to get rid of GenericCustomResource and use GenericKubernetesResource from kubernetes java client

@chambora1312
Copy link
Contributor

Is there a trick to find out in which class GenericCustomResource is used?

@rohanKanojia
Copy link
Member

You can do a git grep -in GenericCustomResource to list all files referencing GenericCustomResource

@chambora1312
Copy link
Contributor

Thanks, i will do this later and maybe i will issue the pull request then :)

@manusa manusa added this to Planned in Sprint #207 Sep 8, 2021
@manusa manusa changed the title deps: Bump Kubernetes|OpenShift client to 5.7.0 (or latest) deps: Bump Kubernetes|OpenShift client to 5.7.3 (or latest) Sep 8, 2021
@chambora1312
Copy link
Contributor

Sorry, I'm still on this, got a lot on my plate right now. Hope that's ok

@manusa manusa modified the milestones: 1.5.0, 1.6.0 Sep 29, 2021
@manusa manusa added this to Backlog in Sprint #210 Nov 8, 2021
@manusa manusa changed the title deps: Bump Kubernetes|OpenShift client to 5.7.3 (or latest) deps: Bump Kubernetes|OpenShift client to 5.10.0 (or latest) Nov 8, 2021
@manusa manusa moved this from Backlog to Planned in Sprint #210 Nov 10, 2021
@rohanKanojia
Copy link
Member

@maycap123 : Hello, I want to pick this task from where you left. Do you have your branch pushed somewhere? I can cherry-pick your commit and include it in the PR,

@rohanKanojia rohanKanojia self-assigned this Nov 11, 2021
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 11, 2021
Signed-off-by: Rohan Kumar <rohaan@redhat.com>
@rohanKanojia rohanKanojia moved this from Planned to In progress in Sprint #210 Nov 12, 2021
@manusa manusa changed the title deps: Bump Kubernetes|OpenShift client to 5.10.0 (or latest) deps: Bump Kubernetes|OpenShift client to 5.10.1 (or latest) Nov 12, 2021
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 12, 2021
Signed-off-by: Rohan Kumar <rohaan@redhat.com>
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 16, 2021
Signed-off-by: Rohan Kumar <rohaan@redhat.com>
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 16, 2021
Signed-off-by: Rohan Kumar <rohaan@redhat.com>
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 16, 2021
Signed-off-by: Rohan Kumar <rohaan@redhat.com>
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 16, 2021
Signed-off-by: Rohan Kumar <rohaan@redhat.com>
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 16, 2021
Signed-off-by: Rohan Kumar <rohaan@redhat.com>
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 16, 2021
Signed-off-by: Rohan Kumar <rohaan@redhat.com>
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 16, 2021
Signed-off-by: Rohan Kumar <rohaan@redhat.com>
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 17, 2021
Signed-off-by: Rohan Kumar <rohaan@redhat.com>
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 17, 2021
+ 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>
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 17, 2021
+ 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>
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 17, 2021
+ 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>
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 17, 2021
+ 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>
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 17, 2021
+ 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>
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Nov 17, 2021
+ 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>
Sprint #210 automation moved this from In progress to Done Nov 17, 2021
manusa pushed a commit that referenced this issue Nov 17, 2021
+ 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>
@robinroos
Copy link
Contributor

5.12.0 is available. Should that be adopted before jkube 1.6.0 is cut?

@manusa
Copy link
Member Author

manusa commented Feb 3, 2022

5.12.0 is available. Should that be adopted before jkube 1.6.0 is cut?

There shouldn't be any notable differences for JKube users between 5.10.1 and 5.12.0.

We'll bump the client version in our next minor release (which shouldn't take as long as this one)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests or issues related to a dependency update
Projects
No open projects
Sprint #207
  
Planned
Sprint #210
  
Done
Development

Successfully merging a pull request may close this issue.

4 participants