Federico Valeri

Results 62 comments of Federico Valeri

@LiamClarkeNZ these changes work fine on my local environment (minikube) and on a remote cluster (OCP 4.10). I ran some manual tests and the reproducer described in [ENTMQST-4087](https://issues.redhat.com/browse/ENTMQST-4087) without any...

@LiamClarkeNZ The Session.stop issue is addressed in https://github.com/strimzi/strimzi-kafka-operator/pull/7165, so you don't need to worry about that.

The `connections.max.reauth.ms` configuration only applies to SASL authentication. The `KafkaClientAuthenticationTls` should not have `maxReauthMs` property, even if setting a value would not have any effect.

Nope. It scales, but given that PDB.maxUnavailable remains fixed at 1, you get warnings when draining the K8s node. On OpenShift the wrong PDB state causes cluster alerts at warning...

Hi @nautiam , I tried your use case and it works as expected: When I use the following command: ```shell $ kubectl exec my-cluster-kafka-0 -- bin/kafka-topics.sh --bootstrap-server :9092 --topic my-topic...

@nautiam you don't necessarily need to write code in order to delete a topic. You can simply use the `kafka-topics.sh`, which is included in the official Apache Kafka distribution.

@chaehni I can confirm that. I can reproduce the issue you describe consistently when running a load test which creates a bunch of test topics (e.g. 20). After a successful...

The UO just closes its watch, but [the TO waits some time for inflight reconciliations](https://github.com/strimzi/strimzi-kafka-operator/blob/main/topic-operator/src/main/java/io/strimzi/operator/topic/Session.java#L148). The problem is that this logic is never executed in case of clean shutdown. We...

> Well, you seem to be adding it to all operators but you say that only the Topic Operator. needs it. That seems weird. What's the point of putting any...

I think the `terminationGracePeriodSeconds` is 30 by default, which gives some chance to complete an inflight reconciliation. Of course the user can increase or decrease it as needed.