Repository navigation
Add lane rates, estimated costs and gateway routing panels - #851
Merged
seathatflowsinourveins merged 1 commit intoOct 8, 2026
Merged
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
seathatflowsinourveins
force-pushed
the
codex/ns2604-row9-observability-20261008
branch
from
October 8, 2026 09:31
0078ccb to
66a7228
Compare
seathatflowsinourveins
deleted the
codex/ns2604-row9-observability-20261008
branch
October 8, 2026 10:02
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Scope
Add per-lane tool/API rates, separate estimated Claude/Codex costs, and gateway provider/model/outcome/HTTP-status panels to the existing cc-lanes dashboard. #848 already supplies the five collector dimensions on main; this PR has no collector YAML delta after its landing rebase. Host publication remains the CC's operation.
e450e378c6bf1b21dd0bb565812f84afe028bad5(CC091807Z landing rebase).lane:foundation.observability/lanes_dashboard.py; generated planconfig/grafana-lanes.json; the existing writer-identity test fixture; dated decision; own evidence registry entries.Pipeline exactness
Dropped the old #851
traces.span.metrics.*statements fromtransform/privacy. They matched only direct OTLP metrics namedtraces.span.metrics.calls/durationentering themetricspipeline. Gateway/Harbor connector output instead usesmetrics/spansandtransform/metric_export_privacy; it never passes through those statements. No reviewed shipped producer is bound to sending connector-named metrics through that direct OTLP metrics route.Main's collector YAML is retained byte-for-byte, including #848's five-dimension flow-form line and comment. Its existing export guard retains provider/model/status/outcome. The synthetic test follows that actual connector pipeline and proves dimension export, not causality from the dropped exceptions.
SOTA sources
traces.span.metrics.calls/durationnames. Main's processor/pipeline source establishes the exact routing and retained dimensions.gen_ai.provider.name; current conventions are Development.omniroute.routing.status/outcome, provider/model fields and absent lane/session parentage. Routing status is distinct from OTelstatus.code.Evidence-class table
observability_config.py otelinto a temporary destination, then nativeotelcol-contrib validateLocal commands run
Decision record
docs/decisions/2026-10-08-cc-lanes-rate-cost-gateway.mdrecords the corrected processor assumption.Costs are estimates with source-specific coverage. Codex provider enrichment is conditional;
increase()is not an exact total or guaranteed lower bound. No zero fill is used for missing costs or gateway routing data. Gateway events lack lane/session parentage and are not assigned to client lanes.Host evidence
No files under
evidence/hosts/change, and no deployment acceptance is claimed. The existing native collector integration uses temporary configuration and loopback ports with synthetic input. The runtime lock, broker keys, live Grafana, services and gateway are untouched.Checklist