Skip to content

Enable docker stats receiver #1081

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

Merged
merged 1 commit into from
Sep 21, 2020
Merged

Enable docker stats receiver #1081

merged 1 commit into from
Sep 21, 2020

Conversation

rmfitzpatrick
Copy link
Contributor

@rmfitzpatrick rmfitzpatrick commented Sep 21, 2020

Description: Adding a feature - Adds the docker stats receiver as a top level component.

Testing: tested manually w/ utilizing config.

Documentation: Updated owner section and changelog.

@rmfitzpatrick rmfitzpatrick requested a review from a team September 21, 2020 20:56
@codecov
Copy link

codecov bot commented Sep 21, 2020

Codecov Report

Merging #1081 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1081      +/-   ##
==========================================
+ Coverage   88.81%   88.82%   +0.01%     
==========================================
  Files         250      250              
  Lines       11924    11924              
==========================================
+ Hits        10590    10592       +2     
+ Misses        990      989       -1     
+ Partials      344      343       -1     
Flag Coverage Δ
#integration 75.42% <ø> (ø)
#unit 87.84% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
receiver/k8sclusterreceiver/watcher.go 97.64% <0.00%> (+2.35%) ⬆️

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 bff19a2...ecf8a10. Read the comment docs.

@bogdandrutu bogdandrutu merged commit bd1d51b into open-telemetry:master Sep 21, 2020
@bogdandrutu bogdandrutu deleted the enable_docker_stats branch September 21, 2020 22:26
ljmsc referenced this pull request in ljmsc/opentelemetry-collector-contrib Feb 21, 2022
* Bump github.com/itchyny/gojq from 0.11.0 to 0.11.1 in /tools

Bumps [github.com/itchyny/gojq](https://github.com/itchyny/gojq) from 0.11.0 to 0.11.1.
- [Release notes](https://github.com/itchyny/gojq/releases)
- [Changelog](https://github.com/itchyny/gojq/blob/master/CHANGELOG.md)
- [Commits](itchyny/gojq@v0.11.0...v0.11.1)

Signed-off-by: dependabot[bot] <[email protected]>

* Auto-fix go.sum changes in dependent modules

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
Co-authored-by: Tyler Yahn <[email protected]>
codeboten pushed a commit that referenced this pull request Nov 23, 2022
* Add Boto3SQS Instrumentation

* Add basic tests

* Add context setting list

* Fix linting

* CR and lint fixes

* Add newline

* Run tox generate

* Change the dependency version

* Fix linting

* PR fixes. Remove walrus operator and use the `start_as_current_span`

* Run lint and generate

Co-authored-by: Srikanth Chekuri <[email protected]>
Co-authored-by: Diego Hurtado <[email protected]>
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