Skip to content

Commit d052604

Browse files
authored
Merge pull request #1801 from passagemath/ci-linux--disambiguate-logs
ci-linux.yml: Disambiguate log artifact names for 'optional', 'experimental' jobs
2 parents 6d72119 + 1a560b3 commit d052604

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

.github/workflows/ci-linux.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,7 @@ jobs:
101101
tox_packages_factors: >-
102102
["standard"]
103103
docker_targets: "with-targets-optional"
104+
logs_artifact_suffix: "-optional"
104105
extra_sage_packages: gdbm libxml2 perl_cpan_polymake_prereq perl_mongodb graphviz
105106
# We remove packages starting with _, in particular package _develop
106107
targets_optional: '$(echo $(export PATH=build/bin:$PATH && sage-package list :optional: --has-file "spkg-install.in|spkg-install|requirements.txt" --no-file "huge|has_nonfree_dependencies" | grep -v sagemath_doc | grep -v ^_))'
@@ -118,6 +119,7 @@ jobs:
118119
tox_packages_factors: >-
119120
["standard"]
120121
docker_targets: "with-targets-optional"
122+
logs_artifact_suffix: "-experimental"
121123
extra_sage_packages: gdbm libxml2 perl_cpan_polymake_prereq perl_mongodb graphviz
122124
targets_optional: '$(echo $(export PATH=build/bin:$PATH && sage-package list :experimental: --has-file "spkg-install.in|spkg-install|requirements.txt" --no-file "huge|has_nonfree_dependencies" | grep -v sagemath_doc))'
123125
max_parallel: 8

.github/workflows/docker.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,9 @@ on:
9797
logs_artifact:
9898
default: true
9999
type: boolean
100+
logs_artifact_suffix:
101+
required: false
102+
type: string
100103
merge_ci_fixes:
101104
default: false
102105
type: boolean
@@ -155,7 +158,7 @@ jobs:
155158
tox_packages_factor: ${{ fromJson(inputs.tox_packages_factors) }}
156159
env:
157160
TOX_ENV: "docker-${{ matrix.tox_system_factor }}-${{ matrix.tox_packages_factor }}${{ inputs.incremental && '-incremental' || '' }}"
158-
LOGS_ARTIFACT_NAME: logs-commit-${{ github.sha }}-tox-docker-${{ matrix.tox_system_factor }}-${{ matrix.tox_packages_factor }}
161+
LOGS_ARTIFACT_NAME: logs-commit-${{ github.sha }}-tox-docker-${{ matrix.tox_system_factor }}-${{ matrix.tox_packages_factor }}${{ inputs.logs_artifact_suffix }}
159162
DOCKER_TARGETS: ${{ inputs.docker_targets }}
160163
TARGETS_PRE: ${{ inputs.targets_pre }}
161164
TARGETS: ${{ inputs.targets }}

0 commit comments

Comments
 (0)