Install script for linux not idempotent on arch and sudo kinks #1630

Open
opened 2025-11-20 06:03:48 -05:00 by saavagebueno · 0 comments
Owner

Originally created by @schichtnudelauflauf on GitHub (Feb 15, 2025).

Describe the problem

If you run the curl | sh install script and it fails, it will not try again on next run.

To Reproduce

Steps to reproduce the behavior:

  1. Spin up arch vm
  2. login as root
  3. curl | bash the linux installer
  4. Script fails because makepkg doesnt want to run as root
  5. run script again from a nonroot user
  6. script requires sudo elevation
  7. give user nopasswd elevation to root in sudoers
  8. run script again
  9. script fails complaining that a path "netbird" already exists but not where

Expected behavior

Installer can cope with being root on an empty vm.
Installer tells where the path is it wants to copy to.
Installer recovers from failure on re run.

Are you using NetBird Cloud?

irrelevant at this point

Additional context

The path it complains was under /tmp
Also complainable that an installer that expects privileges uses a predictable name under /tmp.

Originally created by @schichtnudelauflauf on GitHub (Feb 15, 2025). **Describe the problem** If you run the curl | sh install script and it fails, it will not try again on next run. **To Reproduce** Steps to reproduce the behavior: 1. Spin up arch vm 2. login as root 3. curl | bash the linux installer 4. Script fails because makepkg doesnt want to run as root 5. run script again from a nonroot user 6. script requires sudo elevation 7. give user nopasswd elevation to root in sudoers 8. run script again 9. script fails complaining that a path "netbird" already exists but not where **Expected behavior** Installer can cope with being root on an empty vm. Installer tells where the path is it wants to copy to. Installer recovers from failure on re run. **Are you using NetBird Cloud?** irrelevant at this point **Additional context** The path it complains was under /tmp Also complainable that an installer that expects privileges uses a predictable name under /tmp.
saavagebueno added the triage-needed label 2025-11-20 06:03:48 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/netbird#1630