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

Add initial implementation of the zpages for Otel Trace #894

Merged
merged 4 commits into from
Jul 21, 2021

Conversation

bogdandrutu
Copy link
Member

@bogdandrutu bogdandrutu commented Jul 19, 2021

zpages/bucket.go Outdated Show resolved Hide resolved
zpages/bucket.go Outdated Show resolved Hide resolved
zpages/internal/templates/header.html Outdated Show resolved Hide resolved
zpages/templates.go Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
zpages/boundaries_test.go Outdated Show resolved Hide resolved
zpages/boundaries_test.go Outdated Show resolved Hide resolved
zpages/bucket_test.go Outdated Show resolved Hide resolved
zpages/bucket_test.go Outdated Show resolved Hide resolved
zpages/spanprocessor_test.go Outdated Show resolved Hide resolved
@bogdandrutu bogdandrutu force-pushed the zpages branch 3 times, most recently from 11c90ac to fcc8a07 Compare July 20, 2021 20:41
@codecov
Copy link

codecov bot commented Jul 20, 2021

Codecov Report

Merging #894 (5e7f924) into main (d586bc7) will decrease coverage by 3.7%.
The diff coverage is 41.5%.

Impacted file tree graph

@@           Coverage Diff           @@
##            main    #894     +/-   ##
=======================================
- Coverage   79.1%   75.3%   -3.8%     
=======================================
  Files         62      68      +6     
  Lines       2748    3054    +306     
=======================================
+ Hits        2174    2301    +127     
- Misses       443     613    +170     
- Partials     131     140      +9     
Impacted Files Coverage Δ
zpages/tracez.go 0.0% <0.0%> (ø)
zpages/templates.go 21.0% <21.0%> (ø)
zpages/internal/resources.go 25.3% <25.3%> (ø)
zpages/boundaries.go 100.0% <100.0%> (ø)
zpages/bucket.go 100.0% <100.0%> (ø)
zpages/spanprocessor.go 100.0% <100.0%> (ø)
... and 2 more

@MrAlias
Copy link
Contributor

MrAlias commented Jul 20, 2021

@MadVikingGod are you around to take another look?

@MrAlias
Copy link
Contributor

MrAlias commented Jul 20, 2021

Looks like zpages/go.sum needs to be updated. make precommit should do what you need.

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
@bogdandrutu
Copy link
Member Author

@MrAlias done

@MrAlias MrAlias merged commit 376f3e8 into open-telemetry:main Jul 21, 2021
@Aneurysm9 Aneurysm9 mentioned this pull request Jul 26, 2021
@pellared pellared added this to the untracked milestone Nov 8, 2024
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.

5 participants