fix(e2e): restore messaging proxy reachability - #10788
Conversation
Signed-off-by: Julie Yaunches <jyaunches@nvidia.com>
|
Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually. Contributors can view more details about this message here. |
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueComment |
Code Coverage OverviewLanguages: TypeScript TypeScript / code-coverage/pluginThe overall line coverage in commit 8c5421a in the TypeScript / code-coverage/cliThe overall line coverage in commit 8c5421a in the Show a line coverage summary of the most impacted files.
Updated |
|
Trusted target reconciliation for exact head
This run is historical infrastructure evidence only. It is not a qualifying green latest-head target result. The PR remains draft; no rerun or replacement dispatch was made. |
Signed-off-by: Julie Yaunches <jyaunches@nvidia.com>
Signed-off-by: Julie Yaunches <jyaunches@nvidia.com>
|
PR Review Advisor finished for commit |
Signed-off-by: Julie Yaunches <jyaunches@nvidia.com>
Outcome
The trusted
messaging-providerstarget can expose its credential-free fake API proxy on the OpenShell bridge while keeping the credential-bearing fake API on an internal-only network. The proxy remains unable to use masqueraded outbound egress.Reason
The latest completed target evidence for the superseded #10628 failed the installed WeChat runtime with
ECONNREFUSED 172.18.0.1:32771. The host port was not reachable from the OpenShell sandbox: the main implementation bound it only to host loopback, while the attempted internal-only replacement could not install usable published-port forwarding.Related issues
messaging-providersfailure in full main E2E run 33350578179, job 99363408336.Changes
Verification
npx vitest run --project e2e-support test/e2e/support/messaging-providers-runtime-proofs.test.ts— 30 tests passed.npx vitest run --project integration test/automation/pull-requests/growth-guardrails.test.ts— 33 tests passed.npm run test:e2e-phases:check— 132 tests across 90 files passed semantic phase coverage.npm run checks:repository— repository checks passed.npm run test:titles:check— test title check passed.npx oxfmt --check test/e2e/live/messaging-providers-helpers.ts test/e2e/support/messaging-providers-runtime-proofs.test.ts— formatting passed.Review notes
E2E root cause
db5a9c6793accb0bd9167e9421becdf03940ad39against base9169ab0790d9a1be68b439731c01b696f75ff085.ECONNREFUSED 172.18.0.1:32771after publication and startup traffic proof had passed.messaging-providerstarget run with complete cleanup proof. This PR remains draft until that target and exact-commit CI and review gates are green.Security boundary
The fake API alone receives provider credentials and has no host publication. The host-published proxy receives no provider credentials, drops all capabilities, uses a read-only filesystem, and attaches to a separate bridge with IP masquerading disabled. Docker network names and the OpenShell bridge address fail closed before resources are created.
Signed-off-by: Julie Yaunches jyaunches@nvidia.com