Skip to content

ci(e2e): stop installing python3.12-venv, which Ubuntu 26.04 lacks - #603

Merged
tkislan merged 1 commit into
mainfrom
ci/e2e-ubuntu-26.04
Nov 5, 2026
Merged

tkislan merged 1 commit into
mainfrom
ci/e2e-ubuntu-26.04

Conversation

@tkislan

@tkislan tkislan commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Problem

E2E has failed on main since #602, in all five shards, before any test runs:

E: Unable to locate package python3.12-venv
##[error]Process completed with exit code 100.

#602 didn't cause it: it only touched build/venv-test-ipywidgets8-requirements.txt, which nothing in the repo reads. The cause is the runner image. GitHub is moving ubuntu-latest to Ubuntu 26.04 (actions/runner-images#14748, rollout Oct 19 – Nov 19). Our last green E2E run (41797583713) ran on ubuntu-24.04; the first red one (41922220528) ran on ubuntu-26.04. Ubuntu 26.04's system Python is 3.14, so its archive has no python3.12-venv.

Fix

Stay on ubuntu-latest and drop python3.12-venv from the apt install. The job never needed it:

  • Every venv in the job (test/e2e/helpers/venv.ts, the interpreter suite) is created by bare python3. That resolves to the 3.12 that actions/setup-python puts first on PATH, and that build includes ensurepip.
  • On 24.04 the package came preinstalled (python3.12-venv is already the newest version in the green log), so the line never changed anything there.

The interpreter suite's prerequisites comment pointed at the package, so its CI note is removed.

Verification

  • Checked in an ubuntu:26.04 container, using the runner's actions/python-versions 3.12.15 build installed the way setup-python installs it: python3 -m venv works with no *-venv package, and the new apt list installs cleanly.
  • This PR's E2E run is the end-to-end check on a 26.04 runner.

Summary by CodeRabbit

  • Chores
    • Adjusted automated test setup to require Python that can create virtual environments, without explicitly installing a version-specific virtual-environment package.
    • Other listed runtime components remain unchanged. This update affects test setup only and does not change application functionality or the product experience.

GitHub moved the ubuntu-latest label to Ubuntu 26.04, whose system Python
is 3.14 and whose archive has no python3.12-venv. The apt step failed with
"Unable to locate package python3.12-venv" in every E2E shard.

The package was never needed: every venv in the job is created by the
python3 that actions/setup-python puts first on PATH, and that build ships
its own ensurepip. On 24.04 the package came preinstalled, so the line was
a no-op there.
@codecov

codecov Bot commented Nov 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38%. Comparing base (3bf5784) to head (b4fa12e).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@          Coverage Diff          @@
##            main    #603   +/-   ##
=====================================
  Coverage     38%     38%           
=====================================
  Files        822     822           
  Lines      41098   41098           
  Branches    9046    9046           
=====================================
  Hits       15622   15622           
  Misses     23404   23404           
  Partials    2072    2072           
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@coderabbitai

coderabbitai Bot commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The E2E workflow no longer installs python3.12-venv; it continues to install python3-pip. The interpreter test prerequisite still requires python3 to be available and able to create a virtual environment, but no longer states that CI installs the removed package.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: jamesbhobbs, m1so

Merge Risk: Minimal · up to b4fa1

No actionable merge-blocking risk remains in the reviewed changes.

Pre-merge checks: Passed 6

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely states the main change: removing the unavailable python3.12-venv package from the E2E setup on Ubuntu 26.04.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Updates Docs Passed PASS. This PR changes only the E2E workflow and an inline test prerequisite comment. It does not implement a user-facing feature that requires OSS documentation or roadmap updates. The linked reposito…
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 1 files. (1 skipped: 1 unsupported.)

Full details: Updates Docs

Explanation

PASS. This PR changes only the E2E workflow and an inline test prerequisite comment. It does not implement a user-facing feature that requires OSS documentation or roadmap updates. The linked repositories are not available in this review; review them separately if this CI maintenance is intended to represent a feature change.

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: 203889fd-36a6-4135-8351-c9d8bfca392b
Commits

Reviewing files that changed from the base of the PR and between 3bf5784 and b4fa12e.

Files selected for processing: 2

  • .github/workflows/e2e.yml
  • test/e2e/suite/interpreter/interpreterKernel.e2e.test.ts

Included review availability: This review used your included allowance. 3 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 4 reviews per hour.

@tkislan
tkislan marked this pull request as ready for review November 5, 2026 07:51
@tkislan
tkislan requested a review from a team as a code owner November 5, 2026 07:51
@tkislan
tkislan merged commit 48e4679 into main Nov 5, 2026
20 checks passed
@tkislan
tkislan deleted the ci/e2e-ubuntu-26.04 branch November 5, 2026 08:30
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