[bugfix] Fix uncontrolled job submission for remote autodetection

[teojgo]

Fixes #2226

[codecov-commenter]

Codecov Report

Merging #2229 (51e0106) into master (07c7182) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 51e0106 differs from pull request most recent head 502327d. Consider uploading reports for the commit 502327d to get more accurate results

@@           Coverage Diff           @@
##           master    #2229   +/-   ##
=======================================
  Coverage   85.54%   85.54%           
=======================================
  Files          55       55           
  Lines        9746     9746           
=======================================
  Hits         8337     8337           
  Misses       1409     1409           

Impacted Files	Coverage Δ
reframe/frontend/cli.py	76.31% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 07c7182...502327d. Read the comment docs.

[vkarak]

I think that the problem is that we auto detect the system too early here:

https://github.com/eth-cscs/reframe/blob/e9ec5b94d62a7f2f8b413c296cc866271ef2546e/reframe/frontend/cli.py#L618

We should instead do that after we process the --detect-host-topology option, just before this line:

https://github.com/eth-cscs/reframe/blob/e9ec5b94d62a7f2f8b413c296cc866271ef2546e/reframe/frontend/cli.py#L677

Can you try that out?

[teojgo]

I think that the problem is that we auto detect the system too early here:

https://github.com/eth-cscs/reframe/blob/e9ec5b94d62a7f2f8b413c296cc866271ef2546e/reframe/frontend/cli.py#L618

We should instead do that after we process the --detect-host-topology option, just before this line:

https://github.com/eth-cscs/reframe/blob/e9ec5b94d62a7f2f8b413c296cc866271ef2546e/reframe/frontend/cli.py#L677

Can you try that out?

Yes this does the trick.