Skip to content

[13.0][ADD] queue_job: innocuous test job for debugging/monitoring purposes - #265

Merged
OCA-git-bot merged 1 commit into
OCA:13.0from
nilshamerlinck:13.0_add_test_job
Oct 27, 2020
Merged

[13.0][ADD] queue_job: innocuous test job for debugging/monitoring purposes#265
OCA-git-bot merged 1 commit into
OCA:13.0from
nilshamerlinck:13.0_add_test_job

Conversation

@nilshamerlinck

Copy link
Copy Markdown
Contributor

The goal of this PR is to introduce an easy and innocuous way to confirm that jobs are running fine in a production environment, by triggering the creation of a test job through an http route.

Of course, only accessible to admin users to avoid any abuse :)

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @guewen,
some modules you are maintaining are being modified, check this out!

@nilshamerlincknilshamerlinck changed the title [ADD] queue_job: innocuous test job for debugging purposes[13.0][ADD] queue_job: innocuous test job for debugging/monitoring purposesOct 22, 2020

@guewenguewen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey that's great! I wanted to add a kind of way to create test jobs easily, and didn't think about using a simple controller endpoint. ❤️

Comment threadqueue_job/models/queue_job.py Outdated

@sbidoulsbidoul left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment threadqueue_job/jobrunner/runner.py Outdated
@guewen

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 13.0-ocabot-merge-pr-265-by-guewen-bump-patch, awaiting test results.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 3365bff. Thanks a lot for contributing to OCA. ❤️

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@nilshamerlinck@OCA-git-bot@guewen@simahawk@sbidoul