Skip to content

fakeintake: add --rc-key-data flag for ephemeral-env RC signing keys#51148

Draft
KevinFairise2 wants to merge 1 commit into
mainfrom
kfairise/fakeintake-rc-key-data
Draft

fakeintake: add --rc-key-data flag for ephemeral-env RC signing keys#51148
KevinFairise2 wants to merge 1 commit into
mainfrom
kfairise/fakeintake-rc-key-data

Conversation

@KevinFairise2
Copy link
Copy Markdown
Member

Adds a --rc-key-data CLI flag (and WithRemoteConfigKeyData option) that accepts a hex-encoded 32-byte ed25519 seed. When set, the key is never written to disk and takes precedence over --rc-key-path. This lets ephemeral environments (e.g. ECS Fargate) supply a fixed, pre-known signing key at provisioning time so the agent's config_root/director_root can be baked into its config upfront.

What does this PR do?

Motivation

Describe how you validated your changes

Additional Notes

Adds a `--rc-key-data` CLI flag (and `WithRemoteConfigKeyData` option)
that accepts a hex-encoded 32-byte ed25519 seed. When set, the key is
never written to disk and takes precedence over `--rc-key-path`. This
lets ephemeral environments (e.g. ECS Fargate) supply a fixed, pre-known
signing key at provisioning time so the agent's config_root/director_root
can be baked into its config upfront.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

@codex review

@dd-octo-sts dd-octo-sts Bot added internal Identify a non-fork PR team/agent-devx labels May 21, 2026
@github-actions github-actions Bot added the short review PR is simple enough to be reviewed quickly label May 21, 2026
@datadog-prod-us1-3
Copy link
Copy Markdown
Contributor

datadog-prod-us1-3 Bot commented May 21, 2026

Pipelines

Fix all issues with BitsAI

⚠️ Warnings

🚦 3 Pipeline jobs failed

DataDog/datadog-agent | oracle: [21.3.0-xe]   View in Datadog   GitLab

🛟 This job is unlikely to succeed on retry. Please review your pipeline configuration. Failed to ping oracle instance: ORA-12514: TNS:listener does not currently know of service requested in connect descriptor.

Label analysis | release-note-check   View in Datadog   GitHub Actions

🛟 This job is unlikely to succeed on retry. Please review your pipeline configuration. Release note not found for this PR. Please add one using 'reno' or apply the label 'changelog/no-changelog'.

Label analysis | skip-qa-check   View in Datadog   GitHub Actions

🛟 This job is unlikely to succeed on retry. Please review your pipeline configuration. Missing required QA label. Label must be set to 'qa/no-code-change', 'qa/done', or 'qa/rc-required'.

ℹ️ Info

🎯 Code Coverage (details)
Patch Coverage: 27.59%
Overall Coverage: 50.29% (-0.04%)

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: e727fce | Docs | Datadog PR Page | Give us feedback!

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Chef's kiss.

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@dd-octo-sts
Copy link
Copy Markdown
Contributor

dd-octo-sts Bot commented May 21, 2026

Files inventory check summary

File checks results against ancestor 2785003b:

Results for datadog-agent_7.81.0~devel.git.50.e727fce.pipeline.114421701-1_amd64.deb:

No change detected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Identify a non-fork PR short review PR is simple enough to be reviewed quickly team/agent-devx

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant