Skip to content

Fix DeathStarBench venv install to be distro-agnostic#736

Merged
ericavella merged 4 commits into
mainfrom
users/evellanoweth/dsb-venv-distro-fix
Jun 25, 2026
Merged

Fix DeathStarBench venv install to be distro-agnostic#736
ericavella merged 4 commits into
mainfrom
users/evellanoweth/dsb-venv-distro-fix

Conversation

@ericavella

Copy link
Copy Markdown
Contributor

Summary

The PERF-NETWORK-DEATHSTARBENCH profile installed python3.12-venv via LinuxPackageInstallation. That package only exists on Ubuntu 24.04 (which ships Python 3.12). On Ubuntu 22.04 (Python 3.10) and other distros the package does not exist, so apt fails and the workload can't run.

Fix

Use the generic python3-venv meta-package, which always resolves to the venv module for each distro's default Python 3 (3.8 / 3.10 / 3.12). This makes the dependency distro-agnostic and matches the package the DeathStarBenchExecutor already installs at runtime (apt install python3-venv -y).

- "Packages-Apt": "...,libssl1.0.0,python3.12-venv"
+ "Packages-Apt": "...,libssl1.0.0,python3-venv"

Testing

  • JSON validated.
  • No remaining references to python3.12-venv in the source tree.

Erica Vellanoweth and others added 4 commits June 24, 2026 14:33
The PERF-NETWORK-DEATHSTARBENCH profile installed python3.12-venv, which only exists on Ubuntu 24.04 (Python 3.12) and fails on Ubuntu 22.04 (Python 3.10) and other distros. Use the generic python3-venv meta-package, which resolves to the venv module for each distro's default Python 3. This matches the package the DeathStarBenchExecutor already installs at runtime.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The python3-venv package is now installed by the LinuxPackageInstallation dependency in the PERF-NETWORK-DEATHSTARBENCH profile, so the executor's runtime 'apt install python3-venv -y' is redundant. Removed it and updated the affected unit tests.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

@AlexWFMS AlexWFMS left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice2

@ericavella ericavella enabled auto-merge (squash) June 25, 2026 16:58
@ericavella ericavella merged commit f715b51 into main Jun 25, 2026
5 checks passed
@ericavella ericavella deleted the users/evellanoweth/dsb-venv-distro-fix branch June 25, 2026 17:07
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