Skip to content

[SVLS-7937] add cloud run e2e tests#2332

Merged
ava-silver merged 2 commits into
masterfrom
ava.silver/svls-7937/add-cloud-run-integration-tests
May 22, 2026
Merged

[SVLS-7937] add cloud run e2e tests#2332
ava-silver merged 2 commits into
masterfrom
ava.silver/svls-7937/add-cloud-run-integration-tests

Conversation

@ava-silver

@ava-silver ava-silver commented May 22, 2026

Copy link
Copy Markdown
Contributor

What and why?

Adds Cloud Run e2e coverage for SVLS-7937 so datadog-ci validates the real instrument and uninstrument flow against an ephemeral Cloud Run service.

How?

  • Adds an Ubuntu Cloud Run e2e test that deploys a temporary service from the public Google hello image, instruments it, verifies the service config, uninstrument it, verifies cleanup, and deletes the service.
  • Adds a Cloud Run verifier for the Datadog sidecar, serverless-init image, shared volume, required DD environment variables, and instrumentation labels.
  • Gates Cloud Run e2e execution behind a Cloud Run path filter and conditionally configures Google Cloud OIDC plus gcloud in CI.
  • Skips Cloud Run e2e on Windows and standalone binary runs.
  • Extends e2e helpers with transient GCP retry patterns and quoted e2e/.env.local value parsing.
  • Adds CODEOWNERS coverage for the new Cloud Run e2e files.

Test plan

  • Manual Cloud Run e2e with GCP_PROJECT_ID, GCP_REGION, and DATADOG_CI_COMMAND configured: yarn jest --config jest.config-e2e.js e2e/cloud-run.test.ts --colors

Review checklist

  • Feature or bugfix MUST have appropriate tests (unit, integration)

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@datadog-official

This comment has been minimized.

@ava-silver
ava-silver marked this pull request as ready for review May 22, 2026 15:22
@ava-silver ava-silver added the serverless Related to [aas, cloud-run, lambda, stepfunctions] label May 22, 2026
@ava-silver
ava-silver requested review from a team as code owners May 22, 2026 15:22
@ava-silver
ava-silver requested review from a team and TalUsvyatsky May 22, 2026 17:31
@ava-silver
ava-silver merged commit 96587fd into master May 22, 2026
32 checks passed
@ava-silver
ava-silver deleted the ava.silver/svls-7937/add-cloud-run-integration-tests branch May 22, 2026 19:07
@afontan afontan mentioned this pull request May 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

serverless Related to [aas, cloud-run, lambda, stepfunctions]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants