Keith Wall
Keith Wall
Thank you! I hope to have time to review this over the next couple of days.
@luozhenyu I think the code changes look reasonable. One question - you are relaying the HA PROXY frame to the upstream. Apache Kafka does not understand the HA Proxy protocol...
@carlesarnal thanks for picking this up. > Do we need to support both v2 and v3 wire formats concurrently, or is v2-only sufficient for our use cases? I don't think...
@carlesarnal: Could you fix the Sonar issues please?
> I'd like to take Keith's model of just move to v3 but does fit with our API stability rules? Should we have a switch to throw to enable v3...
Carles Arnal [Today at 10:09 AM](https://kroxylicious.slack.com/archives/C050RNMQW8G/p1763546983911409?thread_ts=1763538908.273049&cid=C050RNMQW8G) I think you're opinion is missing a key point, that I raised in the PR. Even if they were using headers, they're still affected,...
Yeah, I'm seeing this too on my new Mac (apple silicon). I'll investigate as soon as I have time.
> On the request path, short circuiting fetch requests, we don't generally want clients to fail and I don't think we generally want them to. An empty topic is a...
The Proxy CR design proposal https://github.com/kroxylicious/design/blob/main/proposals/001-kroxylicious-operator-api-v1alpha.md#kafkaproxy includes the ability to apply limits to the operand pod.
> It has been tested and it is still not working. As we talked with Dan Davies, this is a problem with writing in a Read-only fs, so we need...