-
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
Don't set origin to EC2 when not on AWS. #1115
Don't set origin to EC2 when not on AWS. #1115
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1115 +/- ##
===========================================
+ Coverage 73.67% 88.98% +15.31%
===========================================
Files 22 258 +236
Lines 1018 12333 +11315
===========================================
+ Hits 750 10975 +10225
- Misses 217 1008 +791
- Partials 51 350 +299
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
if resource.IsNil() { | ||
return OriginEC2 | ||
return "" |
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.
Nice
…ector-contrib into no-aws-on-non-aws-origin
Please fix the conflict and make sure the build passes. |
…ector-contrib into no-aws-on-non-aws-origin
@tigrannajaryan Thanks, merged master |
If ResourceCheckPeriod is more than 1 second, reported resource values are not being changed during each ResourceCheckPeriod, so logging every second doens't provide much value, but makes it hard to read logs in long run running tests
* Change name of ProbabilitySampler to TraceIdRatioBased * Modify behavior to ignore parent span * Add test for inclusivity property on TraceIdRatioBased sampler * Modify tests in `trace_test.go` to reflect change in parent span behavior * Add to CHANGELOG * Satisfy golint * Update CHANGELOG.md Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com> Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
We can't only look at semantic attributes for determining AWS origin values since they may be from a different cloud.