Skip to content

Commit

Permalink
CI: Adjust job parallelism to actually match available resources
Browse files Browse the repository at this point in the history
- Some jobs don't actually run in parallel but have the `parallelism` setting enabled.
- `soltest_all` could be parallelized much more.
  • Loading branch information
cameel committed Dec 9, 2021
1 parent b14a674 commit af02678
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
4 changes: 1 addition & 3 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -695,7 +695,6 @@ jobs:

t_ubu_codecov:
<<: *base_ubuntu2004
parallelism: 6
environment:
EVM: << pipeline.parameters.evm-version >>
OPTIMIZE: 1
Expand Down Expand Up @@ -878,7 +877,7 @@ jobs:

t_ubu_soltest_all: &t_ubu_soltest_all
<<: *base_ubuntu2004
parallelism: 6
parallelism: 15 # 7 EVM versions, each with/without optimization + 1 ABIv1/@nooptions run
<<: *steps_soltest_all

t_archlinux_soltest: &t_archlinux_soltest
Expand Down Expand Up @@ -937,7 +936,6 @@ jobs:

t_ubu_asan_soltest:
<<: *base_ubuntu2004
parallelism: 6
environment:
EVM: << pipeline.parameters.evm-version >>
OPTIMIZE: 0
Expand Down
1 change: 1 addition & 0 deletions .circleci/soltest_all.sh
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ REPODIR="$(realpath "$(dirname "$0")"/..)"
# shellcheck source=scripts/common.sh
source "${REPODIR}/scripts/common.sh"

# NOTE: If you add/remove values, remember to update `parallelism` setting in CircleCI config.
EVM_VALUES=(homestead byzantium constantinople petersburg istanbul berlin london)
DEFAULT_EVM=london
[[ " ${EVM_VALUES[*]} " =~ $DEFAULT_EVM ]]
Expand Down

0 comments on commit af02678

Please sign in to comment.