Skip to content

Commit 9e31c62

Browse files
tosin2013claude
andcommitted
fix(deploy): Add --system flag to all uv pip install calls
Without --system, uv refuses to install packages when no virtual environment is active, causing deployment failures on fresh systems. Co-Authored-By: Claude Opus 4.5 <[email protected]>
1 parent 91d8fcb commit 9e31c62

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ INSTALL_PATH = ~/.ansible-navigator.yml # Navigator configuration destination
5050
SOURCE_FILE = ~/qubinode_navigator/ansible-navigator/release-ansible-navigator.yml # Navigator config source
5151

5252
# 📊 COMMAND DEFINITIONS (build automation commands):
53-
INSTALL_ANSIBLE_NAVIGATOR := uv pip install ansible-navigator>=25.5.0 # Install navigator with minimum version
53+
INSTALL_ANSIBLE_NAVIGATOR := uv pip install --system ansible-navigator>=25.5.0 # Install navigator with minimum version
5454
BUILD_CMD := tag=$(TAG) && cd ~/qubinode_navigator/ansible-builder/ && ansible-builder build -f execution-environment.yml -t qubinode-installer:$${tag} -v 3 # Build execution environment
5555
COPY_NAVIGATOR_CMD := cp $(SOURCE_FILE) $(INSTALL_PATH) # Copy navigator configuration
5656
PODMAN_LOGIN := podman login registry.redhat.io # Authenticate with Red Hat registry

scripts/development/deploy-qubinode.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -942,7 +942,7 @@ deploy_airflow_services() {
942942
if ! command -v podman-compose &> /dev/null; then
943943
log_info "Installing podman-compose for Airflow..."
944944
ensure_uv
945-
if ! uv pip install podman-compose; then
945+
if ! uv pip install --system podman-compose; then
946946
log_warning "Failed to install podman-compose, skipping Airflow deployment"
947947
return 1
948948
fi
@@ -1660,13 +1660,13 @@ configure_navigator() {
16601660

16611661
# Install Python requirements
16621662
ensure_uv
1663-
sudo -E uv pip install -r requirements.txt || {
1663+
sudo -E uv pip install --system -r requirements.txt || {
16641664
log_warning "Failed to install Python requirements"
16651665
}
16661666

16671667
# Install passlib for Ansible password_hash filter
16681668
log_info "Ensuring passlib is installed for password hashing..."
1669-
sudo -E uv pip install passlib || {
1669+
sudo -E uv pip install --system passlib || {
16701670
log_warning "Failed to install passlib, password operations may fail"
16711671
}
16721672

0 commit comments

Comments
 (0)