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

KarafGenerator misses jolokia port 8778 #677

Closed
ffang opened this issue Apr 20, 2021 · 4 comments · Fixed by #678
Closed

KarafGenerator misses jolokia port 8778 #677

ffang opened this issue Apr 20, 2021 · 4 comments · Fixed by #678
Assignees
Milestone

Comments

@ffang
Copy link
Contributor

ffang commented Apr 20, 2021

Description

Info

  • Eclipse JKube version :1.2.0
  • Maven version (mvn -v) :
3.6.3
  • Kubernetes / Red Hat OpenShift setup and version :

  • If it's a bug, how to reproduce :

When building a Karaf based image with jkube plugin, there is no jolokia port 8778 in the auto-generated deploymentconfig.yml. But the jolokia port 8778 is important for Karaf based image, since other App like Fuse Console need to talk to jolokia agent via jolokia port

  • Sample Reproducer Project : [GitHub Clone URL]
@rohanKanojia
Copy link
Member

rohanKanojia commented Apr 20, 2021

Are you using zero configuration mode or providing some XML configuration for building image? I see there is a webPort field in KarafGenerator
https://github.com/eclipse/jkube/blob/3af6339ddfa16efbe42ef6e71687f583d74856ee/jkube-kit/generator/karaf/src/main/java/org/eclipse/jkube/generator/karaf/KarafGenerator.java#L45-L49

@ffang
Copy link
Contributor Author

ffang commented Apr 20, 2021

Hi @rohanKanojia ,

Thanks for your reply!

I know there's a 8181, this is for the http-servlet port. But we also need the jolokia port 8778, just like it in JavaExecGenerator.

Best Regards
Freeman

@rohanKanojia
Copy link
Member

Thanks a lot for sending a quick PR. You just need to sign Eclipse Contributor Agreement and add a Signed-off By footer in your commit message for ECA check to pass[0]

https://www.eclipse.org/jkube/contributing#legal

ffang added a commit to ffang/jkube that referenced this issue Apr 20, 2021
Signed-off-by: Freeman Fang <freeman.fang@gmail.com>
@ffang
Copy link
Contributor Author

ffang commented Apr 20, 2021

Thanks @rohanKanojia for the guidance! All done there.

Freeman

@manusa manusa added this to the 1.3.0 milestone Apr 23, 2021
manusa pushed a commit to ffang/jkube that referenced this issue Apr 23, 2021
Signed-off-by: Freeman Fang <freeman.fang@gmail.com>
manusa pushed a commit that referenced this issue Apr 23, 2021
Signed-off-by: Freeman Fang <freeman.fang@gmail.com>
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 a pull request may close this issue.

3 participants