resource-detector-aws, resource-detector-gcp

[obecny]

Currently the package @opentelemetry/sdk-node from main repo depends on 2 packages from contrib repo

• @opentelemetry/resource-detector-aws
• @opentelemetry/resource-detector-gcp

Those packages should be either moved to main repo or we should remove the dependency on them from @opentelemetry/sdk-node

both should work fine

[legendecas]

I'd find installing potentially unused detectors in every environment very unnecessary.

[github-actions]

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days.

[legendecas]

#2698 can be one of the reasons that those detectors should not be static dependencies of sdk-node as they may load arbitrary vendor dependencies like gcp-metadata and significantly increase the load time of the sdk-node (gcp-metadata takes up ~20ms out of ~140ms sdk-node load time).

[dyladan]

#2698 can be one of the reasons that those detectors should not be static dependencies of sdk-node as they may load arbitrary vendor dependencies like gcp-metadata and significantly increase the load time of the sdk-node (gcp-metadata takes up ~20ms out of ~140ms sdk-node load time).

At the time this issue was created we discussed it in SIG and agreed to remove the dependency from sdk-node we just never got around to actually making the change.

[pichlermarc]

Closing this as we've since removed those. #3024