Skip to content

Add Prometheus Exporter: Step 2 - PrometheusCollector#282

Merged
reyang merged 10 commits into
open-telemetry:masterfrom
erichsueh3:prometheus_collector
Aug 28, 2020
Merged

Add Prometheus Exporter: Step 2 - PrometheusCollector#282
reyang merged 10 commits into
open-telemetry:masterfrom
erichsueh3:prometheus_collector

Conversation

@erichsueh3
Copy link
Copy Markdown
Contributor

This PR implements the PrometheusCollector class and its header, source, test, and build files. This class is derived from the Collectable class in the C++ Prometheus client and allows for data translated from OpenTelemetry for Prometheus to be collected by an end user.

This PR is one in a series of PRs that aims to add a full implementation of a Prometheus Exporter within the OpenTelemetry C++ library.

Note to reviewers: Please keep in mind that this PR is dependent on the Prometheus Exporter Utils PRs(#280) so codecov tests may not fully pass.

@erichsueh3 erichsueh3 requested a review from a team August 18, 2020 17:37
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 18, 2020

Codecov Report

Merging #282 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #282   +/-   ##
=======================================
  Coverage   94.55%   94.55%           
=======================================
  Files         146      146           
  Lines        6610     6610           
=======================================
  Hits         6250     6250           
  Misses        360      360           

Copy link
Copy Markdown
Member

@reyang reyang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't actually got time to go through the change, given the internship is ending soon for @erichsueh3 and this change only touched Prometheus exporter (and a tiny bit on the CI job), based on offline discussion we'll move forward and get this merged for now.

@reyang reyang merged commit 935e7ff into open-telemetry:master Aug 28, 2020
@lalitb lalitb mentioned this pull request Mar 10, 2021
GerHobbelt pushed a commit to GerHobbelt/opentelemetry-cpp that referenced this pull request Jun 17, 2025
[SDK] support aggregation of identical instruments (open-telemetry#3358)
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.

3 participants