You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I noticed that the resource.WithBuiltinDetectors() option, which once included all the builtin detectors, has not been maintained. It could be removed, or it could be updated to actually name all the builtin detectors, which includes WithHost(),WithOSType(), and WithProcess() options which are not included in resource.Default().
Proposed Solution
Remove WithBuiltinDetectors().
Alternatives
Add WithHost(),WithOSType(), and WithProcess() to the result from WithBuiltinDetectors().
The text was updated successfully, but these errors were encountered:
My honor to take this issue, I'll do it as above said, add WithHost(), WithOSType(), WithProcess() instead of resource.WithBuiltinDetectors(). Could you please assign it to me?
Problem Statement
I noticed that the
resource.WithBuiltinDetectors()
option, which once included all the builtin detectors, has not been maintained. It could be removed, or it could be updated to actually name all the builtin detectors, which includesWithHost()
,WithOSType()
, andWithProcess()
options which are not included inresource.Default()
.Proposed Solution
Remove
WithBuiltinDetectors()
.Alternatives
Add
WithHost()
,WithOSType()
, andWithProcess()
to the result fromWithBuiltinDetectors()
.The text was updated successfully, but these errors were encountered: