Skip to content

docs: add an OTel Collector setup best-practices guide - #663

Merged
Jeff Luo (JeffLuoo) merged 1 commit into
agent-substrate:mainfrom
JeffLuoo:docs/otel-collector-setup
Jul 31, 2026
Merged

Jeff Luo (JeffLuoo) merged 1 commit into
agent-substrate:mainfrom
JeffLuoo:docs/otel-collector-setup

Conversation

@JeffLuoo

@JeffLuoo Jeff Luo (JeffLuoo) commented Jul 31, 2026 •

Copy link
Copy Markdown
Collaborator

Substrate documents how to instrument a service but not how to stand up the collector it exports to, so pointing it at your own collector meant reading six manifests.

Covers both topologies: GKE Managed OpenTelemetry as the default on GKE, with why its gateway Deployment suits Substrate's elastic actor workload, and a DaemonSet manifest for operators who need per-node isolation, full config control, or a non-GKE cluster.

Part of #563.

It's a good idea to open an issue first for discussion.

  • Tests pass
  • Appropriate changes to documentation are included in the PR

@JeffLuoo
Jeff Luo (JeffLuoo) force-pushed the docs/otel-collector-setup branch 4 times, most recently from e2dafe3 to f0872df Compare July 31, 2026 17:32
Substrate documents how to instrument a service but not how to stand up the
collector it exports to, so pointing it at your own collector meant reading
six manifests.

Covers both topologies: GKE Managed OpenTelemetry as the default on GKE,
with why its gateway Deployment suits Substrate's elastic actor workload,
and a DaemonSet manifest for operators who need per-node isolation, full
config control, or a non-GKE cluster.

Also records the two constraints from agent-substrate#563: the compiled-in gRPC exporters
require port 4317, not the 4318 GKE's docs advertise, and WithInsecure()
makes https:// endpoints silently plaintext.

Part of agent-substrate#563.
@JeffLuoo
Jeff Luo (JeffLuoo) merged commit b255df3 into agent-substrate:main Jul 31, 2026
11 checks passed
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