Skip to content

Conversation

@estherk15
Copy link
Contributor

What does this PR do? What is the motivation?

  • Add more context in the Overview
  • Update instruction to create pipelines with Agent source and CloudPrem destination
  • Update env variables in Helm command
  • Update Datadog Agent env variables

Merge instructions

Merge readiness:

  • Pending PM approval

@estherk15 estherk15 requested a review from fmassot December 11, 2025 20:19
@estherk15 estherk15 requested a review from a team as a code owner December 11, 2025 20:19
1. Select the [**Datadog CloudPrem** destination][4] to forward logs to your CloudPrem instance. Leave the configuration empty.

{{< img src="/cloudprem/ingest/observability-pipelines-cloudprem-setup.png" alt="Screenshot of the Logs Explorer interface showing how to filter logs by selecting the cloudprem index in the facets panel" style="width:80%;" >}}
<!-- This image shows an example with dual shipping when the instructions say log volume control -->
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@fmassot Could I remove this image?

@github-actions
Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

--set datadog.apiKey=XXXXXXX \
--set datadog.pipelineId=XXXXXXX \
--set env[0].name=DD_OP_SOURCE_HTTP_SERVER_ADDRESS,env[0].value='0.0.0.0:8282' \
--set env[1].name=DD_OP_DESTINATION_HTTP_CLIENT_URI,env[1].value='http://<RELEASE_NAME>-indexer.<NAMESPACE_NAME>.svc.cluster.local:7280/api/v2/datadog' \
Copy link
Contributor Author

@estherk15 estherk15 Dec 11, 2025

Choose a reason for hiding this comment

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

I wasn't sure if this needs to be removed or not? The CloudPrem destination docs state that

The Worker appends /api/v2/logs and /api/v1/validate to the endpoint URL, so these endpoints must be allowed if you are using forwarding or firewall rules.

@cswatt cswatt self-assigned this Dec 11, 2025
Copy link
Contributor

@cswatt cswatt left a comment

Choose a reason for hiding this comment

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

This mostly LGTM so I'm approving for now, but noting that there's ongoing conversations open


{{< img src="/cloudprem/ingest/observability-pipelines-cloudprem-setup.png" alt="Screenshot of the Logs Explorer interface showing how to filter logs by selecting the cloudprem index in the facets panel" style="width:80%;" >}}
<!-- This image shows an example with dual shipping when the instructions say log volume control -->
<!-- {{< img src="/cloudprem/ingest/observability-pipelines-cloudprem-setup.png" alt="Screenshot of the Logs Explorer interface showing how to filter logs by selecting the cloudprem index in the facets panel" style="width:80%;" >}} -->
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
<!-- {{< img src="/cloudprem/ingest/observability-pipelines-cloudprem-setup.png" alt="Screenshot of the Logs Explorer interface showing how to filter logs by selecting the cloudprem index in the facets panel" style="width:80%;" >}} -->

@estherk15 estherk15 merged commit 06ffa93 into master Jan 8, 2026
28 of 32 checks passed
@estherk15 estherk15 deleted the esther/cloudprem-update-observability-pipelines branch January 8, 2026 18:04
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.

3 participants