Skip to content

Add lane rates, estimated costs and gateway routing panels - #851

Merged
seathatflowsinourveins merged 1 commit into
mainfrom
codex/ns2604-row9-observability-20261008
Oct 8, 2026
Merged

seathatflowsinourveins merged 1 commit into
mainfrom
codex/ns2604-row9-observability-20261008

Conversation

@seathatflowsinourveins

@seathatflowsinourveins seathatflowsinourveins commented Oct 8, 2026 •

Copy link
Copy Markdown
Collaborator

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.

  • Base commit: e450e378c6bf1b21dd0bb565812f84afe028bad5 (CC091807Z landing rebase).
  • Lane: lane:foundation.
  • Owned paths: observability/lanes_dashboard.py; generated plan config/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 from transform/privacy. They matched only direct OTLP metrics named traces.span.metrics.calls/duration entering the metrics pipeline. Gateway/Harbor connector output instead uses metrics/spans and transform/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

Evidence-class table

Claim Evidence class Command / receipt
Dimension names, native cost contracts and processor ownership source_review Versioned sources above, immutable main/old-head inspection and dated correction below
Generated dashboard matches source; existing publisher contracts remain passing local_integration Renderer check and 17 dashboard contract tests
Native connector retains provider/status/outcome and merges equal groups synthetic Existing fixture sends four synthetic spans through main's connector pipeline; 3 exported groups have counts 2/1/1
Source publisher produces an exact copy; collector accepts it local_integration Repository observability_config.py otel into a temporary destination, then native otelcol-contrib validate
Native query parsers accept the eight new targets local_integration Read-only Prometheus/Loki query APIs returned success; missing Codex-cost/gateway series remain unknown
Deployed collector/gateway export and complete cost coverage unverified CC host publication/read-back; no host restart or configuration apply in this PR

Local commands run

python3 observability/ns2604_dashboards.py
python3 observability/ns2604_dashboards.py --check
  rc=0
python3 -m unittest tests.test_ns2604_dashboards tests.test_j775_panel_contracts tests.test_observability_writer_identity -v
  rc=0; 43 collected, 39 passed, 4 optional pinned-Prometheus/promtool checks skipped
NS2604_OBSERVABILITY_DATA=<temporary>/data python3 evidence/artifacts/new-wsl-install-plan-20261002/config/observability_config.py otel --source-root <checkout>/evidence/artifacts/new-wsl-install-plan-20261002/config --config-root <temporary>/config
  rc=0; published YAML matches source and named main byte-for-byte
NS2604_OBSERVABILITY_DATA=<temporary>/data otelcol-contrib validate --config=<temporary>/config/otel.yaml
  rc=0; installed collector 0.162.0
python3 scripts/component_matrix.py --write
python3 scripts/new_host_grand_list.py --write
  rc=0; no derived-report changes
python3 scripts/validate.py
  rc=0
git diff --check
  rc=0

Decision record

docs/decisions/2026-10-08-cc-lanes-rate-cost-gateway.md records 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

@seathatflowsinourveins seathatflowsinourveins added the lane:foundation Foundation lane: Claude/Codex setup, hosts, memory, RAG, research, workers label Oct 8, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-08T08:49:26.110565Z 0078ccb PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@seathatflowsinourveins
seathatflowsinourveins force-pushed the codex/ns2604-row9-observability-20261008 branch from 0078ccb to 66a7228 Compare October 8, 2026 09:31
@seathatflowsinourveins seathatflowsinourveins changed the title Add lane rates, estimated costs and gateway routing dimensions Add lane rates, estimated costs and gateway routing panels Oct 8, 2026
@seathatflowsinourveins
seathatflowsinourveins merged commit 41d1375 into main Oct 8, 2026
35 of 46 checks passed
@seathatflowsinourveins
seathatflowsinourveins deleted the codex/ns2604-row9-observability-20261008 branch October 8, 2026 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lane:foundation Foundation lane: Claude/Codex setup, hosts, memory, RAG, research, workers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant