stan-confluent commented on a change in pull request #10915:
URL: https://github.com/apache/kafka/pull/10915#discussion_r657636156
##########
File path: tests/README.md
##########
@@ -51,6 +51,40 @@ bash tests/docker/ducker-ak up -j 'openjdk:11';
tests/docker/run_tests.sh
```
REBUILD="t" bash tests/docker/run_tests.sh
```
+* Debug tests in VS Code:
+ - Run test with `--debug` flag (can be before or after file name):
Review comment:
We can still keep this - I do need to update run_tests.sh to pass
_DUCKTAPE_OPTIONS after the `--`.
My idea was that I didn't want to make sure the flags don't overlap between
ducktape and ducker-ak, hence I changed ducker-ak to expect ducktape args after
the `--` - like this `ducker-ak test my_test.py -- --ducktape-flag`.
I am, however, open to simply using a different flag name - it is unlikely
we'll add too many flags to the ducker-ak run command anyway, so we can simply
make sure they don't match the ducktape ones.
@omkreddy and @ewencp (with whom we had similar conversation in a different
repo) - which one do you prefer?
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
