[Tooling] Adjust run_test_suite.py to no t use SIGKILL on the test device about connectedhomeip HOT 3 CLOSED

The Python used is process.kill(). We could process.terminate() but there is no guarantee that this will actually kill the process, and some of the bits here absolutely need to make sure the process dies (e.g. so it releases the port).

If you have specific suggestions on how to improve this while preserving the existing invariants, a PR is probably best....

from connectedhomeip.

I'm unfortunately not a python developer. I created the issue on request of @cecille. But yes I it would be the only way I could try to add this as a PR. I might do that if noone with more python experience jumps on it :-)

I just read that process.kill("SIGINT") could be used to send a SIGINT, but yes that would require maybe to check delayed afterwards if the process was really stopped and maybe send a kill 1-2s afterwards again or such ...

A proper teardown including sending MDNS expires and such is also very important because else the test see wrong MDNS devices and react strange or error.

from connectedhomeip.

@bzbarsky-apple @cecille Ok, I tried my best and came up with that after digging a bit into python ... Pleas ebe nice to my ... my first python changes ;-))

#31650

I tested it locally and it works from my perspective

from connectedhomeip.