Skip to content

ci: disable cloud metadata queries in integration tests - #666

Open
michael-richey wants to merge 1 commit into
mainfrom
ci/disable-cloud-metadata-in-integration-tests
Open

ci: disable cloud metadata queries in integration tests#666
michael-richey wants to merge 1 commit into
mainfrom
ci/disable-cloud-metadata-in-integration-tests

Conversation

@michael-richey

@michael-richey michael-richey commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Disable cloud-provider metadata endpoint queries in the integration-test Agent.
  • Leave the trace endpoint used by the tests unchanged.

What this prevents

During service-container teardown, the Agent currently prints a UUID-shaped Azure cloud account identifier twice in the public workflow log. Based on the GitHub-hosted runner environment and the Agent message, this is the Azure subscription/account ID for the runner infrastructure, not a Datadog customer identifier.

The identifier is not a credential, token, or secret and does not grant access by itself, so the direct security severity is low. It is nevertheless a real production-infrastructure identifier that the integration tests do not need to discover or publish.

GitHub publishes service-container output during job teardown. Configuring the Agent with an empty cloud-provider metadata list prevents the metadata endpoint query that produces this output; the tests only require the Agent trace endpoint.

Validation

  • git diff --check
  • No test changes; this is an Agent service configuration change.

@michael-richey
michael-richey marked this pull request as ready for review August 13, 2026 19:37
@michael-richey
michael-richey requested a review from a team as a code owner August 13, 2026 19:37
@michael-richey
michael-richey requested a lite review from Copilot August 13, 2026 19:37

Copilot AI left a comment

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.

Pull request overview

This PR updates the GitHub Actions integration-test workflow to prevent the Datadog Agent service container from querying cloud-provider metadata endpoints during test runs, reducing the risk of leaking host metadata via runner/service-container logs while keeping the APM trace intake path intact.

Changes:

  • Set DD_CLOUD_PROVIDER_METADATA to an empty list ("[]") for the datadog-agent service in the integration test workflow.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@michael-richey
michael-richey marked this pull request as draft August 13, 2026 19:43
@michael-richey
michael-richey marked this pull request as ready for review August 13, 2026 19:45
@michael-richey
michael-richey marked this pull request as draft August 13, 2026 19:52
@michael-richey
michael-richey marked this pull request as ready for review August 13, 2026 19:57
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