Skip to content

feat: add --destination-logs-intake-url flag#579

Merged
michael-richey merged 1 commit into
mainfrom
hamr/logs-pipelines-intake-url
May 26, 2026
Merged

feat: add --destination-logs-intake-url flag#579
michael-richey merged 1 commit into
mainfrom
hamr/logs-pipelines-intake-url

Conversation

@michael-richey

@michael-richey michael-richey commented May 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Adds --destination-logs-intake-url CLI flag (and DD_DESTINATION_LOGS_INTAKE_URL env var) to override the logs intake endpoint used when triggering integration pipeline creation
  • When set, post_unauthenticated is used (clean session, no auth headers) to POST to the supplied URL directly, bypassing the http-intake.logs.* subdomain construction that fails for internal haproxy hosts
  • When unset, existing subdomain-construction behavior is unchanged
  • Custom pipelines (is_read_only=False) are unaffected

Context

Integration pipelines fail to sync when the destination API URL is an internal haproxy URL because sync-cli constructs the intake URL by prepending http-intake.logs. to the API hostname

Test plan

  • tests/unit/test_logs_pipelines.py — 7 new unit tests (3 red/green + 4 green/green), all passing

🤖 Generated with Claude Code

…endpoint

Integration pipelines (is_read_only=True) fail to sync when the destination
API URL is an internal haproxy host because the constructed http-intake.logs.*
subdomain doesn't resolve. The new flag lets callers supply the intake URL
directly; managed-sync will inject it using the org's EvP endpoint.

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
@datadog-official

This comment has been minimized.

@michael-richey
michael-richey marked this pull request as ready for review May 26, 2026 15:59
@michael-richey
michael-richey requested a review from a team as a code owner May 26, 2026 15:59
@michael-richey
michael-richey merged commit 346a3d2 into main May 26, 2026
23 of 26 checks passed
@michael-richey
michael-richey deleted the hamr/logs-pipelines-intake-url branch May 26, 2026 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants