Repository navigation
Add report-only packaged release compatibility checks - #26280
Stella Huang (StellaHuang95) wants to merge 5 commits into
Conversation
36931ab to
b698286
Compare
b698286 to
06cca70
Compare
|
🔒 Automated review in progress — Rich Chiodo (@rchiodo) is auto-reviewing this PR. |
|
Reviewed commit: Result: 🔴 Verification detailsVerification: Automated verification is best effort. The PR author remains responsible for testing. Summary: [unavailable] Container verification could not start and local execution was not authorized for this PR HEAD: C:\Program Files\RedHat\Podman\podman.EXE --connection pyrx-automation failed: stderr: Error: unable to start container "89d90b700e7aaa0e060af07bfb22beefb11c8c01c5a9262464bcb99cb3880890": crun: open `memory.max` for writing: No such file or directory: OCI runtime attempted to invoke a command that was not found Test runs: none recorded. |
06cca70 to
837ed28
Compare
Rich Chiodo (rchiodo)
left a comment
There was a problem hiding this comment.
Approved via Review Center.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
837ed28 to
2af7f47
Compare
Summary
Add Windows x64 and Linux x64 compatibility checks for the actual packaged Python extension, paired with a fixed released Python Environments extension. The goal is to catch packaging and cross-extension integration issues before release by verifying discovery, interpreter switching, and real Python execution with Environments explicitly enabled and disabled.
Existing source, native, and packaged smoke tests remain in place. This closes a specific coverage gap: the current Run Python File packaged smoke test explicitly disables Environments, and its execution case skips Windows. It does not imply that all integration coverage was missing or that the released packages are incompatible.
Before and after
Current flows with this change
build-vsixjobs -> new Windows/Linux candidate compatibility jobs -> retained reports and raw logs. Each candidate is paired with the pinned released Environments package. Existing build and test jobs are unchanged.The fixed controls currently use Environments 1.42.0 and VS Code 1.110.0; the harness-control pair also uses Core 2026.10.0. Missing or mismatched candidate inputs fail explicitly rather than falling back to that released pair.
What the checks verify
falseopt-out.python.execInTerminalcommand must produce fresh output matching the selectedsys.executableandsys.prefix; opening a terminal is not a pass.The reusable harness also supports the bundled-PET to Core-PET fallback upgrade used by the companion Environments PR #1994. Upgrade phases use normal isolated windows with disk-backed storage, verify independent storage continuity, and read the saved interpreter selection before any reselection. Product state is not copied or reseeded. This upgrade is not an additional default Core candidate job.
Scope and release impact
There are no production runtime, integration-default, opt-out, dependency, PET-bundling, or existing build/sign/publish changes in this PR. The harness is excluded from the shipped VSIX. No branch-protection or publication gate is enabled.
GitHub candidate checks can fail CI; the separate Azure consumer is deliberately report-only. CI-built package results are not final signed-publication certification. Actual Azure artifact permissions/hosted acceptance, a post-Sign hook, and reconciliation with the exact signed/published bytes remain rollout work.
Coverage is bounded to the pinned pairs and local venv workflow. The current bundled-Environments pair can use Environments' PET when enabled and JavaScript discovery when disabled, so it does not guarantee execution of the candidate Core's own PET in every run.
Runner inputs, reports, and release handoff details are documented in
build/releaseCompatibility/README.md.