queue icon indicating copy to clipboard operation
queue copied to clipboard

Migration to version 19.0

Open OCA-git-bot opened this issue 3 months ago • 8 comments

Todo

https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-19.0

Modules to migrate

  • [ ] base_import_async - By @tishmen - #841
  • [ ] queue_job - By @tishmen - #840
  • [ ] queue_job_batch
  • [ ] queue_job_cron
  • [ ] queue_job_cron_jobrunner
  • [ ] queue_job_subscribe
  • [ ] test_queue_job - By @tishmen - #840
  • [ ] test_queue_job_batch

Missing module? Check https://github.com/OCA/maintainer-tools/wiki/%5BFAQ%5D-Missing-modules-in-migration-issue-list

OCA-git-bot avatar Sep 29 '25 14:09 OCA-git-bot

Hi all - I’ll take care of the 19.0 migration for queue_job (starting there, and time permitting I’ll also look at queue_job_cron and queue_job_batch). I’ll follow the OCA 19.0 migration guidelines and open a first draft PR by the end of this week. If anyone is already working on related submodules, please let me know so I can coordinate.

tishmen avatar Sep 29 '25 21:09 tishmen

Quick update: I’m porting all queue addons to 19.0 (queue_job, queue_job_batch, queue_job_cron, queue_job_cron_jobrunner, queue_job_subscribe, base_import_async). Core migration is done and installs cleanly; I’m now running the test suites and fixing any regressions. I’ll open a PR as soon as the tests are green (most likely tomorrow). If anyone is working in parallel on these modules, please let me know so we can coordinate.

tishmen avatar Sep 30 '25 01:09 tishmen

Any progress? I ran into an MRO issue in the test 😢 — it needs more attention to fix.

mostafabarmshory avatar Oct 29 '25 11:10 mostafabarmshory

@mostafabarmshory can you please post the MRO issue that you have encountered in the appropriate PR. I will take a look as soon as possible.

tishmen avatar Oct 29 '25 12:10 tishmen

@tishmen Actually, when the class OdooDocTestCase is enable, I face with MRO issue. I commit it and all things goes well. By the way, the error raise by running unit tests. See the attachement.

Image

mostafabarmshory avatar Nov 06 '25 09:11 mostafabarmshory

In Odoo19, odoo.tools.config does not have misc anymore.

This has implications for queue_job/jobrunner/__init__.py, which uses config.misc around line:20 or so.

daemonblade avatar Nov 20 '25 20:11 daemonblade

@daemonblade Please comment in the relevant code section on the PR so I can review. Thanks.

tishmen avatar Nov 21 '25 00:11 tishmen

@tishmen From 19.0 branch, queue_job/jobrunner/__init__.py, lines 9-20:

from odoo.service import server
from odoo.tools import config

try:
    from odoo.addons.server_environment import serv_config

    if serv_config.has_section("queue_job"):
        queue_job_config = serv_config["queue_job"]
    else:
        queue_job_config = {}
except ImportError:
    queue_job_config = config.misc.get("queue_job", {})

The last line's config.misc isn't valid anymore.

daemonblade avatar Nov 21 '25 19:11 daemonblade