Skip to content

chore: update & fix otel example#3466

Merged
ndyakov merged 1 commit into
redis:masterfrom
vmihailenco:chore/update-otel-example
Aug 11, 2025
Merged

chore: update & fix otel example#3466
ndyakov merged 1 commit into
redis:masterfrom
vmihailenco:chore/update-otel-example

Conversation

@vmihailenco

@vmihailenco vmihailenco commented Aug 9, 2025

Copy link
Copy Markdown
Collaborator

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

ndyakov commented Aug 11, 2025

Copy link
Copy Markdown
Member

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
Copy Markdown
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

ndyakov commented Aug 11, 2025

Copy link
Copy Markdown
Member

@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
Copy Markdown
Collaborator Author

@ndyakov PTAL

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

@ndyakov ndyakov left a comment

Copy link
Copy Markdown
Member

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