Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add Prometheus Remote Write Exporter supporting Cortex - helper #1548

Closed
wants to merge 21 commits into from

Conversation

huyan0
Copy link
Member

@huyan0 huyan0 commented Aug 14, 2020

This PR is part of a series of PRs implementing a Prometheus remote write exporter supporting Cortex. PR #1525 contains all the code. Note that ~100 lines of code are coming from changes in go.mod

Description: This PR adds helper methods used by the exporter, including functionalities such as sanitizing strings and creating Prometheus labels from OTLP labels.

helper.go: helper functions that cortex.go uses.

testutil_test.go: constant values used in tests for all files in the project.

Documentation:

cc: @huyan0 @alolita

Request for review: @jmacd @bogdandrutu @annanay25 @gouthamve @open-telemetry/collector-approvers @open-telemetry/collector-maintainers

@huyan0 huyan0 changed the title Helper Add Prometheus Remote Write Exporter supporting Cortex - helper Aug 14, 2020
@codecov
Copy link

codecov bot commented Aug 14, 2020

Codecov Report

Merging #1548 into master will decrease coverage by 0.00%.
The diff coverage is 88.29%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1548      +/-   ##
==========================================
- Coverage   91.42%   91.42%   -0.01%     
==========================================
  Files         248      249       +1     
  Lines       17219    17313      +94     
==========================================
+ Hits        15743    15828      +85     
- Misses       1064     1074      +10     
+ Partials      412      411       -1     
Impacted Files Coverage Δ
exporter/prometheusremotewriteexporter/helper.go 88.29% <88.29%> (ø)
translator/internaldata/resource_to_oc.go 87.50% <0.00%> (+2.08%) ⬆️

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 e68440e...a38f6bc. Read the comment docs.

@huyan0 huyan0 closed this Aug 14, 2020
hughesjj pushed a commit to hughesjj/opentelemetry-collector that referenced this pull request Apr 27, 2023
…y#1548)

Bumps [boto3](https://github.com/boto/boto3) from 1.22.10 to 1.22.11.
- [Release notes](https://github.com/boto/boto3/releases)
- [Changelog](https://github.com/boto/boto3/blob/develop/CHANGELOG.rst)
- [Commits](boto/boto3@1.22.10...1.22.11)

---
updated-dependencies:
- dependency-name: boto3
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant