Skip to content

Update the rolldice example #1566

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 8 commits into from
Jan 13, 2025
Merged

Update the rolldice example #1566

merged 8 commits into from
Jan 13, 2025

Conversation

@MrAlias
Copy link
Contributor

@MrAlias MrAlias commented Jan 8, 2025 •
edited
Loading

Part of #236

This adapts the example found in https://github.com/MrAlias/otel-auto-demo into our current rolldice example.

This change highlights ...

  • support for context propagation by splitting the application into two services
  • multiple instrumentation probes by including a database to manage users
  • the integration with otel global API and hybrid instrumentation

@MrAlias MrAlias added the documentation Improvements or additions to documentation label Jan 8, 2025
@MrAlias MrAlias added this to the v0.20.0-alpha milestone Jan 8, 2025
@MrAlias MrAlias force-pushed the rolldice-example-update branch from c25900c to cbd61a3 Compare January 8, 2025 23:04
@MrAlias MrAlias marked this pull request as ready for review January 8, 2025 23:08
@MrAlias MrAlias requested a review from a team as a code owner January 8, 2025 23:08
RonFed
RonFed reviewed Jan 9, 2025
View reviewed changes
@MrAlias MrAlias force-pushed the rolldice-example-update branch from cbd61a3 to 05a7a32 Compare January 9, 2025 20:34
@MrAlias MrAlias force-pushed the rolldice-example-update branch from 05a7a32 to e471bab Compare January 9, 2025 20:40
@MrAlias MrAlias merged commit 190fef5 into open-telemetry:main Jan 13, 2025
27 checks passed
@MrAlias MrAlias deleted the rolldice-example-update branch January 13, 2025 17:04
@MrAlias MrAlias mentioned this pull request Jan 28, 2025
Merged
@MrAlias MrAlias mentioned this pull request Feb 7, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@RonFed RonFed RonFed approved these changes

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

v0.20.0 - Beta Release

Development

Successfully merging this pull request may close these issues.

2 participants

@MrAlias @RonFed