Skip to content

Syncing from upstream OCA/queue (17.0)#480

Merged
bt-admin merged 20 commits intobrain-tec:17.0from
OCA:17.0
Jun 7, 2025
Merged

Syncing from upstream OCA/queue (17.0)#480
bt-admin merged 20 commits intobrain-tec:17.0from
OCA:17.0

Conversation

@bt-admin
Copy link
Copy Markdown

@bt-admin bt-admin commented Jun 7, 2025

bt_gitbot

sbidoul and others added 20 commits May 8, 2025 12:47
By default, odoo never runs the same cron job in parallel. This commit uses the identity key mechanism to enforce this mechanism when a cron job is run as a queue job. This behaviour can be controlled by a new setting on the cron definition but is activated by default to keep the original behaviour
Configuration of randomized retry intervals is not possible due to the formatting checks not being updated.
This should fix it.
So the time required to compute such fields by the ORM is taken into
account when the 'date_done' and 'exec_time' values are set on the job.
Starting from 16.0, we should call `env.flush_all()` instead of
`env["base"].flush()`, like it is done few lines below.
* use FIFO, firt createad job will be treat first
* if priority are different it take the precedent

Yet we are not using channel priority into account
Otherwise, you cannot use any channel other than default (
root.ir_cron)
Signed-off-by hbrunn
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: queue-17.0/queue-17.0-queue_job
Translate-URL: https://translation.odoo-community.org/projects/queue-17-0/queue-17-0-queue_job/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: queue-17.0/queue-17.0-queue_job_cron
Translate-URL: https://translation.odoo-community.org/projects/queue-17-0/queue-17-0-queue_job_cron/
@bt-admin bt-admin added the 17.0 label Jun 7, 2025
@bt-admin bt-admin merged commit 1374149 into brain-tec:17.0 Jun 7, 2025
4 checks passed
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 7, 2025

Codecov Report

Attention: Patch coverage is 95.79832% with 10 lines in your changes missing coverage. Please review.

Project coverage is 83.98%. Comparing base (9757a9f) to head (0af9187).
Report is 50 commits behind head on 17.0.

Files with missing lines Patch % Lines
queue_job/tests/test_delayable_split.py 96.05% 2 Missing and 1 partial ⚠️
queue_job_cron/models/ir_cron.py 76.92% 2 Missing and 1 partial ⚠️
queue_job/models/queue_job_function.py 84.61% 1 Missing and 1 partial ⚠️
queue_job/controllers/main.py 0.00% 1 Missing ⚠️
queue_job/delay.py 94.73% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             17.0     #480      +/-   ##
==========================================
+ Coverage   83.13%   83.98%   +0.85%     
==========================================
  Files          69       71       +2     
  Lines        4156     4383     +227     
  Branches      387      397      +10     
==========================================
+ Hits         3455     3681     +226     
+ Misses        623      622       -1     
- Partials       78       80       +2     

☔ 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.