Skip to content

test(e2e): remove python gpu smoke test#1948

Merged
maxamillion merged 2 commits into
mainfrom
1812-remove-python-gpu-smoke/elezar
Jun 17, 2026
Merged

test(e2e): remove python gpu smoke test#1948
maxamillion merged 2 commits into
mainfrom
1812-remove-python-gpu-smoke/elezar

Conversation

@elezar

@elezar elezar commented Jun 17, 2026

Copy link
Copy Markdown
Member

Summary

Remove the stale Python GPU smoke test and task aliases. This PR is stacked on #1947 so the pre-commit Helm dependency fix stays isolated from the e2e test cleanup.

Related Issue

Related to #1812
Stacked on #1947

Changes

  • Removed the Python GPU fixture.
  • Removed e2e/python/test_sandbox_gpu.py.
  • Removed the stale e2e:python:gpu and e2e:k3s:gpu task entries.

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (if applicable)

mise run pre-commit was attempted before stacking this PR on #1947. The Helm lint failure is addressed by #1947; the remaining local blocker was a GLIBC mismatch while loading libsqlx_macros.

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

@copy-pr-bot

copy-pr-bot Bot commented Jun 17, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@elezar

elezar commented Jun 17, 2026

Copy link
Copy Markdown
Member Author

/ok-to-test d879ee0

@elezar elezar force-pushed the 1812-remove-python-gpu-smoke/elezar branch from d879ee0 to a28ac64 Compare June 17, 2026 12:05
elezar added 2 commits June 17, 2026 14:55
Signed-off-by: Evan Lezar <elezar@nvidia.com>
Remove the Python GPU smoke test and its fixture. The e2e:k3s:gpu task only depended on e2e:python:gpu and did not have a separate k3s implementation, so remove that stale alias with the task it pointed at.

Signed-off-by: Evan Lezar <elezar@nvidia.com>
(cherry picked from commit 221a103)
@elezar elezar force-pushed the 1812-remove-python-gpu-smoke/elezar branch from a28ac64 to 70c59ca Compare June 17, 2026 14:59
@elezar elezar changed the base branch from main to no-issue-helm-lint-deps-el June 17, 2026 15:00
pimlock
pimlock previously approved these changes Jun 17, 2026
@pimlock

pimlock commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

/ok to test 70c59ca

Base automatically changed from no-issue-helm-lint-deps-el to main June 17, 2026 22:20
@maxamillion maxamillion dismissed pimlock’s stale review June 17, 2026 22:20

The base branch was changed.

@maxamillion maxamillion merged commit f23c2c8 into main Jun 17, 2026
27 checks passed
@maxamillion maxamillion deleted the 1812-remove-python-gpu-smoke/elezar branch June 17, 2026 22:21
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.

3 participants