Skip to content

Commit 68dd9d9

Browse files
committed
introduce separate nightly release level, make fsanitize nightly-secondary
1 parent 83f0809 commit 68dd9d9

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

.github/workflows/ci.yml

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,8 @@ jobs:
108108
109109
# 0: PRs without special label
110110
# 1: PRs with `merge-ci` label, merge queue checks, master commits
111-
# 2: PRs with `release-ci` label, releases (incl. nightlies)
111+
# 2: nightlies
112+
# 2: PRs with `release-ci` label, full releases
112113
- name: Set check level
113114
id: set-level
114115
# We do not use github.event.pull_request.labels.*.name here because
@@ -118,14 +119,16 @@ jobs:
118119
check_level=0
119120
fast=false
120121
121-
if [[ -n "${{ steps.set-nightly.outputs.nightly }}" || -n "${{ steps.set-release.outputs.RELEASE_TAG }}" || -n "${{ steps.set-release-custom.outputs.RELEASE_TAG }}" ]]; then
122+
if [[ -n "${{ steps.set-release.outputs.RELEASE_TAG }}" || -n "${{ steps.set-release-custom.outputs.RELEASE_TAG }}" ]]; then
123+
check_level=3
124+
elif [[ -n "${{ steps.set-nightly.outputs.nightly }}" ]]; then
122125
check_level=2
123126
elif [[ "${{ github.event_name }}" != "pull_request" ]]; then
124127
check_level=1
125128
else
126129
labels="$(gh api repos/${{ github.repository_owner }}/${{ github.event.repository.name }}/pulls/${{ github.event.pull_request.number }} --jq '.labels')"
127130
if echo "$labels" | grep -q "release-ci"; then
128-
check_level=2
131+
check_level=3
129132
elif echo "$labels" | grep -q "merge-ci"; then
130133
check_level=1
131134
fi
@@ -214,11 +217,13 @@ jobs:
214217
"name": "Linux fsanitize",
215218
"os": "ubuntu-latest",
216219
"enabled": level >= 2,
220+
# do not have nightly release wait for this
221+
"secondary": level <= 2,
217222
"test": true,
218223
// turn off custom allocator & symbolic functions to make LSAN do its magic
219224
"CMAKE_PRESET": "sanitize",
220225
// exclude seriously slow/problematic tests (laketests crash, async_base_functions timeouts)
221-
"CTEST_OPTIONS": "-E '(interactive|pkg|lake|bench)/|StackOverflow|async_base_functions'"
226+
"CTEST_OPTIONS": "-E '(interactive|pkg|lake|bench)/|treemap|StackOverflow|async_base_functions'"
222227
},
223228
{
224229
"name": "macOS",
@@ -251,7 +256,7 @@ jobs:
251256
},
252257
{
253258
"name": "Windows",
254-
"os": large && (fast || level == 2) ? "namespace-profile-windows-amd64-4x16" : "windows-2022",
259+
"os": large && (fast || level >= 2) ? "namespace-profile-windows-amd64-4x16" : "windows-2022",
255260
"release": true,
256261
"enabled": level >= 2,
257262
"test": true,

0 commit comments

Comments
 (0)