chia7712 commented on code in PR #23110:
URL: https://github.com/apache/kafka/pull/23110#discussion_r3791699565
##########
tests/kafkatest/utils/util.py:
##########
@@ -57,7 +57,7 @@ def is_version(node, version_list, proc_grep_string="kafka",
logger=None):
"""Heuristic to check that only the specified version appears in the
classpath of the process
A useful tool to aid in checking that service version apis are working
correctly.
"""
- lines = [l for l in node.account.ssh_capture("ps ax | grep %s | grep -v
grep" % proc_grep_string)]
+ lines = [l for l in node.account.ssh_capture("ps axww | grep %s | grep -v
grep" % proc_grep_string)]
Review Comment:
No E2E tests were harmed in the making of this PR :)
```
================================================================================
SESSION REPORT (ALL TESTS)
ducktape version: 0.14.0
session_id: 2026-08-16--001
run time: 1 minute 28.489 seconds
tests run: 21
passed: 21
flaky: 0
failed: 0
ignored: 0
================================================================================
test_id:
kafkatest.sanity_checks.test_verifiable_producer.TestVerifiableProducer.test_simple_run.producer_version=dev.acks=-1.enable_idempotence=False.metadata_quorum=COMBINED_KRAFT
status: PASS
run time: 22.556 seconds
```
##########
bin/kafka-server-stop.sh:
##########
@@ -40,7 +40,7 @@ else
if [ -z "$INPUT_PROCESS_ROLE" ] && [ -z "$INPUT_NID" ]; then
kill -s $SIGNAL $PIDS
else
- RelativePathToConfig=$(ps ax | grep ' kafka\.Kafka ' | grep java | grep -v
grep | sed 's/--override property=[^ ]*//g' | awk 'NF>1{print $NF}' | xargs)
+ RelativePathToConfig=$(ps axww | grep ' kafka\.Kafka ' | grep java | grep
-v grep | sed 's/--override property=[^ ]*//g' | awk 'NF>1{print $NF}' | xargs)
Review Comment:
I have tested the script locally, and it works well. Also, I have opened
https://issues.apache.org/jira/browse/KAFKA-20944 to enhance the test coverage
:)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
