-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Enable awsecscontainermetricsreceiver- update components and go.mod #1160
Enable awsecscontainermetricsreceiver- update components and go.mod #1160
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1160 +/- ##
==========================================
+ Coverage 89.97% 89.99% +0.02%
==========================================
Files 283 283
Lines 13772 13772
==========================================
+ Hits 12391 12394 +3
+ Misses 1013 1010 -3
Partials 368 368
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Hi @tigrannajaryan and @bogdandrutu can I get this reviewed and merged. It's just two line change for enabling the receiver. Thanks in advanced. |
go.mod
Outdated
@@ -6,7 +6,7 @@ require ( | |||
github.com/client9/misspell v0.3.4 | |||
github.com/go-lintpack/lintpack v0.5.2 // indirect | |||
github.com/golangci/golangci-lint v1.31.0 | |||
github.com/google/addlicense v0.0.0-20200622132530-df58acafd6d5 | |||
github.com/google/addlicense v0.0.0-20200906110928-a0294312aa76 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated to the old version.
Description:
This change will enable the
awsecscontainermetricsreceiver
in OT components.Link to tracking Issue:
#457
Testing:
Unit test added.
Documentation:
README added.