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

feat(integrations): Add support for celery-redbeat cron tasks #2643

Merged
merged 3 commits into from Mar 18, 2024

Conversation

kwigley
Copy link
Contributor

@kwigley kwigley commented Jan 15, 2024

This add support for celery-redbeat (resolves #2616). Thanks to #2616 (comment)


General Notes

Thank you for contributing to sentry-python!

Please add tests to validate your changes, and lint your code using tox -e linters.

Running the test suite on your PR might require maintainer approval. Some tests (AWS Lambda) additionally require a maintainer to add a special label to run and will fail if the label is not present.

For maintainers

Sensitive test suites require maintainer review to ensure that tests do not compromise our secrets. This review must be repeated after any code revisions.

Before running sensitive test suites, please carefully check the PR. Then, apply the Trigger: tests using secrets label. The label will be removed after any code changes to enforce our policy requiring maintainers to review all code revisions before running sensitive tests.

@sentrivana
Copy link
Contributor

Thanks a lot @kwigley! We will take a look.

@sentrivana
Copy link
Contributor

Hey @kwigley, sorry for the wait.

I've tried the PR but it doesn't work for me -- did it work for you? As far as I can tell the problem is we attempt to patch the RedBeatScheduler class, but we're not patching it in the right place or at the right time, because when I run this, only the original maybe_due is called, but not our wrapped one. Lmk if the code worked for you as is -- in that case it might be something with my setup.

@sentrivana
Copy link
Contributor

Ok it was a problem with my setup (and our docs :) ) -- I'm getting monitors now. Will give the PR one more look but looks good at first glance!

@kwigley
Copy link
Contributor Author

kwigley commented Mar 18, 2024

Ok it was a problem with my setup (and our docs :) ) -- I'm getting monitors now. Will give the PR one more look but looks good at first glance!

ah, missed your previous comment. Sounds good! lmk if you need anything else

@sentrivana
Copy link
Contributor

@kwigley All good, no action needed on your end, this is good to merge. Thank you!

@sentrivana sentrivana enabled auto-merge (squash) March 18, 2024 16:07
@sentrivana sentrivana merged commit 68b9180 into getsentry:master Mar 18, 2024
124 of 127 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Trigger: tests using secrets PR code is safe; run CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automatically create monitors when using Redbeat
2 participants