qa: Remove portseed_offset from test runner #13421

1. 
   MarcoFalke commented at 5:33 PM on June 8, 2018: member

   The portseed_offset is no longer needed in the test runner, since we already kill leftover processes (see #12904). This "fixes" #10869 because we deterministically pick ports starting at 11000

2. qa: Remove portseed_offset from test runner fa6edfef35
3. MarcoFalke requested review from jnewbery on Jun 8, 2018
4. fanquake added the label Tests on Jun 9, 2018
5. 
   laanwj commented at 12:35 PM on June 11, 2018: member

   utACK fa6edfef358518022ee86c0abc77c1c068f106a3 I think making the port range more deterministic is, overall, a good thing. We don't support multiple test_runner instances at the same time, and if we did, it would be better as an explicit option to change the port offset.

6. laanwj merged this on Jun 11, 2018
7. laanwj closed this on Jun 11, 2018

   ---

8. laanwj referenced this in commit 531a0337ca on Jun 11, 2018
9. MarcoFalke deleted the branch on Jun 11, 2018
10. 
    laanwj commented at 1:42 PM on June 11, 2018: member

    Something that might be useful is to do an initial sweep over all the ports that are going to be used, so if something is in the way, there's an early warning/error.

11. UdjinM6 referenced this in commit 8d22516df1 on Jun 19, 2021
12. UdjinM6 referenced this in commit ad11ad8ec5 on Jun 24, 2021
13. UdjinM6 referenced this in commit 4fb22e4c31 on Jun 26, 2021
14. UdjinM6 referenced this in commit b02642d53f on Jun 26, 2021
15. UdjinM6 referenced this in commit 69fd5dee3d on Jun 26, 2021
16. UdjinM6 referenced this in commit 05dd5c7079 on Jun 28, 2021
17. Fabcien referenced this in commit 3ba5cf0a5e on Jul 8, 2021
18. MarcoFalke locked this on Sep 8, 2021

Review Requested
jnewbery