-
Notifications
You must be signed in to change notification settings - Fork 22
✨ feat: upgrade to Kafka 4.x #543
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
Conversation
|
Hi, |
|
I don't have much time to work on this currently. If anyone wants to give it a look, feel free. Tests are hanging ; probably some configuration issue somewhere. |
|
@Dichotomia I've included your change, it seems tests are still hanging indefinitely. But it also seems tests are succesful ; maybe a subprocess/thread is still running and not closed properly. |
|
Ok the tests are now successful locally on my computer. |
|
Hi. We could merge the PR about plugin / dependency updates proposed by steward and renovate first ? |
|
@Dichotomia sure, I was not notified there was some pending PRs, will check my settings! |
be1dc3f to
abcf72e
Compare
|
@Dichotomia good news, tests are passing on the CI now. There was one place where I missed to changed a port and there was a conflict. I don't have any "real project" at work or somewhere else to do some kind of integration testing. I will trust the unit tests and cut a release :) |
Resolves #541