Skip to content

rework free port detection in tests

Dmitry Rodionov requested to merge dkr/rework_port_distribution into master

Summary

  • rework free port detection in tests

The patch changes how ports are assigned to nodes started in tests. Previously port range was given to any xdist worker and each test chose ports from the beginning of the range which can lead to conflicts. So with the patch tests can no longer have interleaving port ranges.

Additionally ports were previously checked but the check was failing the whole test in case of a failure. With the patch occupied ports are skipped instead.

The patch also fixes a source of flakiness in PluginReflection PluginReflection

  • refactor: dedup instance fixture for gostech, avoid flakiness

Make use of single instance fixture for non gostech and gostech instances. Avoid flakiness caused by unsynchronized shutdown of audit log server and messages lookup.

Close: #644 (closed)

  • test: raise wait_online timeout to 10 seconds

Close: #638 (closed)


Edited by Dmitry Rodionov

Merge request reports