[GitHub] [kafka] rondagostino commented on pull request #11238: MINOR: Fix force kill of KRaft colocated controllers in system tests

[GitBox <gi...@apache.org>]

rondagostino commented on pull request #11238:
URL: https://github.com/apache/kafka/pull/11238#issuecomment-904824877


   > we can have ceil method to use, which will be easier 
   
   Good suggestion!  Done.  Here's some system tests runs that show the new code is syntactically correct (and I observed the correct kill behavior).
   
   ```
   test_id:    kafkatest.sanity_checks.test_bounce.TestBounce.test_simple_run.metadata_quorum=COLOCATED_KRAFT.quorum_size=1
   status:     PASS
   run time:   1 minute 49.278 seconds
   --------------------------------------------------------------------------------
   test_id:    kafkatest.sanity_checks.test_bounce.TestBounce.test_simple_run.metadata_quorum=COLOCATED_KRAFT.quorum_size=3
   status:     PASS
   run time:   2 minutes 6.602 seconds
   ```
   
   Thank for the review!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org