-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
[bitnami/ksql] problem connecting to broker #75665
Comments
adding in guava for the com.google.common.collect.ImmutableSet failure then fails with:
this one is present in /opt/bitnami/ksql/share/java/kafka/ the connect-api jar
|
I finally got it to run with the following additional jar files:
I also added the following class path (which is probably overkill):
I note that the run script /opt/bitnami/ksql/bin/ksql-run-class has some logic for adding in exta paths non of which exist in this image
we could in the same way add all the folders under /opt/bitnami/ksql/share/java/
the dockerfile https://github.com/bitnami/containers/blob/main/bitnami/ksql/7.8/debian-12/Dockerfile |
I think log4j 1.2 should really be removed as logging is two of the issues here it might be worth including that requires removing:
and then I add the following (for completeness):
it also needs the log4j.properties replacing with a log4j2 file |
I would look at adding a PR if someone could point me to the thing that creates the tar file |
OK I've looked again and it seems the only thing needed is the classpath fix
the folder /opt/bitnami/ksql/share/java/acl contains a shaded jar with all the dependencies but the startup sequence of scripts aren't including it it's a shame it's all in one jar as I really don't want to use log4j 1.2 but that's another issue |
hi @sillencem Then, finally you have found a solution for your problem no? Can I help you with something else? Thanks Álvaro |
I think the start scripts need to be modified as the image doesn't work as it stands I can probably create a PR, I think there is a path issue. Do you know where the tar file is generated I'd be interested in reviewing that as well? |
@sillencem it would be wonderful to receive a PR with this. I invite you to do it. Thanks a lot Álvaro |
This Issue has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thanks for the feedback. |
Due to the lack of activity in the last 5 days since it was marked as "stale", we proceed to close this Issue. Do not hesitate to reopen it later if necessary. |
I think this does need fixing |
Name and Version
bitnami/ksql:7.8.0-debian-12-r0
What architecture are you using?
amd64
What steps will reproduce the bug?
I also have a docker compose file that I'm using to test but the missing jar files seem to be causing the failure and this is the simplest example
What is the expected behavior?
problem connecting to broker
What do you see instead?
a stack trace:
Additional information
adding in the slf4j-api-2.0.9.jar then subsequently fails with:
The text was updated successfully, but these errors were encountered: