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.

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

[IMP] Allow to specify number of workers through environment variable #64

Closed
wants to merge 1 commit into from
Closed

[IMP] Allow to specify number of workers through environment variable #64

wants to merge 1 commit into from

Conversation

zamberjo
Copy link
Member

We are running a test that involves the generation of several reports.
The process would look something like:

    report_pdf = (
        self.env.ref("module.name_report")
        .with_context(force_report_rendering=True)
        ._render_qweb_pdf([obj.id])
    )

We set the force_report_rendering flag to not render in HTML since we want to check the PDF result but this process does not finish since the main process is blocked by the same execution.

Increasing the workers this does not happen. By default I have left it at 0 which is the default.

@sbidoul
Copy link
Member

sbidoul commented Jan 29, 2024

@zamberjo have you tried running your test in a HttpCase instead of TransactionCase?

@zamberjo
Copy link
Member Author

zamberjo commented Jan 31, 2024

Hi @sbidoul, I just tested it and using HttpCase it is no longer blocked, thank you. You can close the PR if you see that it does not contribute anything, even if it is for another case.

@sbidoul
Copy link
Member

sbidoul commented Jan 31, 2024

Thanks for the feedback!

@sbidoul sbidoul closed this Jan 31, 2024
@zamberjo zamberjo deleted the imp-workers-run-tests branch January 31, 2024 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants