Skip to content

[demo/accountingservice] instrument by OTel .NET Auto#4448

Merged
cartermp merged 9 commits intoopen-telemetry:mainfrom
RassK:update-accountingservice
Jul 2, 2024
Merged

[demo/accountingservice] instrument by OTel .NET Auto#4448
cartermp merged 9 commits intoopen-telemetry:mainfrom
RassK:update-accountingservice

Conversation

@RassK
Copy link
Copy Markdown
Contributor

@RassK RassK commented May 8, 2024

Why

related to open-telemetry/opentelemetry-demo#1538

What

Updates accounting service documentation according to the demo PR.

@RassK RassK force-pushed the update-accountingservice branch 2 times, most recently from 8cb2def to 2d80588 Compare May 8, 2024 13:04
@svrnm svrnm marked this pull request as ready for review May 10, 2024 08:33
@svrnm svrnm requested review from a team May 10, 2024 08:33
Copy link
Copy Markdown
Member

@svrnm svrnm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM overall, I mark it as "blocked" since it depends on the upstream PR to be merged

cc @open-telemetry/demo-approvers

@svrnm svrnm added the blocked label May 10, 2024
@svrnm svrnm changed the title [Draft] Update accountingservice Update accountingservice May 13, 2024
@svrnm
Copy link
Copy Markdown
Member

svrnm commented May 13, 2024

/fix:all

@opentelemetrybot
Copy link
Copy Markdown
Contributor

@Kielek Kielek changed the title Update accountingservice [demo/accountingservice] instrument by OTel .NET Auto Jul 2, 2024
@Kielek
Copy link
Copy Markdown
Member

Kielek commented Jul 2, 2024

Ready to be merged. OTel demo 1.11.0 is out https://github.com/open-telemetry/opentelemetry-demo/releases/tag/1.11.0

@cartermp cartermp merged commit 9b5e318 into open-telemetry:main Jul 2, 2024
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.

6 participants