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

stats/opentelemetry: CSM Observability client side component changes #7256

Merged
merged 6 commits into from
May 23, 2024

Conversation

zasweq
Copy link
Contributor

@zasweq zasweq commented May 22, 2024

This PR adds the CSM Observability logic for the client side of the OpenTelemetry component.

RELEASE NOTES: N/A

@zasweq zasweq requested a review from dfawley May 22, 2024 21:46
@zasweq zasweq added the Type: Feature New features or improvements in behavior label May 22, 2024
@zasweq zasweq added this to the 1.65 Release milestone May 22, 2024
stats/opentelemetry/client_metrics.go Outdated Show resolved Hide resolved
stats/opentelemetry/client_metrics.go Outdated Show resolved Hide resolved
stats/opentelemetry/client_metrics.go Outdated Show resolved Hide resolved
stats/opentelemetry/client_metrics.go Outdated Show resolved Hide resolved
stats/opentelemetry/opentelemetry.go Outdated Show resolved Hide resolved
stats/opentelemetry/client_metrics.go Outdated Show resolved Hide resolved
stats/opentelemetry/client_metrics.go Outdated Show resolved Hide resolved
stats/opentelemetry/opentelemetry.go Outdated Show resolved Hide resolved
stats/opentelemetry/opentelemetry.go Outdated Show resolved Hide resolved
stats/opentelemetry/opentelemetry.go Outdated Show resolved Hide resolved
@dfawley dfawley assigned zasweq and unassigned dfawley May 22, 2024
@zasweq zasweq assigned dfawley and unassigned zasweq May 22, 2024
Copy link

codecov bot commented May 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.68%. Comparing base (adf976b) to head (52ee505).
Report is 65 commits behind head on master.

Current head 52ee505 differs from pull request most recent head 3c46cdd

Please upload reports for the commit 3c46cdd to get more accurate results.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7256      +/-   ##
==========================================
- Coverage   81.24%   80.68%   -0.56%     
==========================================
  Files         345      349       +4     
  Lines       33941    33993      +52     
==========================================
- Hits        27574    27426     -148     
- Misses       5202     5392     +190     
- Partials     1165     1175      +10     

see 37 files with indirect coverage changes

@zasweq zasweq assigned dfawley and unassigned dfawley May 23, 2024
@dfawley dfawley assigned zasweq and unassigned dfawley May 23, 2024
@zasweq zasweq merged commit f7d3d3e into grpc:master May 23, 2024
12 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Feature New features or improvements in behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants