From 2891f7b72037251b0b3985306201c307301034d6 Mon Sep 17 00:00:00 2001 From: jackgopack4 Date: Fri, 10 Oct 2025 09:45:56 -0400 Subject: [PATCH 1/2] [pkg/datadog] remove previously deprecated logs::dump_payloads --- ...tadogexporter-logs-dump_payloads copy.yaml | 27 +++++++++++++++++++ ...remove-pkg-datadog-logs-dump_payloads.yaml | 27 +++++++++++++++++++ exporter/datadogexporter/factory.go | 4 --- pkg/datadog/config/logs.go | 5 ---- 4 files changed, 54 insertions(+), 9 deletions(-) create mode 100644 .chloggen/jackgopack4-remove-datadogexporter-logs-dump_payloads copy.yaml create mode 100644 .chloggen/jackgopack4-remove-pkg-datadog-logs-dump_payloads.yaml diff --git a/.chloggen/jackgopack4-remove-datadogexporter-logs-dump_payloads copy.yaml b/.chloggen/jackgopack4-remove-datadogexporter-logs-dump_payloads copy.yaml new file mode 100644 index 0000000000000..8e1c15791741f --- /dev/null +++ b/.chloggen/jackgopack4-remove-datadogexporter-logs-dump_payloads copy.yaml @@ -0,0 +1,27 @@ +# Use this changelog template to create an entry for release notes. + +# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix' +change_type: breaking + +# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver) +component: exporter/datadogexporter + +# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`). +note: Remove `logs::dump_payloads` config option from `datadogexporter` config. + +# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists. +issues: [43427] + +# (Optional) One or more lines of additional information to render under the primary note. +# These lines will be padded with 2 spaces and then inserted directly into the document. +# Use pipe (|) for multiline entries. +subtext: Please remove the previously deprecated `logs::dump_payloads` config option from your `datadogexporter` config. + +# If your change doesn't affect end users or the exported elements of any package, +# you should instead start your pull request title with [chore] or use the "Skip Changelog" label. +# Optional: The change log or logs in which this entry should be included. +# e.g. '[user]' or '[user, api]' +# Include 'user' if the change is relevant to end users. +# Include 'api' if there is a change to a library API. +# Default: '[user]' +change_logs: [user] diff --git a/.chloggen/jackgopack4-remove-pkg-datadog-logs-dump_payloads.yaml b/.chloggen/jackgopack4-remove-pkg-datadog-logs-dump_payloads.yaml new file mode 100644 index 0000000000000..4c3da65fd2c80 --- /dev/null +++ b/.chloggen/jackgopack4-remove-pkg-datadog-logs-dump_payloads.yaml @@ -0,0 +1,27 @@ +# Use this changelog template to create an entry for release notes. + +# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix' +change_type: deprecation + +# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver) +component: pkg/datadog + +# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`). +note: Remove `logs::dump_payloads` config option from `pkg/datadog` config. + +# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists. +issues: [43427] + +# (Optional) One or more lines of additional information to render under the primary note. +# These lines will be padded with 2 spaces and then inserted directly into the document. +# Use pipe (|) for multiline entries. +subtext: + +# If your change doesn't affect end users or the exported elements of any package, +# you should instead start your pull request title with [chore] or use the "Skip Changelog" label. +# Optional: The change log or logs in which this entry should be included. +# e.g. '[user]' or '[user, api]' +# Include 'user' if the change is relevant to end users. +# Include 'api' if there is a change to a library API. +# Default: '[user]' +change_logs: [api] diff --git a/exporter/datadogexporter/factory.go b/exporter/datadogexporter/factory.go index 772ec33978ec9..b3a1feeda3d3c 100644 --- a/exporter/datadogexporter/factory.go +++ b/exporter/datadogexporter/factory.go @@ -522,10 +522,6 @@ func (f *factory) createLogsExporter( ) (exporter.Logs, error) { cfg := checkAndCastConfig(c, set.Logger) - if cfg.Logs.DumpPayloads { - set.Logger.Warn("logs::dump_payloads is not valid") - } - var pusher consumer.ConsumeLogsFunc var logsAgent logsagentpipeline.LogsAgent hostProvider, err := f.SourceProvider(set.TelemetrySettings, cfg.Hostname, cfg.HostnameDetectionTimeout) diff --git a/pkg/datadog/config/logs.go b/pkg/datadog/config/logs.go index b1d4cf5a577b2..0a8e760e99277 100644 --- a/pkg/datadog/config/logs.go +++ b/pkg/datadog/config/logs.go @@ -11,11 +11,6 @@ type LogsConfig struct { // If unset, the value is obtained from the Site. confignet.TCPAddrConfig `mapstructure:",squash"` - // DumpPayloads report whether payloads should be dumped when logging level is debug. - // Note: this config option does not apply when the `exporter.datadogexporter.UseLogsAgentExporter` feature flag is enabled (now enabled by default). - // Deprecated: This config option is not supported in the Datadog Agent logs pipeline. - DumpPayloads bool `mapstructure:"dump_payloads"` - // UseCompression enables the logs agent to compress logs before sending them. // Note: this config option does not apply when the `exporter.datadogexporter.UseLogsAgentExporter` feature flag is disabled. UseCompression bool `mapstructure:"use_compression"` From 22e7a0ba4419f11307a456f69adbf27606b82888 Mon Sep 17 00:00:00 2001 From: jackgopack4 Date: Fri, 10 Oct 2025 09:51:07 -0400 Subject: [PATCH 2/2] [chore] fix chloggen --- ...kgopack4-remove-datadogexporter-logs-dump_payloads copy.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.chloggen/jackgopack4-remove-datadogexporter-logs-dump_payloads copy.yaml b/.chloggen/jackgopack4-remove-datadogexporter-logs-dump_payloads copy.yaml index 8e1c15791741f..e607da09a6c1d 100644 --- a/.chloggen/jackgopack4-remove-datadogexporter-logs-dump_payloads copy.yaml +++ b/.chloggen/jackgopack4-remove-datadogexporter-logs-dump_payloads copy.yaml @@ -4,7 +4,7 @@ change_type: breaking # The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver) -component: exporter/datadogexporter +component: exporter/datadog # A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`). note: Remove `logs::dump_payloads` config option from `datadogexporter` config.