Skip to content

Conversation

@JasonXZLiu
Copy link
Member

@JasonXZLiu JasonXZLiu commented Oct 26, 2020

Description

This PR was introduced to fix the logging exporter to increment the histogram bucket labels (when logging out metrics). Previously, the bucket count stayed at 0. However, bucket labels should increment to conform with the Explicit Bound labels.

Related issue: #2008

@JasonXZLiu JasonXZLiu requested a review from a team October 26, 2020 19:23
@JasonXZLiu JasonXZLiu changed the title Add index count for histogram buckets in logging exporter Logging Exporter: Add index for histogram buckets count Oct 26, 2020
@codecov
Copy link

codecov bot commented Oct 26, 2020

Codecov Report

Merging #2009 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2009   +/-   ##
=======================================
  Coverage   91.50%   91.50%           
=======================================
  Files         282      282           
  Lines       16628    16628           
=======================================
  Hits        15216    15216           
  Misses        977      977           
  Partials      435      435           
Impacted Files Coverage Δ
exporter/loggingexporter/logging_exporter.go 88.92% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b31b91b...7bb491a. Read the comment docs.

@bogdandrutu bogdandrutu merged commit b792ca6 into open-telemetry:master Oct 26, 2020
@JasonXZLiu JasonXZLiu deleted the fix-logging-exporter-bucket-index branch November 2, 2020 20:16
MovieStoreGuy pushed a commit to atlassian-forks/opentelemetry-collector that referenced this pull request Nov 11, 2021
* Ensure that no-op tracer still progates non-recording spans

Signed-off-by: Anthony J Mirabella <[email protected]>

* Add `TracerProvider` to the `trace.Span` interface

Signed-off-by: Anthony J Mirabella <[email protected]>

* Remove config from oteltest.Tracer as it can be accessed through the provider

Signed-off-by: Anthony J Mirabella <[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.

2 participants