Skip to content

[wip] salt-call and salt-pip honor configured user#68685

Open
dwoz wants to merge 1 commit intosaltstack:3006.xfrom
dwoz:salt-call-user
Open

[wip] salt-call and salt-pip honor configured user#68685
dwoz wants to merge 1 commit intosaltstack:3006.xfrom
dwoz:salt-call-user

Conversation

@dwoz
Copy link
Contributor

@dwoz dwoz commented Feb 5, 2026

Fixes #68684

  • Ensure salt-call drops privileges to the configured 'user' to prevent root-owned cache files.
  • Add --priv flag to salt-call for explicit user switching.
  • Update sudo executor to use --priv to maintain sudo_user context.
  • Ensure salt-pip drops privileges to the configured 'user' before package installation.
  • Add unit and integration tests for privilege dropping and file ownership.

- Ensure salt-call drops privileges to the configured 'user' to prevent root-owned cache files.
- Add --priv flag to salt-call for explicit user switching.
- Update sudo executor to use --priv to maintain sudo_user context.
- Ensure salt-pip drops privileges to the configured 'user' before package installation.
- Add unit and integration tests for privilege dropping and file ownership.
@dwoz dwoz requested a review from a team as a code owner February 5, 2026 09:00
@dwoz dwoz added the test:full Run the full test suite label Feb 5, 2026
@dwoz dwoz changed the title salt-call and salt-pip honor configured user [wip] salt-call and salt-pip honor configured user Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:full Run the full test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant