Skip to content

chore: update & fix otel example #3466

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

Merged
merged 1 commit into from
Aug 11, 2025

Conversation

vmihailenco
Copy link
Collaborator

@vmihailenco vmihailenco commented Aug 9, 2025

Update the example to use Uptrace v2 that was recently released. It comes with a new config formatting.

Also updated README with new instructions and screenshots.

@ndyakov
Copy link
Member

ndyakov commented Aug 11, 2025

Hello @vmihailenco, glad to see you back opening a PR. Could you please add more description in the PR to what this fixes and what the additions are. Thank you!

@vmihailenco
Copy link
Collaborator Author

@ndyakov basically I've just upgraded the example to use Uptrace v2 we've recently released. It comes with a new config formatting so it might look scary, but it is just a copy/paste of the default config.

Also updated README with new instructions and screenshots.

Is there anything in particular that concerns you?

@ndyakov
Copy link
Member

ndyakov commented Aug 11, 2025

@vmihailenco not necessarily, I just want to add a bit of context for the commit that will be merged in master. Than you, will add this information as part of the commit message.

Uptrace v2 comes with a new config file and new defaults.
Also updated dependencies to the latest versions.
@vmihailenco vmihailenco force-pushed the chore/update-otel-example branch from e9bf81b to c7a01e7 Compare August 11, 2025 09:11
@vmihailenco
Copy link
Collaborator Author

@ndyakov PTAL

@ndyakov ndyakov self-requested a review August 11, 2025 09:31
Copy link
Member

@ndyakov ndyakov 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!

@ndyakov ndyakov merged commit 2c29ded into redis:master Aug 11, 2025
20 checks passed
Sign up for free to 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.

2 participants