[PR #3] [MERGED] Use virtualenv for python packages installation in deploy.sh, give correct user instructions for index.html copy #82

New issue

Closed

opened 2026-03-02 15:47:49 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 15:47:49 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/PegaProx/project-pegaprox/pull/3
Author: @phantium
Created: 1/25/2026
Status: ✅ Merged
Merged: 1/25/2026
Merged by: @MrMasterbay

Base: main ← Head: main

📝 Commits (3)

f974499 Use virtualenv and already created requirements.txt
45220fa There is no index.html, it's at web/index.html
5300d04 Command must be executed with sudo or you'll need to fix permissions

📊 Changes

1 file changed (+26 additions, -26 deletions)

View changed files

📝 deploy.sh (+26 -26)

📄 Description

Hello guys!

I moved the Python dependencies installation into a virtualenv and also updated the systemd service to reflect this
Instead of loading a list of packages inside of deploy.sh now using requirements.txt as a reference instead.
During installation I found out the index.html instructions were incorrect, as it is located in web/index.html, I have updated those instructions as well.

After installation I also noticed the permissions were broken causing a restart loop in the pegaprox service.
I've updated the instructions to be:
sudo -u pegaprox bash -c "cd /opt/PegaProx && ./venv/bin/python3 pegaprox_multi_cluster.py --download-static"

Rather than potentially running this command from the root user and having to fix permissions after with chown.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/PegaProx/project-pegaprox/pull/3 **Author:** [@phantium](https://github.com/phantium) **Created:** 1/25/2026 **Status:** ✅ Merged **Merged:** 1/25/2026 **Merged by:** [@MrMasterbay](https://github.com/MrMasterbay) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (3) - [`f974499`](https://github.com/PegaProx/project-pegaprox/commit/f97449925473baa80c36e3d373fa6d722bbfb25e) Use virtualenv and already created requirements.txt - [`45220fa`](https://github.com/PegaProx/project-pegaprox/commit/45220fab1609923ea1fc2fbf9fa578e3fd1e2ecb) There is no index.html, it's at web/index.html - [`5300d04`](https://github.com/PegaProx/project-pegaprox/commit/5300d04407386ec597f9d82685f19410af30bcf9) Command must be executed with sudo or you'll need to fix permissions ### 📊 Changes **1 file changed** (+26 additions, -26 deletions) <details> <summary>View changed files</summary> 📝 `deploy.sh` (+26 -26) </details> ### 📄 Description Hello guys! I moved the Python dependencies installation into a virtualenv and also updated the systemd service to reflect this Instead of loading a list of packages inside of deploy.sh now using `requirements.txt` as a reference instead. During installation I found out the index.html instructions were incorrect, as it is located in web/index.html, I have updated those instructions as well. After installation I also noticed the permissions were broken causing a restart loop in the pegaprox service. I've updated the instructions to be: `sudo -u pegaprox bash -c "cd /opt/PegaProx && ./venv/bin/python3 pegaprox_multi_cluster.py --download-static"` Rather than potentially running this command from the root user and having to fix permissions after with chown. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>