Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[15.0][UPD] Forward port changes from 14.0 #730

Open
wants to merge 14 commits into
base: 15.0
Choose a base branch
from
Open

[15.0][UPD] Forward port changes from 14.0 #730

wants to merge 14 commits into from

Conversation

florentx
Copy link
Contributor

@florentx florentx commented Dec 27, 2024 •
edited
Loading

@florentx florentx changed the base branch from 16.0 to 15.0 December 27, 2024 10:09
sbidoul and others added 3 commits December 27, 2024 13:16
@sbidoul @florentx
Update test with proper way to pass company to job
b239879
@lmignon @florentx
[IMP] queue_job_cron: Avoid parallel run
8dcd8b1 
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
@InigoGR @florentx
queue_job: fix retry format with tuple values
2c71391 
Configuration of randomized retry intervals is not possible due to the formatting checks not being updated.
This should fix it.
petrus-v and others added 9 commits December 27, 2024 14:02
@petrus-v @florentx
[FIX] queue_job_cron_jobrunner: use priority to select job
61f1376 
* 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
@paradoxxxzero @florentx
[IMP] queue_job: Display warning before displaying big dependency graphs
2015e7a
@paradoxxxzero @florentx
[IMP] queue_job: Add split method
33bfb87
@QuocDuong1306 @florentx
[IMP] queue_job: Cancel child jobs when the parent is cancelled
c666807
@simahawk @florentx
queue_job: unify no delay option
d449652 
Current situation:

* multiple keys for no good reason
* half baked: not all of them used everywhere
* no centralization
* poor naming

With this change we'll have:

* 1 and only one key to disable via ctx: ``queue_job__no_delay``
* 1 and only one key to disable via os env: ``QUEUE_JOB__NO_DELAY``
* backward compatibility with deprecation for old keys
@florentx
[FIX] queue_job: typo
01eed82
@florentx
[IMP] queue_job: add filter on Date Created
e6d5e01
@florentx
[REF] remove explicit 'object' inheritance
c376ef7
@florentx
[REF] remove explicit super() arguments
88df44d
@LoisRForgeFlow @florentx
[13.0][FIX] queue_job_cron: channel_id must be storable.
c223425 
Otherwise, you cannot use any channel other than default (
root.ir_cron)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@florentx @sebalix @sbidoul @lmignon @InigoGR @petrus-v @paradoxxxzero @QuocDuong1306 @simahawk @LoisRForgeFlow