Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 10:26
    codecov[bot] commented #1933
  • 10:25
    baruKreddy review_requested #1933
  • 10:16
    manusa commented #1879
  • 09:52
    coding-cosmos commented #1879
  • 09:29
    manusa commented #1948
  • 09:28
    manusa commented #1948
  • 09:25
    manusa commented #1879
  • 09:25
    manusa reopened #1879
  • 09:22

    manusa on master

    fix (jkube-kit/config/service) … (compare)

  • 09:22
    manusa closed #1940
  • 09:22
    manusa closed #1948
  • 09:21
    manusa commented #1948
  • 07:46
    manusa commented #1948
  • 07:45
    manusa commented #1948
  • 07:45
    manusa commented #1948
  • 07:44
    manusa commented #1948
  • 07:44
    codecov[bot] commented #1948
  • 07:43
    manusa commented #1948
  • 07:43
    codecov[bot] commented #1948
  • 07:42
    codecov[bot] commented #1948
Jendrik Badde
@maycap123
Ok will try it later :)
Kartik Sapra
@theexplorist

On a quick try I wasn't able to get Eclipse working with JKube :-(

Eclipse does not works with jkube, it is very hard to make it work, intellij community edition is the best option. @maycap123

@rohanKanojia @manusa #825 I have updated the PR with required changes and kuberenetes testing, I think everything is working fine. Kindly give your reviews on it.

Have a nice day!

Sandra
@Onyinye91-ctrl
Hello @rohanKanojia for the issue #914. I have done all you said i do. I actually want to share a screenshot but can't find where to. Will do on github
Rohan Kumar
@rohanKanojia
Hi, Have you pushed updated commit to github?
git push origin master -f
Sandra
@Onyinye91-ctrl
@rohanKanojia I have done it. Is it visible in your end?
Rohan Kumar
@rohanKanojia
No
oh, sorry. It seems to be fixed now(ECA check)
Sandra
@Onyinye91-ctrl
Okay. Thanks
Rohan Kumar
@rohanKanojia
sorry for not being clear but I wasn't referring to this. I wanted to get commit title updated. It's first commit right now. could you please do git commit --amend --signoff and set title same as this comment? You would need to push to github after doing this
Sandra
@Onyinye91-ctrl
How do i set the title
Rohan Kumar
@rohanKanojia
run git commit --amend, it will open up an editor where you can do it
Sandra
@Onyinye91-ctrl
Okay
Jendrik Badde
@maycap123
I'm working on #802 right now. The task is to replace GenericCustomResource with GenericKubernetesResource throughout the whole project. Is there an easy way to find all occurrences of GenericCustomResource?
Sandra
@Onyinye91-ctrl
@rohanKanojia I have made the changes but it seems that it didn't work. Can you check it out?
Rohan Kumar
@rohanKanojia
Sorry, it doesn't look like your changes are actually pushed. It should get reflected on eclipse/jkube#855
@maycap123 : git grep -in GenericCustomResource
You can also do Ctrl+Shift+F in IntelliJ to search across project
Sandra
@Onyinye91-ctrl
@rohanKanojia I have done it. Please confirm
Rohan Kumar
@rohanKanojia
yes, it seems to be updated. Thanks
Jendrik Badde
@maycap123
For my understanding: i should replace GenericCustomResource with GenericKubernetesResource. In every method, class and so on?
Should i then delete the class GenericCustomResource as well?
Rohan Kumar
@rohanKanojia
yes
Jendrik Badde
@maycap123
Ok thanks
yberkov
@yaakov-berkovitch
Hi, I have my own customized helm chart files and want jkube only to package it and push it to repo (using mvn k8s:helm). I cannot make it working. How should I specify the location of my helm chart files ?
Rohan Kumar
@rohanKanojia
I don't think we support this
yberkov
@yaakov-berkovitch
Thanks @rohanKanojia . I read the answer @manusa gave to stackoverflow and explained there that JKube is meanly for generating. But for building docker image I can provide my own Dockerfile. So I expect that the same behavior will be also for helm charts, that is giving a folder where my chart files are located. BTW, our charts are not so simple, and we are using chart libraries, templates...And I would prefer to use only one plugin (jkube).
Rohan Kumar
@rohanKanojia
good point.
Marc is on PTO and will be back next week
Shall we create an issue about it for discussion?
yberkov
@yaakov-berkovitch
@rohanKanojia, not sure if you addressed your question to me, but anyway do you want me to open a feature request ?
Rohan Kumar
@rohanKanojia
Yes, please.
Jendrik Badde
@maycap123
What do i do to include the added lines in tests so that i pass the sonarcloud test? In Pull request 860
Rohan Kumar
@rohanKanojia
You would need to add tests. Sonar usually expects at least 80% test coverage
Jendrik Badde
@maycap123
Ok never did that. Is there a good explanation anywhere?
IntelliJ's unit test video can give you a quick walkthrough of writing tests and checking code coverage via IDE
Jendrik Badde
@maycap123
Thanks, i will look into it! :)
Slok
@kumar-cloud

Issue: I have docker limiting issue with busybox. This is impicitly creaed/encriched by Jkube in k8s environment.

is there anyway to change busybox image name in jkube-volume-permission container enricher .....I want to use my own ECR image ,
Is there any coniguration for the same ?

WDYT? Shall we create an issue?
Slok
@kumar-cloud
Thanks @rohanKanojia
I feel so , we need to create an issue. I did create an issue eclipse/jkube#891
These days mostly every one have their own ECR from aws,azure..etc..
But anybody have temporary fix for this ..?
Rohan Kumar
@rohanKanojia
Could you please try by providing a custom yaml fragment in src/main/jkube ?
Slok
@kumar-cloud
I did that, we have custom yamls with initContainer named same as : jkube-volume-permission and with different image name , but that not picksup.....