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.

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

[Cart] Added instrumentation for Asp.Net Core (http server) #394

Merged
merged 9 commits into from
Sep 30, 2022

Conversation

cijothomas
Copy link
Member

@cijothomas cijothomas commented Sep 30, 2022

Also updated version from 1.3.0 to 1.3.1 as 1.3.0 was deprecated due to vulnerability.
See metrics:
image

@cijothomas cijothomas requested a review from a team September 30, 2022 00:35
@cijothomas cijothomas changed the title Added instrumentation for Asp.Net Core (http server) in CartService [Cart] Added instrumentation for Asp.Net Core (http server) Sep 30, 2022
@cartersocha
Copy link
Contributor

Probably worth adding a changelog update for these. Slipped my mind on the first PR

@cartersocha cartersocha merged commit b6e75ee into open-telemetry:main Sep 30, 2022
@cijothomas cijothomas deleted the cijothomas/more-metrics branch September 30, 2022 16:33
jmichalak9 pushed a commit to jmichalak9/opentelemetry-demo that referenced this pull request Mar 22, 2024
…emetry#394)

* Add runtime metrics to cartservice

* reorder

* Add http server metrics for cart service

* update doc

* bump version

* Update src/cartservice/src/cartstore/RedisCartStore.cs

Co-authored-by: Reiley Yang <[email protected]>

* Update src/cartservice/src/Startup.cs

* Update src/cartservice/src/Startup.cs

Co-authored-by: Reiley Yang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants