Skip to content

Add service-route observability - #635

Merged
scotwells merged 4 commits into
mainfrom
feat/service-route-observability
Oct 7, 2026
Merged

scotwells merged 4 commits into
mainfrom
feat/service-route-observability

Conversation

@scotwells

@scotwells scotwells commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Add bounded-cardinality observability for the node-local private-service controller and TC-eBPF dataplane introduced by #634.

  • Record reconciliation outcomes and duration.
  • Record eBPF apply/remove outcomes and the number of attachment-scoped paths currently tracked.
  • Emit structured policy, endpoint, node, attachment-count, dependency, and programming failure context in logs.
  • Export service_route_denied and service_route_redirect_failed without exposing diagnostic trace slots as metrics.
  • Scrape galactic-cni port 9180 with a PodMonitor.
  • Keep resource names out of metric labels.

Validation

go test ./internal/plumbing/ebpf/prog ./internal/plumbing/ebpf/serviceroutemap ./internal/plumbing/ebpf/attach ./internal/plumbing/ebpf/metrics ./internal/serviceroute ./internal/controller ./cmd/galactic-router -count=1
go vet ./internal/plumbing/ebpf/metrics ./internal/serviceroute ./internal/controller ./cmd/galactic-router

@scotwells
scotwells force-pushed the feat/service-route-local-attachments branch from 9b8d7ff to 5d4c848 Compare October 6, 2026 17:09
@scotwells
scotwells force-pushed the feat/service-route-observability branch 6 times, most recently from c4c21ec to bd5b5f6 Compare October 6, 2026 20:55
@scotwells
scotwells force-pushed the feat/service-route-local-attachments branch from d49a7cd to e8c8158 Compare October 7, 2026 13:40
@scotwells
scotwells force-pushed the feat/service-route-observability branch 5 times, most recently from a032bf0 to dad6fb7 Compare October 7, 2026 16:25
@scotwells
scotwells force-pushed the feat/service-route-observability branch from dad6fb7 to f10c9fa Compare October 7, 2026 16:31
privateip
privateip previously approved these changes Oct 7, 2026
Base automatically changed from feat/service-route-local-attachments to main October 7, 2026 19:58
@scotwells
scotwells dismissed privateip’s stale review October 7, 2026 19:58

The base branch was changed.

@scotwells
scotwells marked this pull request as ready for review October 7, 2026 19:58
@scotwells
scotwells requested a review from a team as a code owner October 7, 2026 19:59
@scotwells
scotwells merged commit 1da3b6b into main Oct 7, 2026
14 checks passed
@scotwells
scotwells deleted the feat/service-route-observability branch October 7, 2026 19:59
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