Skip to content

fix: python virtual environment single node configurations #6029

Open
SarahAsad23 wants to merge 4 commits into
apache:mainfrom
SarahAsad23:fix-pve-k8s
Open

fix: python virtual environment single node configurations #6029
SarahAsad23 wants to merge 4 commits into
apache:mainfrom
SarahAsad23:fix-pve-k8s

Conversation

@SarahAsad23

Copy link
Copy Markdown
Contributor

What changes were proposed in this PR?

This PR fixes the single-node deployment configuration for the Python Virtual Environments (PVE) feature.

Changes:

  • Adds the missing /pve route to the single-node NGINX configuration.
  • Updates PveManager to correctly locate the system requirements files during deployment.

Note: This fix applies only to the Release 1.2 implementation of the Python Virtual Environments feature. The feature has since been significantly refactored on main.

Any related issues, documentation, discussions?

Closes #5963.

How was this PR tested?

Tested in a deployment setting.

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

No.

@github-actions

Copy link
Copy Markdown
Contributor

Automated Reviewer Suggestions

Based on the git blame history of the changed files, we recommend the following reviewers:

  • Contributors with relevant context: @bobbai00
    You can notify them by mentioning @bobbai00 in a comment.

@github-actions

Copy link
Copy Markdown
Contributor

📊 Arrow Flight E2E bench

(no arrow-flight-e2e.csv in artifact)

Full dashboard · Workflow run

@codecov-commenter

Copy link
Copy Markdown

⚠️ JUnit XML file not found

The CLI was unable to find any JUnit XML files to upload.
For more help, visit our troubleshooting guide.

@xuang7

xuang7 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Hi @SarahAsad23, it seems this PR has conflicts with the main branch. Could you resolve the conflicts and update the PR description?

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PVE REST endpoints (/pve/*) return 404 in single-node Docker deployment

3 participants