fix: prefix Proxmox commands with sudo for non-root SSH user #44

2026-07-22T22:58:30Z

beatzaplenty commented

2026-07-22 22:58:30 +00:00

PROXMOX_SSH_USER was changed from root to wayne, but all remote pvesh/qm/pct
invocations assumed root. When run as a non-root user these commands fail with
ipcc_send_rec errors because they can't reach the pve-cluster IPC socket.

Adds a global sudo_prefix (empty when PROXMOX_SSH_USER=root, "sudo" otherwise)
and applies it to every remote Proxmox command in the script, including the
duplicate-host heredoc check, pvesh nextid, vmid existence checks, resource
destruction, and all create/start commands. Removes the now-redundant local
sudo_prefix definition that was previously only in the VM image build branch.

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com

PROXMOX_SSH_USER was changed from root to wayne, but all remote pvesh/qm/pct invocations assumed root. When run as a non-root user these commands fail with ipcc_send_rec errors because they can't reach the pve-cluster IPC socket. Adds a global sudo_prefix (empty when PROXMOX_SSH_USER=root, "sudo" otherwise) and applies it to every remote Proxmox command in the script, including the duplicate-host heredoc check, pvesh nextid, vmid existence checks, resource destruction, and all create/start commands. Removes the now-redundant local sudo_prefix definition that was previously only in the VM image build branch. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

beatzaplenty added 1 commit 2026-07-22 22:58:31 +00:00

fix: prefix Proxmox commands with sudo for non-root SSH user

Check NixOS configurations / eval-hosts (pull_request) Successful in 10m18s

Details

7779f3e137

PROXMOX_SSH_USER was changed from root to wayne, but all remote pvesh/qm/pct
invocations assumed root. When run as a non-root user these commands fail with
ipcc_send_rec errors because they can't reach the pve-cluster IPC socket.

Adds a global sudo_prefix (empty when PROXMOX_SSH_USER=root, "sudo" otherwise)
and applies it to every remote Proxmox command in the script, including the
duplicate-host heredoc check, pvesh nextid, vmid existence checks, resource
destruction, and all create/start commands. Removes the now-redundant local
sudo_prefix definition that was previously only in the VM image build branch.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

beatzaplenty merged commit 98409f4502 into main

2026-07-22 22:59:36 +00:00

This repo is archived. You cannot comment on pull requests.

1 Participants

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: beatzaplenty/nixos#44