Skip to content

Set OMP_NUM_THREADS=1 for the Konflux image build - #597

Closed
TamiTakamiya wants to merge 1 commit into
mainfrom
TamiTakamiya/OMP_NUM_THREADS
Closed

TamiTakamiya wants to merge 1 commit into
mainfrom
TamiTakamiya/OMP_NUM_THREADS

Conversation

@TamiTakamiya

Copy link
Copy Markdown
Contributor

Summary

  • Set OMP_NUM_THREADS=1 in the builder stage of Containerfile-aap before custom_processor_aap.py runs.
  • This is an experiment to see whether limiting OpenMP to a single thread helps the Konflux pull-request image build (embedding generation).

Test plan

  • Confirm the Konflux PR pipeline starts and the build-container step exports OMP_NUM_THREADS=1
  • Confirm vector DB generation completes within the Konflux task timeout
  • Compare build duration and memory use against a recent PR build without this env var

Made with Cursor

@codecov-commenter

codecov-commenter commented Oct 27, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.98%. Comparing base (a384d0a) to head (68a166f).
⚠️ Report is 4 commits behind head on main.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #597   +/-   ##
=======================================
  Coverage   86.98%   86.98%           
=======================================
  Files          12       12           
  Lines        1222     1222           
=======================================
  Hits         1063     1063           
  Misses        159      159           
Flag Coverage Δ
python 86.98% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

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

…ys single-threaded.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants