Skip to content

[8.2] [MOD-11799] ci: add back MAX_WORKER_THREADS option to Makefile#7054

Merged
JoanFM merged 2 commits into8.2from
backport-7039-to-8.2
Oct 14, 2025
Merged

[8.2] [MOD-11799] ci: add back MAX_WORKER_THREADS option to Makefile#7054
JoanFM merged 2 commits into8.2from
backport-7039-to-8.2

Conversation

@redisearch-backport-pull-request
Copy link
Contributor

@redisearch-backport-pull-request redisearch-backport-pull-request bot commented Oct 14, 2025

Description

Backport of #7039 to 8.2.


Note

Adds MAX_WORKER_THREADS option to the Makefile, forwarding it to build.sh via BUILD_ARGS.

  • Build System:
    • Makefile: Support MAX_WORKER_THREADS by appending MAX_WORKER_THREADS=$(MAX_WORKER_THREADS) to BUILD_ARGS when set, passing through to build.sh.

Written by Cursor Bugbot for commit 6d4ecf7. This will update automatically on new commits. Configure here.

* ci: add back MAX_WORKER_THREADS option to Makefile

* ci: test in CMakeCache.txt

* revert changes on actions

(cherry picked from commit b4ef348)
@codecov
Copy link

codecov bot commented Oct 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.40%. Comparing base (f9caadf) to head (6d4ecf7).
⚠️ Report is 2 commits behind head on 8.2.

Additional details and impacted files
@@            Coverage Diff             @@
##              8.2    #7054      +/-   ##
==========================================
- Coverage   89.43%   89.40%   -0.04%     
==========================================
  Files         253      253              
  Lines       41665    41665              
  Branches     3725     3725              
==========================================
- Hits        37264    37249      -15     
- Misses       4352     4367      +15     
  Partials       49       49              
Flag Coverage Δ
flow 82.00% <ø> (-0.16%) ⬇️
unit 47.58% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JoanFM JoanFM requested a review from alonre24 October 14, 2025 09:05
@redisearch-backport-pull-request redisearch-backport-pull-request bot added this pull request to the merge queue Oct 14, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 14, 2025
@JoanFM JoanFM added this pull request to the merge queue Oct 14, 2025
@JoanFM JoanFM removed this pull request from the merge queue due to a manual request Oct 14, 2025
@JoanFM JoanFM added this pull request to the merge queue Oct 14, 2025
Merged via the queue into 8.2 with commit 4c92797 Oct 14, 2025
18 checks passed
@JoanFM JoanFM deleted the backport-7039-to-8.2 branch October 14, 2025 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant