Skip to content

Refactor the CI configuration to use inclusion instead of exclusion #3616

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
boegel merged 4 commits into from
Mar 31, 2021
Merged

Refactor the CI configuration to use inclusion instead of exclusion #3616

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
824f37d
Refactor the CI script to use inclusion instead of exclusion
Flamefire Mar 18, 2021
af0b3b6
bump to latest Lmod in test suite configurations
boegel Mar 18, 2021
1b23831
Avoid repetition of module tool versions
Flamefire Mar 19, 2021
8999ed1
Add/Improve comments
Flamefire Mar 22, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
65 changes: 28 additions & 37 deletions .github/workflows/unit_tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,55 +6,46 @@ jobs:
runs-on: ubuntu-18.04
strategy:
matrix:
python: [2.7, 3.5, 3.6, 3.7, 3.8, 3.9]
python: [2.7, 3.6]
modules_tool: [Lmod-7.8.22, Lmod-8.2.9, modules-tcl-1.147, modules-3.2.10, modules-4.1.4]
module_syntax: [Lua, Tcl]
lc_all: [""]
# exclude some configuration for non-Lmod modules tool:
# - don't test with Lua module syntax (only supported in Lmod)
# - exclude Python 3.x versions other than 3.6, to limit test configurations
# don't test with Lua module syntax (only supported in Lmod)
exclude:
- modules_tool: modules-tcl-1.147
module_syntax: Lua
- modules_tool: modules-3.2.10
module_syntax: Lua
- modules_tool: modules-4.1.4
module_syntax: Lua
- modules_tool: modules-tcl-1.147
python: 3.5
- modules_tool: modules-tcl-1.147
python: 3.7
- modules_tool: modules-tcl-1.147
python: 3.8
- modules_tool: modules-tcl-1.147
python: 3.9
- modules_tool: modules-3.2.10
python: 3.5
- modules_tool: modules-3.2.10
python: 3.7
- modules_tool: modules-3.2.10
python: 3.8
- modules_tool: modules-3.2.10
python: 3.9
- modules_tool: modules-4.1.4
python: 3.5
- modules_tool: modules-4.1.4
python: 3.7
- modules_tool: modules-4.1.4
python: 3.8
- modules_tool: modules-4.1.4
python: 3.9
- modules_tool: Lmod-7.8.22
python: 3.5
- modules_tool: Lmod-7.8.22
python: 3.7
- modules_tool: Lmod-7.8.22
python: 3.8
- modules_tool: Lmod-7.8.22
python: 3.9
include:
# Test different python versions with latest Lmod
- python: 3.5
modules_tool: Lmod-8.2.9
module_syntax: Lua
- python: 3.5
modules_tool: Lmod-8.2.9
module_syntax: Tcl
- python: 3.7
modules_tool: Lmod-8.2.9
module_syntax: Lua
- python: 3.7
modules_tool: Lmod-8.2.9
module_syntax: Tcl
- python: 3.8
modules_tool: Lmod-8.2.9
module_syntax: Lua
- python: 3.8
modules_tool: Lmod-8.2.9
module_syntax: Tcl
- python: 3.9
modules_tool: Lmod-8.2.9
module_syntax: Lua
- python: 3.9
modules_tool: Lmod-8.2.9
module_syntax: Tcl
# There may be encoding errors in Python 3 which are hidden when an UTF-8 encoding is set
# Hence run the tests (again) with LC_ALL=C and Python 3.6 (or any < 3.7)
include:
- python: 3.6
modules_tool: Lmod-8.2.9
module_syntax: Lua
Expand Down