[v0.91.8][runtime-v3] Simplify and harden local launch adapters#5659
Merged
Conversation
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
1 participant
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\n- remove the live DegradedOperationExecutor construction\n- build all required Runtime v3 adapter bindings as identity-bearing in-process executors\n- fail closed if the required adapter set is incomplete\n- keep the runtime API-only for the separate Observatory client\n- exclude TLS private-key material from continuity identity\n\n## Validation\n-
running 5 tests
test production_readiness_accepts_complete_in_process_bindings ... ok
test live_assembly_refuses_a_missing_executor_binding ... ok
test live_assembly_has_the_frozen_service_inventory ... ok
test live_assembly_starts_and_qualifies_time ... ok
test canonical_ingress_dispatches_allowlisted_work_and_commits_only_success ... ok
test result: ok. 5 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.11s
running 4 tests
test observatory_websocket_rejects_a_token_after_rotation ... ok
test observatory_websocket_requires_allowed_origin_and_session_auth ... ok
test observatory_websocket_rejects_bad_auth_and_client_data ... ok
test observatory_websocket_revokes_an_authenticated_session_after_rotation ... ok
test result: ok. 4 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 1.03s\n- \n- \n- \n\nThe full workspace suite still has unrelated timing-sensitive failures in governed_operations; this PR does not claim those lanes as green.\n\nCloses #5657