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.

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

Update the rolldice example #1566

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Update the rolldice example #1566

wants to merge 6 commits into from

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
examples/rolldice/README.md Outdated Show resolved Hide resolved
@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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RonFed RonFed RonFed left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
v0.20.0-alpha
Development

Successfully merging this pull request may close these issues.
2 participants
@MrAlias @RonFed