-
Notifications
You must be signed in to change notification settings - Fork 238
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
Include AWS Resource Detectors by Default #62
Comments
Hi William, looked the #32 , I think we need to fix the following things before we can merge it into the collector,
|
I closed #32, I think a larger change is going to be necessary based on your comments. We also have not yet added many of the translation capabilities to convert AWS resource attributes to fields in X-Ray segments. I think once open-telemetry/opentelemetry-collector-contrib#1360 is merged along with a corresponding data model translation PR, I'll circle back to this and the integ test changes. |
…ty#62) * add data generator support for otlp trace and xray trace * remove debug logs * remove the unused commented code * Integrate high load emitter with soaking test * Integrate soaking test with load generator * fix some naming issue * fix some naming issue * fix the port problem for xray soaking test * add soaking test cases for otlp and xray data models * add soaking test cases for otlp and xray data models * refactor soaking test cases * remove some unneccessary vars
It's already included
|
We should modify our vended
config.yaml
to include the all AWS-specific resource detectors on the collector. #32 is a first attempt at this, but it is still TBD whether we should include theenv
resource detector. Eventually, we should have detectors for:The text was updated successfully, but these errors were encountered: