Repository navigation
feat(observability): distinguish HTTP response selection from delivery failure - #13
Draft
afourniernv wants to merge 4 commits into
Draft
afourniernv wants to merge 4 commits into
afourniernv wants to merge 4 commits into
Conversation
…ivery Signed-off-by: Alex Fournier <afournier@nvidia.com>
2 of 5 tasks
Signed-off-by: Alex Fournier <afournier@nvidia.com>
Signed-off-by: Alex Fournier <afournier@nvidia.com>
…tion Signed-off-by: Alex Fournier <afournier@nvidia.com>
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.
Summary
Distinguish response selection from completed delivery, retaining late failures
without rewriting the primary security record. This is layer 6/6, based on
MCP #12.
The HTTP core remains NVIDIA #5214.
Related Issue
Closes NVIDIA#5398. Tracking: NVIDIA#5210.
This remains the last draft in the author-fork stack, not upstream promotion.
Changes
openshell.http.responsespan through requiredpipeline and transport completion, including final flush/shutdown.
Keep interim responses distinct from accepted final headers.
errors; distinguish a complete local error response from failed delivery.
Keep legitimate middleware findings and the immutable primary record.
custody, and intentional early-response upload cancellation.
The lifecycle works without exported spans. No new public API, configuration,
gateway RPC, protobuf, dependency or payload capture. No broad test-file moves.
Testing
6b81f8784: network 1,742, OCSF 188,middleware 166 and OpenTelemetry 70 passed: 2,166 total; two existing ignores.
writes, flush/shutdown failures, cancellation, reload and late failures.
middleware rejection and upstream errors, and actual response/root timing.
in this layer.
6b81f8784: fullmise run testpassed unchanged(9,189 Rust tests, 94 auxiliary tests, Python 211 and TypeScript 206).
target returned 200 then truncated its declared body: the immutable primary
retained 200 and exactly one terminal record captured incomplete delivery.
This also passed with supervisor trace export disabled, retaining OCSF and
receipts with zero exported supervisor roots.
Live evidence is cumulative on the final stack, not isolated-layer E2E. Initial
fixture failures and incomplete agent attempts are retained; isolated and final
full reruns passed. Full repository
mise run ciis not claimed.Checklist
25df1d0ddand8801b1fa2.