making k8sattribute processor default for logs and metrics

Author: samiura

.changelog/3976.changed.txt

@@ -0,0 +1 @@
+change: use upstream k8sattributeprocessor as default for logs and metrics

deploy/helm/sumologic/README.md

@@ -464,7 +464,7 @@ The following table lists the configurable parameters of the Sumo Logic chart an
 | `metadata.metrics.enabled`                                                            | Flag to control deploying the otelcol metrics statefulsets.                                                                                                                                                                                                                                                | `true`                                                                                                                                                                               |
 | `metadata.metrics.logLevel`                                                           | Flag to control logging level for OpenTelemetry Collector for metrics. Can be `debug`, `info`, `warn`, `error`, `dpanic`, `panic`, `fatal`.                                                                                                                                                                | `info`                                                                                                                                                                               |
 | `metadata.metrics.metricsLevel`                                                       | Metrics level for the OpenTelemetry Collector. Can be `basic`, `normal`, `detailed`.                                                                                                                                                                                                                       | `normal`                                                                                                                                                                             |
-| `metadata.metrics.useSumoK8sProcessor`                                                | Use the SumoLogic K8s processor                                                                                                                                                                                                                                                                            | `true`                                                                                                                                                                               |
+| `metadata.metrics.useSumoK8sProcessor`                                                | Use the SumoLogic K8s processor                                                                                                                                                                                                                                                                            | `false`                                                                                                                                                                              |
 | `metadata.metrics.waitForMetadata`                                                    | Wait for Metadata before processing the data                                                                                                                                                                                                                                                               | `true`                                                                                                                                                                               |
 | `metadata.metrics.waitForMetadataTimeout`                                             | Wait for Metadata timeout                                                                                                                                                                                                                                                                                  | `10s`                                                                                                                                                                                |
 | `metadata.metrics.config.merge`                                                       | Configuration for metrics metadata otelcol, merged with defaults. See also https://github.com/SumoLogic/sumologic-otel-collector/blob/main/docs/configuration.md.                                                                                                                                          | {}                                                                                                                                                                                   |
@@ -497,7 +497,7 @@ The following table lists the configurable parameters of the Sumo Logic chart an
 | `metadata.logs.enabled`                                                               | Flag to control deploying the otelcol logs statefulsets.                                                                                                                                                                                                                                                   | `true`                                                                                                                                                                               |
 | `metadata.logs.logLevel`                                                              | Flag to control logging level for OpenTelemetry Collector for logs. Can be `debug`, `info`, `warn`, `error`, `dpanic`, `panic`, `fatal`.                                                                                                                                                                   | `info`                                                                                                                                                                               |
 | `metadata.logs.metricsLevel`                                                          | Metrics level for the OpenTelemetry Collector. Can be `basic`, `normal`, `detailed`.                                                                                                                                                                                                                       | `normal`                                                                                                                                                                             |
-| `metadata.logs.useSumoK8sProcessor`                                                   | Use the SumoLogic K8s processor                                                                                                                                                                                                                                                                            | `true`                                                                                                                                                                               |
+| `metadata.logs.useSumoK8sProcessor`                                                   | Use the SumoLogic K8s processor                                                                                                                                                                                                                                                                            | `false`                                                                                                                                                                              |
 | `metadata.logs.waitForMetadata`                                                       | Wait for Metadata before processing the logs                                                                                                                                                                                                                                                               | `true`                                                                                                                                                                               |
 | `metadata.logs.waitForMetadataTimeout`                                                | Wait for Metadata timeout                                                                                                                                                                                                                                                                                  | `10s`                                                                                                                                                                                |
 | `metadata.logs.config.merge`                                                          | Configuration for logs metadata otelcol, merged with defaults. See also https://github.com/SumoLogic/sumologic-otel-collector/blob/main/docs/configuration.md.                                                                                                                                             | {}                                                                                                                                                                                   |

deploy/helm/sumologic/values.yaml

@@ -1603,7 +1603,7 @@ metadata:
 
     ## Use the sumologic k8s processor to add metadata to the logs.
     ## This is enabled by default, but can be disabled if you want to use the upstream processor.
-    useSumoK8sProcessor: true
+    useSumoK8sProcessor: false
 
     ## Only used if useSumoK8sProcessor is set to false
     waitForMetadata: true
@@ -1720,7 +1720,7 @@ metadata:
 
     ## Use the sumologic k8s processor to add metadata to the logs.
     ## This is enabled by default, but can be disabled if you want to use the upstream processor.
-    useSumoK8sProcessor: true
+    useSumoK8sProcessor: false
 
     ## Only used if useSumoK8sProcessor is set to false
     waitForMetadata: true

docs/README.md

@@ -119,8 +119,9 @@ The following table displays the currently used software versions for our Helm c
 
 The
 [k8sattributesprocessor](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/processor/k8sattributesprocessor/README.md)
-is now supported for logs and metrics metadata enrichment. This processor is disabled by default. To enable this processor for logs, set
-`metadata.logs.useSumoK8sProcessor` to `false`. To enable this processor for metrics, set `metadata.metrics.useSumoK8sProcessor` to `false`.
+is now supported for logs and metrics metadata enrichment. This processor is enabled by default. To disable this processor for logs, set
+`metadata.logs.useSumoK8sProcessor` to `true`. To disable this processor for metrics, set `metadata.metrics.useSumoK8sProcessor` to `true`.
+If disabled then SumoK8sProcessor will be the default automatically.
 
 > [!NOTE] The Service name isn't part of the metadata enrichment with this new processor.