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(celery): Send Celery retry count #2971

Open
wants to merge 4 commits into
base: szokeasaurusrex/celery-queue-consumer-default-exchange
Choose a base branch
from

Conversation

szokeasaurusrex
Copy link
Member

@szokeasaurusrex szokeasaurusrex commented Apr 11, 2024

Depends on #2984

ref #2959

@szokeasaurusrex szokeasaurusrex self-assigned this Apr 12, 2024
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/celery-retry-count branch from 4162d95 to bbf6bd3 Compare April 12, 2024 11:15
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/celery-retry-count branch from bbf6bd3 to 57f1d44 Compare April 24, 2024 09:58
@szokeasaurusrex szokeasaurusrex changed the base branch from sentry-sdk-2.0 to szokeasaurusrex/celery-queue-consumer-default-exchange April 24, 2024 09:58
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/celery-retry-count branch 3 times, most recently from 8a657d8 to ef03803 Compare April 24, 2024 11:39
@szokeasaurusrex szokeasaurusrex marked this pull request as ready for review April 24, 2024 13:09
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/celery-queue-consumer-default-exchange branch from 8028917 to 5a99aad Compare April 24, 2024 13:55
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/celery-retry-count branch from 79d584d to 57bd97e Compare April 24, 2024 13:56
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/celery-queue-consumer-default-exchange branch from 2fea931 to 3bf4c12 Compare April 26, 2024 10:18
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/celery-retry-count branch from edf4c0a to 5baf5dd Compare April 26, 2024 10:21
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/celery-queue-consumer-default-exchange branch from f1671f5 to 67e2531 Compare April 30, 2024 15:16
szokeasaurusrex and others added 3 commits April 30, 2024 17:18
Add a test that ensures the retry count is set on the Celery task span. This test only tests a retry count of zero.
Add a test to verify that retry count gets set correctly, also for nonzero values.
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

3 participants