Skip to content

feat: Add Python Virtual Environment Support: Uninstalling User Defined Packages#5035

Merged
kunwp1 merged 11 commits into
apache:mainfrom
SarahAsad23:pve-delete-package
May 13, 2026
Merged

feat: Add Python Virtual Environment Support: Uninstalling User Defined Packages#5035
kunwp1 merged 11 commits into
apache:mainfrom
SarahAsad23:pve-delete-package

Conversation

@SarahAsad23
Copy link
Copy Markdown
Contributor

What changes were proposed in this PR?

This PR is an extension of PR #4484 and #4902. Previously, we introduced support for creating Python Virtual Environments (PVEs) with system-level dependencies preinstalled, along with support for installing user-defined packages. This PR extends that functionality by allowing users to uninstall user-installed packages from their PVEs.

Any related issues, documentation, discussions?

This change is part of ongoing efforts to support environment isolation and reproducibility within Texera. Related issue includes #4296. This PR closes sub-issue #4466.

How was this PR tested?

Tested Manually and PveResourceSpec test file updated.

To test:

On CU click "+" Python Environments.
Input environment name.
Input package name and version.
Click "OK" and wait for pip logs.
To delete click on "Delete Icon" and click "OK"

Was this PR authored or co-authored using generative AI tooling?

Co-authored using: ChatGPT (OpenAI)

@SarahAsad23 SarahAsad23 marked this pull request as draft May 12, 2026 17:11
@kunwp1 kunwp1 linked an issue May 12, 2026 that may be closed by this pull request
5 tasks
@github-actions github-actions Bot added the frontend Changes related to the frontend GUI label May 12, 2026
@SarahAsad23 SarahAsad23 marked this pull request as ready for review May 12, 2026 22:01
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 12, 2026

Codecov Report

❌ Patch coverage is 30.63063% with 77 lines in your changes missing coverage. Please review.
✅ Project coverage is 42.76%. Comparing base (ed2f775) to head (448d003).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...power-button/computing-unit-selection.component.ts 0.00% 43 Missing ⚠️
...wer-button/computing-unit-selection.component.html 0.00% 15 Missing ⚠️
...resource/pythonvirtualenvironment/PveManager.scala 79.06% 5 Missing and 4 partials ⚠️
...esource/pythonvirtualenvironment/PveResource.scala 0.00% 8 Missing ⚠️
...virtual-environment/virtual-environment.service.ts 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #5035      +/-   ##
============================================
- Coverage     42.78%   42.76%   -0.03%     
  Complexity     2199     2199              
============================================
  Files          1045     1045              
  Lines         39989    40092     +103     
  Branches       4220     4232      +12     
============================================
+ Hits          17111    17145      +34     
- Misses        21817    21884      +67     
- Partials       1061     1063       +2     
Flag Coverage Δ *Carryforward flag
access-control-service 39.53% <ø> (ø) Carriedforward from 9fd178d
agent-service 33.72% <ø> (ø) Carriedforward from 9fd178d
amber 43.60% <66.66%> (+0.08%) ⬆️
computing-unit-managing-service 0.00% <ø> (ø) Carriedforward from 9fd178d
config-service 0.00% <ø> (ø) Carriedforward from 9fd178d
file-service 32.18% <ø> (ø) Carriedforward from 9fd178d
frontend 33.93% <0.00%> (-0.12%) ⬇️
python 88.89% <ø> (-0.01%) ⬇️ Carriedforward from 9fd178d
workflow-compiling-service 47.72% <ø> (ø) Carriedforward from 9fd178d

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Contributor

@kunwp1 kunwp1 left a comment

Choose a reason for hiding this comment

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

LGTM! One minor comment.

@kunwp1 kunwp1 enabled auto-merge (squash) May 13, 2026 03:45
auto-merge was automatically disabled May 13, 2026 04:05

Head branch was pushed to by a user without write access

@SarahAsad23 SarahAsad23 requested a review from kunwp1 May 13, 2026 04:05
@kunwp1 kunwp1 enabled auto-merge (squash) May 13, 2026 04:12
@kunwp1 kunwp1 merged commit df23815 into apache:main May 13, 2026
15 checks passed
@SarahAsad23 SarahAsad23 deleted the pve-delete-package branch May 13, 2026 04:16
@chenlica
Copy link
Copy Markdown
Contributor

@SarahAsad23 It will be good to include a screenshot in the description.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

engine frontend Changes related to the frontend GUI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Delete Python Packages from Virtual Environment

4 participants