Skip to content

Add note about env var - #39010

Merged
maycmlee merged 2 commits into
masterfrom
may/splunk-tcp-new-en-var
Aug 7, 2026
Merged

Add note about env var#39010
maycmlee merged 2 commits into
masterfrom
may/splunk-tcp-new-en-var

Conversation

@maycmlee

@maycmlee maycmlee commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

Merge readiness

  • Ready for merge

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

AI assistance

Additional notes

@maycmlee
maycmlee requested a review from a team as a code owner August 7, 2026 16:36
{{% observability_pipelines/secrets_env_var_note %}}
**Notes**:
- By default, the Splunk TCP source doesn't limit the size of an event. To prevent unbounded memory consumption, such as from malformed connections or connections that remain open indefinitely, use the environment variable `DD_OP_SPLUNK_TCP_MAX_FRAME_LENGTH` to set a maximum frame length in bytes.
- If you enter secret identifiers and then choose to use environment variables, the environment variable is the identifier entered and prepended with `DD_OP_`. For example, if you entered <code>PASSWORD_1</code> for a password identifier, the environment variable for that password is `DD_OP_PASSWORD_1`.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was secret identifiers something we wanted to add as well or just adding because of environement variable? In this case its not meant to be added as a secret identifier so those notes would be unrelated

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DM'd. They are unrelated. I just pulled the secrets note out of the shortcode.

@jhgilbert jhgilbert added the astro-reorg-no-conflicts Needs manual conflict resolution after replatforming reorg label Aug 7, 2026
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@evazorro evazorro self-assigned this Aug 7, 2026
@evazorro evazorro removed their assignment Aug 7, 2026
@maycmlee
maycmlee merged commit 251d056 into master Aug 7, 2026
29 of 31 checks passed
@maycmlee
maycmlee deleted the may/splunk-tcp-new-en-var branch August 7, 2026 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astro-reorg-no-conflicts Needs manual conflict resolution after replatforming reorg

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants