Skip to content

Fix Cloud Run sample: canonical collector names, deploy command, service account - #558

Merged
seanbollin merged 1 commit into
mainfrom
fix-cloud-run-collector-config
Oct 6, 2026
Merged

seanbollin merged 1 commit into
mainfrom
fix-cloud-run-collector-config

Conversation

@seanbollin

@seanbollin seanbollin commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Fixes found via an end-to-end Cloud Run deploy test:

  • Collector config: use canonical otelcol-google component names — rename the resourcedetection processor to resource_detection (the deprecated alias logs runtime warnings). The receiver stays otlp and the exporter stays otlp_grpc; those are already the canonical names for this distribution (there is no otlp_grpc receiver, and no plain otlp exporter). Verified with otelcol-google:0.156.0 validate (exit 0).
  • Deploy command: drop the invalid --region flag from gcloud beta run worker-pools replace (region comes from the cloud.googleapis.com/location label in the YAML).
  • Worker pool: add serviceAccountName: <SERVICE_ACCOUNT>, and document in the README that it needs roles/monitoring.metricWriter, roles/telemetry.tracesWriter, and roles/secretmanager.secretAccessor.
  • Starter: drop the trailing ! from the workflow input so the result is Hello, Cloud Run Worker! (was Hello, Cloud Run Worker!!), matching the README.

@seanbollin
seanbollin requested a review from a team as a code owner October 6, 2026 20:59
@seanbollin
seanbollin force-pushed the fix-cloud-run-collector-config branch from 6e3c340 to 5b405ab Compare October 6, 2026 22:39
@seanbollin seanbollin changed the title Fix invalid OTLP exporter name in Cloud Run collector config Fix Cloud Run sample: canonical collector names, deploy command, service account Oct 6, 2026
@seanbollin
seanbollin merged commit 402a3bc into main Oct 6, 2026
4 checks passed
@seanbollin
seanbollin deleted the fix-cloud-run-collector-config branch October 6, 2026 23:02
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