You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In existing Prometheus Receiver tests, func TestEndtoEnd translates the OTLP format metrics to OpenCensus format before proceeding for test validations. The data-point flag to identify staleness markers, is not accessible using the OpenCensus format, and there are existing issues that cannot be resolved without datapoint flag for staleness markers.
See Issue: open-telemetry/prometheus-interoperability-spec#38
Describe the solution you'd like Exiting Prometheus Receiver test receives metrics from the consumertest.MetricsSink in OTLP format. Modify this test to remove the translation from OTLP to OpenCensus format and modify all the existing functions associated with validations and assertions (as in the link) to accept OTLP format metrics instead of OpenCensus.
… directly for validations and accept staleness markers (#6219)
* Modifies existing Prometheus Receiver Tests to use pdata directly for validation
* Removed dead code from helper file
* Fixed the lint issue
* Fixing the imports lint error
* Skipping start_timestamp check for histogram and summary in target1scrape2
* Resolved feedback
* Resolved review feedback comments
* Trimming the assert messages
* Fix to avoid panic situation in assertMetricPresent method
* Moved testing as first argument in doCompare method
* Removed bool propogation in comparators and changed setupMockPrometheus to avoid replaceall on config
PaurushGarg
changed the title
Modify existing Prometheus Receiver tests to validate metrics using OTLP format instead of OpenCensus format
[Prometheus Receiver] Modify existing tests to validate metrics using OTLP format instead of OpenCensus format
Nov 17, 2021
Describe the Problem
Existing Prometheus Receiver test translates OTLP format metrics to OpenCensus format, and then uses these translated metrics for validation and assertions. Currently, Prometheus Receiver uses OpenCensus metricbuilder, but this is planned to be switched to OTLP metricbuilder, in the near future. After this change, staleNaN can be identified using data-point flags (MetricDataPointFlagNoRecordedValue).
See Issues:
open-telemetry/opentelemetry-proto#316
open-telemetry/opentelemetry-proto#310
In existing Prometheus Receiver tests,
func TestEndtoEnd
translates the OTLP format metrics to OpenCensus format before proceeding for test validations. The data-point flag to identify staleness markers, is not accessible using the OpenCensus format, and there are existing issues that cannot be resolved without datapoint flag for staleness markers.See Issue:
open-telemetry/prometheus-interoperability-spec#38
Describe the solution you'd like
Exiting Prometheus Receiver test receives metrics from the
consumertest.MetricsSink
in OTLP format. Modify this test to remove the translation from OTLP to OpenCensus format and modify all the existing functions associated with validations and assertions (as in the link) to accept OTLP format metrics instead of OpenCensus.Additional context
Related to open-telemetry/prometheus-interoperability-spec#57
Related Issues:
#6000
#6087
cc: @alolita @Aneurysm9
The text was updated successfully, but these errors were encountered: