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

Update prometheus deps #1673

Merged
merged 1 commit into from
Sep 17, 2020
Merged

Conversation

bogdandrutu
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Aug 29, 2020

Codecov Report

Merging #1673 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1673      +/-   ##
==========================================
+ Coverage   91.95%   91.96%   +0.01%     
==========================================
  Files         263      263              
  Lines       18821    18821              
==========================================
+ Hits        17307    17309       +2     
+ Misses       1083     1082       -1     
+ Partials      431      430       -1     
Impacted Files Coverage Δ
receiver/prometheusreceiver/internal/ocastore.go 93.75% <100.00%> (ø)
receiver/prometheusreceiver/metrics_receiver.go 79.54% <100.00%> (ø)
translator/internaldata/resource_to_oc.go 91.91% <0.00%> (+2.02%) ⬆️

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 e92658a...036856f. Read the comment docs.

@tigrannajaryan
Copy link
Member

@bogdandrutu please resolve the conflicts.

@tigrannajaryan
Copy link
Member

@bogdandrutu please resolve the conflicts.

@github-actions
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Sep 17, 2020
Signed-off-by: Bogdan Drutu <[email protected]>
@bogdandrutu bogdandrutu merged commit 639b9a8 into open-telemetry:master Sep 17, 2020
@bogdandrutu bogdandrutu deleted the updateprom branch September 17, 2020 17:01
bogdandrutu added a commit to bogdandrutu/opentelemetry-collector that referenced this pull request Sep 18, 2020
MovieStoreGuy pushed a commit to atlassian-forks/opentelemetry-collector that referenced this pull request Nov 11, 2021
* Jaeger exporter now populate Jaeger's Span Process from Resource

* Remove jaeger.WithProcess

* Fix tests

* Change the type of default service name into string

* Add tests

* Update CHANGELOG

* Use the API from `Set` to fetch service name in exporter

* Fix nits

* Add more test cases for jaegerBatchList function

* precommit

Co-authored-by: Anthony Mirabella <[email protected]>
Troels51 pushed a commit to Troels51/opentelemetry-collector that referenced this pull request Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants