Skip to content

feat(datadog): expose agents.containers.otelAgent.syncDelay to configure otel-agent startup sync timeout - #2858

Open
gb8may wants to merge 3 commits into
DataDog:mainfrom
gb8may:expose-otel-agent-sync-delay
Open

feat(datadog): expose agents.containers.otelAgent.syncDelay to configure otel-agent startup sync timeout#2858
gb8may wants to merge 3 commits into
DataDog:mainfrom
gb8may:expose-otel-agent-sync-delay

Conversation

@gb8may

@gb8may gb8may commented Aug 11, 2026

Copy link
Copy Markdown

What this PR does / why we need it:

What

Adds agents.containers.otelAgent.syncDelay (default: 30s) to control
how long otel-agent waits for the core agent to be ready on startup,
replacing the previously hardcoded --sync-delay=30s flag on both Linux
and Windows.

Why

datadog/agent:7.81.1 (shipped with chart >=3.234.0) initializes slower
than previous versions. On fresh node starts with otelCollector.enabled: true,
otel-agent times out waiting for the core agent and enters CrashLoopBackOff
before the core agent is ready to accept connections.

The --sync-delay flag already exists in the binary but was not exposed as a
chart value, leaving users no way to increase it without a post-render patch.

How to use

agents:
  containers:
    otelAgent:
      syncDelay: 60s  # increase from default 30s for agent >= 7.81.1

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

  • fixes #

Special notes for your reviewer:

  • Default remains 30s — fully backwards compatible
  • Applies to both Linux and Windows target systems
  • Verified fix on Kubernetes / AWS EKS with chart 3.235.0

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • All commits are signed and show as "Verified" on GitHub (see: signing commits)
  • Chart Version semver bump label has been added (use <chartName>/minor-version, <chartName>/patch-version, or <chartName>/no-version-bump)
  • For datadog or datadog-operator chart or value changes, update the test baselines (run: make update-test-baselines)
  • For datadog chart changes, received ✅ from a member of your team

GitHub CI takes care of the below, but are still required:

  • Documentation has been updated with helm-docs (run: .github/helm-docs.sh)
  • CHANGELOG.md has been updated
  • Variables are documented in the README.md

@gb8may
gb8may requested review from a team as code owners August 11, 2026 20:44
@gb8may
gb8may requested review from gpalmz and removed request for a team August 11, 2026 20:44

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: fbc7a8ceca

ℹ️ 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".

Comment thread charts/datadog/templates/_container-otel-agent.yaml
Comment thread charts/datadog/values.yaml
Mayara Gouveia and others added 2 commits August 12, 2026 15:21
@gb8may
gb8may force-pushed the expose-otel-agent-sync-delay branch from 3161cbe to 4f12889 Compare August 12, 2026 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants