Skip to content

Conversation

hibachrach
Copy link
Contributor

This still permits static values (and environment variable to configure them) but allows users to opt-in to having a callable/dynamic value for the given option.

Resolves #991

@hibachrach hibachrach force-pushed the make-sidekiq-propagation-style-dynamic branch 2 times, most recently from 6d698ab to f568e48 Compare August 7, 2025 21:56
…dynamic option

This still permits static values (and environment variable to configure
them) but allows users to opt-in to having a callable/dynamic value for
the given option.

Resolves open-telemetry#991
@hibachrach hibachrach force-pushed the make-sidekiq-propagation-style-dynamic branch from f568e48 to c16cb5f Compare August 12, 2025 23:45
@hibachrach hibachrach marked this pull request as ready for review August 18, 2025 18:26
@hibachrach
Copy link
Contributor Author

FYI, I opted to change the config handling code to allow for more complex config value types, but happy to add a new callable config option that would take precedent over the current enum value instead and revert all of that. Not sure which would be preferred.

Copy link
Contributor

👋 This pull request has been marked as stale because it has been open with no activity. You can: comment on the issue or remove the stale label to hold stale off for a while, add the keep label to hold stale off permanently, or do nothing. If you do nothing this pull request will be closed eventually by the stale bot

@github-actions github-actions bot added the stale Marks an issue/PR stale label Sep 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale Marks an issue/PR stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make Sidekiq propagation_style able to be changed on a job-by-job basis
1 participant