Skip to content

Syncing from upstream OCA/queue (18.0)#515

Merged
bt-admin merged 9 commits intobrain-tec:18.0from
OCA:18.0
Jan 4, 2026
Merged

Syncing from upstream OCA/queue (18.0)#515
bt-admin merged 9 commits intobrain-tec:18.0from
OCA:18.0

Conversation

@bt-admin
Copy link

@bt-admin bt-admin commented Jan 4, 2026

bt_gitbot

sbidoul and others added 9 commits January 2, 2026 19:36
Signed-off-by sbidoul
In this commit we cleanly separate the job acquisition (i.e. verifying the job is in the exepected state, marking it started and locking it) from job execution.

We also avoid trying to start the job if it is already locked by using SKIP LOCKED
and exiting early. Indeed in such situations the job is likely already being handled by another worker so there is no point trying to start it, so we exit early
and let it be handled either by the other worker or the dead job requeuer.
Extract the logic to run one job out of the /queue_job/runjob route.
Towards making this logic reusable in other job executors.
Signed-off-by sbidoul
@bt-admin bt-admin added the 18.0 label Jan 4, 2026
@bt-admin bt-admin merged commit 5ab7287 into brain-tec:18.0 Jan 4, 2026
4 checks passed
@codecov
Copy link

codecov bot commented Jan 4, 2026

Codecov Report

❌ Patch coverage is 79.36508% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.63%. Comparing base (2b01d85) to head (48f5707).
⚠️ Report is 47 commits behind head on 18.0.

Files with missing lines Patch % Lines
queue_job/controllers/main.py 59.37% 12 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             18.0     #515      +/-   ##
==========================================
+ Coverage   82.37%   82.63%   +0.26%     
==========================================
  Files          83       84       +1     
  Lines        4699     4735      +36     
  Branches      409      410       +1     
==========================================
+ Hits         3871     3913      +42     
+ Misses        758      752       -6     
  Partials       70       70              

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants