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

Elastic Exporter: Metrics: DoubleValueRecorder and LongValueRecorder are silently ignored #1610

Closed
cyrille-leclerc opened this issue Nov 16, 2020 · 4 comments

Comments

@cyrille-leclerc
Copy link
Member

Describe the bug

DoubleValueRecorder and LongValueRecorder are silently ignored. We could document it and maybe emit a least one warning message per metric waiting to add support for it.

Steps to reproduce

Create a DoubleValueRecorderwith the OpenTelemetry SDK and export the metrics to Elastic. The metric will not appear

Meter meter = OpenTelemetry.getGlobalMeter("my-application");
DoubleValueRecorder orderValueRecorder = meter.doubleValueRecorderBuilder("order").build();

public void createOrder(){
    double orderPrice = ...
    // business logic

    orderValueRecorder.record(orderPrice);
}

What did you expect to see?

An histogram metric in Elastic or at least, two counters similarly to what the Elastic Java Agent does with Micrometer.

What did you see instead?

No metric is created in Elastic for LongValueRecorder and DoubleValueRecorder without message to help understand the limitation.
The source code of the Elastic Exporter indicates "TODO(axw) requires elastic/apm-server#3195" (see metrics.go#EncodeMetrics())

What version did you use?

  • OpenTelemetry Collector 0.14
  • OpenTelemetry Java Auto Instrumentation 0.10.1

What config did you use?

extensions:
  zpages:
    endpoint: :55679

receivers:
  otlp:
    protocols:
      grpc:
        endpoint: '127.0.0.1:55680'
  hostmetrics:
    collection_interval: 1m
    scrapers:
      cpu:
      load:
      memory:

processors:
  batch: null
  queued_retry: null

exporters:
  elastic:
    apm_server_url: 'http://localhost:8200'
    secret_token: 'my_secret_token'
  logging:
    loglevel: info
  prometheus:
    endpoint: 0.0.0.0:8889

service:
  pipelines:
    metrics:
      receivers:
        - otlp
        - hostmetrics
      exporters:
        - logging
        - prometheus
        - elastic
    traces:
      receivers:
        - otlp
      processors:
        - batch
      exporters:
        - elastic
  extensions: [zpages]

Environment
OS: MacOS

@cyrille-leclerc cyrille-leclerc added the bug Something isn't working label Nov 16, 2020
@bogdandrutu
Copy link
Member

/cc @axw @simitt @jalvz

@axw
Copy link
Contributor

axw commented Dec 3, 2020

Thanks for the ping. There are some enhancements required on the backend before we can support this.

Cross-referencing: elastic/apm#377

@github-actions
Copy link
Contributor

This issue has been inactive for 60 days. It will be closed in 120 days if there is no activity.

@github-actions github-actions bot added the Stale label Oct 24, 2022
@cyrille-leclerc
Copy link
Member Author

Close as Elastic now has an OTLP intake making the Elastic Exporter less relevant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants