Fork Sync Branch 19.0 - #10
Merged
Merged
Conversation
…n to avoid errors closes #315 Signed-off-by: rov-adhoc <rov@adhoc.com.ar>
closes #316 Signed-off-by: rov-adhoc <rov@adhoc.com.ar>
Updated by "Update PO files to match POT (msgmerge)" add-on in Weblate. Translation: miscellaneous-19.0/miscellaneous-19.0-base_bg Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-base_bg/
Currently translated at 90.3% (28 of 31 strings) Translation: miscellaneous-19.0/miscellaneous-19.0-maintenance_ux Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-maintenance_ux/es/
Currently translated at 100.0% (25 of 25 strings) Translation: miscellaneous-19.0/miscellaneous-19.0-portal_backend Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-portal_backend/es/
Currently translated at 67.6% (44 of 65 strings) Translation: miscellaneous-19.0/miscellaneous-19.0-base_bg Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-base_bg/es/
Currently translated at 100.0% (36 of 36 strings) Translation: miscellaneous-19.0/miscellaneous-19.0-base_dynamic_message Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-base_dynamic_message/es/
Currently translated at 100.0% (3 of 3 strings) Translation: miscellaneous-19.0/miscellaneous-19.0-mail_log_only_internal Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-mail_log_only_internal/es/
Currently translated at 100.0% (16 of 16 strings) Translation: miscellaneous-19.0/miscellaneous-19.0-base_ux Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-base_ux/es/
Currently translated at 100.0% (14 of 14 strings) Translation: miscellaneous-19.0/miscellaneous-19.0-portal_holidays Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-portal_holidays/es/
Currently translated at 85.7% (6 of 7 strings) Translation: miscellaneous-19.0/miscellaneous-19.0-mail_activity_automation Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-mail_activity_automation/es/
Currently translated at 71.4% (5 of 7 strings) Translation: miscellaneous-19.0/miscellaneous-19.0-mail_ux Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-mail_ux/es/
…mize cron job retrieval order closes #321 Signed-off-by: Nicolas Mac Rouillon (ADV) <nmr@adhoc.inc>
closes #323 Signed-off-by: Alexis Lopez <loa@adhoc.inc>
closes #301 Signed-off-by: Felipe Garcia Suez <feg@adhoc.com.ar> Signed-off-by: rov-adhoc <rov@adhoc.com.ar>
stdout: Auto-merging account_statement_import_sheet_file_bg/models/account_statement_import.py CONFLICT (content): Merge conflict in account_statement_import_sheet_file_bg/models/account_statement_import.py stderr: closes #325 Signed-off-by: Filoquin adhoc <maq@adhoc.com.ar> Signed-off-by: rov-adhoc <rov@adhoc.com.ar>
Updated by "Update PO files to match POT (msgmerge)" add-on in Weblate. Translation: miscellaneous-19.0/miscellaneous-19.0-account_statement_import_sheet_file_bg Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-account_statement_import_sheet_file_bg/
…ue method to include job reference and adapt AccountStatementImport enqueue usage
…e timeout notification closes #329 Signed-off-by: Nicolas Mac Rouillon (ADV) <nmr@adhoc.inc>
closes #328 Signed-off-by: matiasperalta1 <mnp@adhoc.com.ar>
closes #330 Signed-off-by: matiasperalta1 <mnp@adhoc.com.ar>
The quick badges customization was only applied to the mail.activity.schedule wizard, used when scheduling a new activity. Editing an existing activity opens an act_window on mail.activity instead (view mail_activity_view_form_popup), so that dialog kept rendering a badge for every activity type in the system. Inherit that view with the same treatment and share the lookup of the suggested types through mail.activity.type._get_quick_activity_types(). On mail.activity the computed field also includes the type currently set on the record: without it, editing an activity whose type is not among the suggested ones would not render the stored value at all. closes #428 Signed-off-by: Luciano Esperlazza <les@adhoc.inc>
Updated by "Update PO files to match POT (msgmerge)" add-on in Weblate. Translation: miscellaneous-19.0/miscellaneous-19.0-base_ux Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-base_ux/
… and less cron churn Serialization contention raised while *running* a job (SQLSTATE 40001, e.g. row conflicts on discuss_channel_member from message posting) was retried instantly and, once max_retries was hit, turned healthy jobs into permanent failures, while finish() triggered every cron on each completed job, adding ir_cron churn. Acquisition-time serialization is already retried in _cron_run_enqueued_jobs; this covers the execution side. - Serialization failures raised while running a job are retried with exponential backoff + jitter and their own budget (base_bg.serialization_max_retries, default 10), so a transient spike no longer fails healthy jobs. - The job picker skips jobs still inside their backoff window (next_retry_at), and the runner re-trigger only counts eligible jobs. - finish() no longer triggers the cron on every job; it relies on the runner's own re-trigger while eligible jobs remain. - Optional global cap of concurrently running jobs (base_bg.max_concurrent_jobs, disabled by default) as an emergency brake with no deploy. Part-of: #430 Signed-off-by: Nicolas Mac Rouillon (ADV) <nmr@adhoc.inc>
- Transient (40001) retries use a dedicated serialization_retry_count so a contention spike no longer eats the normal max_retries budget; retry_count stays the total attempt count. - _handle_job_error returns whether it re-enqueued (for overrides). - Robust transient detection: walk the __cause__/__context__ chain for SerializationFailure/40001; drop the loose substring match. - Safe integer parsing of base_bg.* system parameters; a malformed value no longer crashes the runner cron on every tick. - Eligibility gate and start_time use a Python UTC timestamp bind param instead of SQL NOW() (session-tz safe; also fixes duration on non-UTC sessions). - Concurrency cap ignores jobs stuck 'running' past the cron timeout (orphans) so they cannot saturate it; documented as best-effort, not a hard ceiling. - Backoff schedules a cron wake-up at next_retry_at so a lone job does not wait for the periodic tick. - fail()/cancel() clear next_retry_at; run() documents at-least-once semantics. - Tests updated/added (mixed-error budget, malformed param, orphan cap, gate clearing, batch continuation); param mutations restored via addCleanup. Part-of: #430 Signed-off-by: Nicolas Mac Rouillon (ADV) <nmr@adhoc.inc>
…concerns) - Reuse Odoo's PG_CONCURRENCY_EXCEPTIONS_TO_RETRY (serialization / deadlock / lock-not-available) instead of a hand-rolled 40001 check; broadens transient coverage to deadlocks and lock timeouts, which are equally retryable. - Rename serialization_retry_count -> transient_retry_count and the parameter to base_bg.transient_max_retries to match the generic "transient" vocabulary. - Move the concurrency throttle out of the _get_next_job locking SQL into a separate _can_acquire_job admission check called before acquisition; the picker is now a pure locker with no f-string SQL assembly. - Extract _get_cron_timeout, shared by _can_acquire_job and _cron_check_running_jobs. - Extract _give_up (fail + cancel + return False) shared by both permanent-failure paths; name non_transient_attempts; simplify the backoff exponent clamp. Part-of: #430 Signed-off-by: Nicolas Mac Rouillon (ADV) <nmr@adhoc.inc>
Part-of: #430 Signed-off-by: Nicolas Mac Rouillon (ADV) <nmr@adhoc.inc>
The backoff + separate transient budget + reduced cron churn address the contention directly. The global cap was an optional emergency valve that also throttled unrelated jobs; drop it for now to keep the change focused. Can be reintroduced (ideally per-model) if a future storm needs it. Part-of: #430 Signed-off-by: Nicolas Mac Rouillon (ADV) <nmr@adhoc.inc>
Addresses part B of ticket 124538: the Background Job Runner loops many jobs in one long-lived cron process, and run() never released each job's ORM cache, so memory accumulated between jobs and pushed already-tight pods toward OOM. Call env.invalidate_all() after the successful commit and after the rollback in the error path, flattening the runner's memory curve between jobs. closes #430 Signed-off-by: Nicolas Mac Rouillon (ADV) <nmr@adhoc.inc>
Updated by "Update PO files to match POT (msgmerge)" add-on in Weblate. Translation: miscellaneous-19.0/miscellaneous-19.0-base_bg Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-base_bg/
…company
_check_company_dependent_m2o() built a company domain for every
company_dependent Many2one, including the ones whose comodel is shared by
all companies (res.currency, uom.uom, ...) and has no company_id nor
company_ids field. On those models the domain is invalid, so any load() of
a record holding such a value raised "Invalid field res.currency.company_id
in condition ('company_id', 'in', [1, False])" and aborted the whole import.
Concretely, importing anything on res.partner (tags, for instance) failed
for every vendor with a supplier currency set. Module CSV data is loaded
through load() too, so it was exposed as well.
Skip those fields: with no company on the comodel there is nothing to
cross-check.
closes #432
Signed-off-by: Franco Leyes <lef@adhoc.com.ar>
…yee_overtime on hr.leave
hr_holidays_attendance defines employee_overtime on both hr.leave and
hr.leave.allocation restricted to base.group_user. This module already
extends the field groups on hr.leave.allocation so its portal backend
holiday group can read it, but never did the same on hr.leave — and the
leave form it exposes to that group contains an element that depends on
the field.
Since Odoo 19, view validation flags this as an access rights
inconsistency on every update:
This view may not work for all users: some users may have a
combination of groups where the elements <div> are displayed, but
they depend on the field employee_overtime that is not accessible.
Extend the field groups on hr.leave exactly like the allocation one.
Updated by "Update PO files to match POT (msgmerge)" add-on in Weblate. Translation: miscellaneous-19.0/miscellaneous-19.0-portal_holidays Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-portal_holidays/
closes #420 Signed-off-by: Franco Leyes <lef@adhoc.com.ar>
Failing a job whose target records were deleted raised MissingError: fail() browsed those ids to link them in the notification, and browsing a dropped id is truthy. That aborted the whole _cron_check_running_jobs loop, so the poisoned job stayed running forever and none of the other stuck jobs were ever handled. Check real existence before linking the records, and handle each job -- and the recovery path itself -- in its own savepoint, so an override that raises no longer takes the reaper down with it. closes #437 Signed-off-by: Nicolas Mac Rouillon (ADV) <nmr@adhoc.inc>
Override base_import.import._parse_float_from_data and _parse_date_from_data so a single badly-formatted number or date no longer raises ImportValidationError and cancels the whole import. The raw value is deferred to the per-record ORM converter, which reports it with its row number alongside every other field/record error in a single pass. Well-formed values are still pre-cleaned (currency symbols, thousands/decimal separators) as before; genuinely unexpected (non-ValueError) date failures are still raised. closes #429 Signed-off-by: Juan Carreras <jc@adhoc.com.ar>
Updated by "Update PO files to match POT (msgmerge)" add-on in Weblate. Translation: miscellaneous-19.0/miscellaneous-19.0-base_import_ux Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-base_import_ux/
A job can outlive its records: a GC or an FK cascade deletes them while the job is still enqueued, waiting or running. Such a job did not fail — there is nothing left to run it on — but it ended up failed (timeout or retries exhausted) with its batch canceled as "Previous job in batch failed", polluting failure metrics and notifying about nothing actionable. The runner and the reaper now detect a job whose records are all gone and cancel it (and the rest of its batch) instead, without notifying. Jobs that point to no records at all (model-level methods) and jobs with surviving records are untouched. closes #438 Signed-off-by: Nicolas Mac Rouillon (ADV) <nmr@adhoc.inc>
Updated by "Update PO files to match POT (msgmerge)" add-on in Weblate. Translation: miscellaneous-19.0/miscellaneous-19.0-base_bg Translate-URL: https://translation.dev-adhoc.com/projects/miscellaneous-19-0/miscellaneous-19-0-base_bg/
sireko
approved these changes
Aug 26, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.