From 762ba27036e717aeceeb1b43f3090f0e9a049869 Mon Sep 17 00:00:00 2001 From: Tom Scogland Date: Wed, 7 Sep 2022 11:12:57 -0700 Subject: Make GHA tests parallel by using xdist (#32361) * Add two no-op jobs named "all-prechecks" and "all" These are a suggestion from @tgamblin, they are stable named markers we can use from gitlab and possibly for required checks to make CI more resilient to refactors changing the names of specific checks. * Enable parallel testing using xdist for unit testing in CI * Normalize tmp paths to deal with macos * add -u flag compatibility to spack python As of now, it is accepted and ignored. The usage with xdist, where it is invoked specifically by `python -u spack python` which is then passed `-u` by xdist is the entire reason for doing this. It should never be used without explicitly passing -u to the executing python interpreter. * use spack python in xdist to support python 2 When running on python2, spack has many import cycles unless started through main. To allow that, this uses `spack python` as the interpreter, leveraging the `-u` support so xdist doesn't error out when it unconditionally requests unbuffered binary IO. * Use shutil.move to account for tmpdir being in a separate filesystem sometimes --- .flake8 | 1 + 1 file changed, 1 insertion(+) (limited to '.flake8') diff --git a/.flake8 b/.flake8 index 226e694894..ddd6b6c2fd 100644 --- a/.flake8 +++ b/.flake8 @@ -29,6 +29,7 @@ max-line-length = 99 # per-file-ignores = var/spack/repos/*/package.py:F403,F405,F821 + *-ci-package.py:F403,F405,F821 # exclude things we usually do not want linting for. # These still get linted when passed explicitly, as when spack flake8 passes -- cgit v1.2.3-60-g2f50