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(active-job)!: Normalize event messages #1080

Merged

Conversation

lavoiesl
Copy link
Contributor

@lavoiesl lavoiesl commented Jul 22, 2024

Designed to be merged after #1079

Related to:

Currently, active job events are:

Notification Before After
enqueue.active_job #{prefix} publish #{prefix} publish
enqueue_at.active_job #{prefix} publish #{prefix} publish
enqueue_retry.active_job enqueue_retry.active_job #{prefix} enqueue_retry
perform.active_job #{prefix} process #{prefix} process
retry_stopped.active_job retry_stopped.active_job #{prefix} retry_stopped
discard.active_job discard.active_job #{prefix} discard

Prefix being the queue name or the job class, depending on config.

I think this is more consistent.

⚠️ Note that this is a BREAKING CHANGE with regards to the spans added in #677

Copy link
Contributor

@bensheldon bensheldon left a comment

Choose a reason for hiding this comment

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

Thank you for making this more consistent across all of the events 👍🏻

@lavoiesl lavoiesl changed the title feat(active_job): Normalize event messages feat(active-job): Normalize event messages Jul 22, 2024
@lavoiesl lavoiesl force-pushed the seb-active_job-default-events branch 2 times, most recently from 67bda51 to b5fc587 Compare July 22, 2024 18:15
@kaylareopelle
Copy link
Contributor

@lavoiesl - It looks merging #1079 caused some conflicts here. Can you take a look?

@lavoiesl lavoiesl force-pushed the seb-active_job-default-events branch from b5fc587 to fb6b493 Compare July 23, 2024 23:04
@lavoiesl
Copy link
Contributor Author

Here we go 👍🏼

@kaylareopelle
Copy link
Contributor

@lavoiesl - Thank you for fixing the merge conflicts! I missed your update after another PR was merged into main. 😞

Could you merge main into your branch?

@lavoiesl lavoiesl force-pushed the seb-active_job-default-events branch from fb6b493 to ea1cc33 Compare July 25, 2024 21:39
@lavoiesl
Copy link
Contributor Author

Rebased.

@kaylareopelle kaylareopelle changed the title feat(active-job): Normalize event messages feat!(active-job): Normalize event messages Jul 30, 2024
@kaylareopelle kaylareopelle enabled auto-merge (squash) July 30, 2024 21:21
@kaylareopelle kaylareopelle changed the title feat!(active-job): Normalize event messages feat(active-job)!: Normalize event messages Jul 30, 2024
@lavoiesl
Copy link
Contributor Author

@kaylareopelle updated

@kaylareopelle kaylareopelle merged commit 39056a3 into open-telemetry:main Jul 30, 2024
55 checks passed
@lavoiesl lavoiesl deleted the seb-active_job-default-events branch July 30, 2024 21:54
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.

3 participants