fix(detectors): reduce diag level on detectors failing to detect (#2382)

* Squashed commit of the following: commit 1289068 Author: John Li <[email protected]> Date: Tue Aug 6 13:54:15 2024 +0800 fix: update detector log level Signed-off-by: John Li <[email protected]> Signed-off-by: John Li <[email protected]> * fix: reduce diag level on detectors Signed-off-by: John Li <[email protected]> --------- Signed-off-by: John Li <[email protected]>
open-telemetry · Aug 27, 2024 · d7a5bd4 · d7a5bd4
1 parent d30c85c
commit d7a5bd4
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/detectors/node/opentelemetry-resource-detector-aws/src/detectors/AwsEcsDetectorSync.ts b/detectors/node/opentelemetry-resource-detector-aws/src/detectors/AwsEcsDetectorSync.ts
@@ -137,7 +137,7 @@ export class AwsEcsDetectorSync implements DetectorSync {
         }
       }
     } catch (e) {
-      diag.warn('AwsEcsDetector failed to read container ID', e);
+      diag.debug('AwsEcsDetector failed to read container ID', e);
     }
 
     if (hostName || containerId) {

diff --git a/detectors/node/opentelemetry-resource-detector-aws/src/detectors/AwsEksDetectorSync.ts b/detectors/node/opentelemetry-resource-detector-aws/src/detectors/AwsEksDetectorSync.ts
@@ -95,7 +95,7 @@ export class AwsEksDetectorSync implements DetectorSync {
             [SEMRESATTRS_CONTAINER_ID]: containerId || '',
           };
     } catch (e) {
-      diag.warn('Process is not running on K8S', e);
+      diag.debug('Process is not running on K8S', e);
       return {};
     }
   }
@@ -138,7 +138,7 @@ export class AwsEksDetectorSync implements DetectorSync {
     try {
       return JSON.parse(response).data['cluster.name'];
     } catch (e) {
-      diag.warn('Cannot get cluster name on EKS', e);
+      diag.debug('Cannot get cluster name on EKS', e);
     }
     return '';
   }
@@ -154,7 +154,7 @@ export class AwsEksDetectorSync implements DetectorSync {
       );
       return 'Bearer ' + content;
     } catch (e) {
-      diag.warn('Unable to read Kubernetes client token.', e);
+      diag.debug('Unable to read Kubernetes client token.', e);
     }
     return '';
   }
@@ -189,7 +189,7 @@ export class AwsEksDetectorSync implements DetectorSync {
         }
       }
     } catch (e: any) {
-      diag.warn(`AwsEksDetector failed to read container ID: ${e.message}`);
+      diag.debug(`AwsEksDetector failed to read container ID: ${e.message}`);
     }
     return undefined;
   }

diff --git a/detectors/node/opentelemetry-resource-detector-container/src/detectors/ContainerDetector.ts b/detectors/node/opentelemetry-resource-detector-container/src/detectors/ContainerDetector.ts
@@ -62,7 +62,7 @@ export class ContainerDetector implements DetectorSync {
             [SEMRESATTRS_CONTAINER_ID]: containerId,
           };
     } catch (e) {
-      diag.info(
+      diag.debug(
         'Container Detector did not identify running inside a supported container, no container attributes will be added to resource: ',
         e
       );
@@ -128,7 +128,7 @@ export class ContainerDetector implements DetectorSync {
     } catch (e) {
       if (e instanceof Error) {
         const errorMessage = e.message;
-        diag.info(
+        diag.debug(
           'Container Detector failed to read the Container ID: ',
           errorMessage
         );