Skip to content

docs(w64): finalize operational analytics provider truth - #248

Merged
ammarheidari merged 1 commit into
mainfrom
w64-closeout-docs
Oct 4, 2026
Merged

ammarheidari merged 1 commit into
mainfrom
w64-closeout-docs

Conversation

@ammarheidari

@ammarheidari ammarheidari commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

Authority

W64 #213 is in final closeout after PR #245 backend runtime/history/SLO and PR #247 operator UI parity.

Purpose

Record the exact v0.8 provider truth for operational analytics:

  • consumer total lag is Supported from the governed Kafka consumer read view;
  • consumer consume-rate and broker/topic throughput have no admitted inbound metrics provider in v0.8 and therefore remain explicit Unavailable;
  • OTLP and Prometheus are exporter/scrape surfaces, not inbound analytics query providers;
  • no numeric zero is fabricated from missing evidence;
  • historical trends/SLOs are exposed only for metric families with real admitted observations;
  • adding JMX/Prometheus-query/APM readers requires a separately reviewed typed provider contract.

This closes the remaining W64 compatibility/documentation gap without silently expanding provider scope.

Exact head: 51c176faf178896da8f53464c273f8c73032331b.

Refs #213 #209.

Capability-state corrective

  • unadmitted inbound metrics providers are classified as Unsupported, not Unavailable;
  • Unavailable/Unknown apply only to admitted runtime evidence gaps/failures;
  • Partial is reserved for real valid-but-incomplete numeric evidence;
  • all review threads are resolved;
  • source is frozen pending fresh exact-head CI, CODEOWNER approval and Codex review.

Capability-vs-observation truth corrective

  • capability-level Unsupported is now explicitly separated from live observation-level Unavailable;
  • current runtime behavior via UnavailableMetricsObservationPort is documented without implying provider support;
  • consumers are warned not to infer capability support from observation state alone;
  • all review threads are resolved;
  • source is frozen pending fresh exact-head CI, CODEOWNER approval and Codex review.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 4, 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-04T14:58:46.444221Z 51c176f Manual request
ℹ️ 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.

Copy link
Copy Markdown
Contributor Author

@codex review

Please review exact head d04dadc3707d13dcce6cfd98d7b1e4d206491183 only. Focus on whether the W64 provider-truth closeout accurately distinguishes inbound analytics evidence sources from OTLP/Prometheus export surfaces, preserves explicit Unavailable semantics, and avoids expanding provider scope without admission.

akhiabanchian
akhiabanchian previously approved these changes Oct 4, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d04dadc370

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/architecture/v0.8-provider-client-capabilities.md Outdated
Comment thread docs/architecture/v0.8-provider-client-capabilities.md Outdated

Copy link
Copy Markdown
Contributor Author

@codex review

Please review exact head 1e395c9dafc6a804dbab92904618e58f7ae700f3 only. The prior P2 findings are addressed: unadmitted inbound metrics providers are now Unsupported, Unavailable/Unknown are reserved for admitted runtime evidence gaps/failures, and Partial is explicitly restricted to real valid-but-incomplete numeric evidence. Please focus on semantic consistency and regressions.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1e395c9daf

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/architecture/v0.8-provider-client-capabilities.md Outdated
Signed-off-by: Ammar Heidari <ammar@arad-itc.org>

Copy link
Copy Markdown
Contributor Author

@codex review

Please review exact head 51c176faf178896da8f53464c273f8c73032331b only. The prior P2 is addressed by explicitly separating capability truth from live observation truth: unadmitted metric families are Unsupported capabilities, while current v0.8 requests may still return observation-level Unavailable through the explicit unavailable port. Please focus on semantic consistency with current runtime/contracts.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Another round soon, please!

Reviewed commit: 51c176faf1

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ammarheidari
ammarheidari merged commit 3c76915 into main Oct 4, 2026
4 checks passed
@ammarheidari
ammarheidari deleted the w64-closeout-docs branch October 4, 2026 15:00
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