Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PAUSED] pmm: Add auto_escalate[] logic #214

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

NICHOLAS85
Copy link
Contributor

@NICHOLAS85 NICHOLAS85 commented Nov 14, 2020

Allows a pm to set auto_escalate[]. When this var is set its value will be used to populate ${root_user} for other pms. This is useful to mimic autosudoing uis behaviors like yay for other pms.

I tested every command to check whether they needed root permissions or not to function and prepended the command which did. However, this doesn't mean I didn't miss any or miss-categorize so I'll keep checking each commands behavior as I use this pr.
The Bedrock documentation would need to be updated to reflect this new variable of course.

@NICHOLAS85
Copy link
Contributor Author

NICHOLAS85 commented Nov 14, 2020

Current issues after a day of usage:
Caching package database requires root and does not currently automatically escalate. I believe this was an issue already present in pmm that hadn't made itself apparent yet. yay, the only pm and pmm ui which a user would run without privileges, is affected. Potential solutions can be to not cache when run as user like cache_package_manager_list or to escalate automatically using auto_escalate[]. What do you think?

I believe pmm has some internal logic for the ${unprivileged_user} var which cuts the extra space out from the command preview when it is unpopulated, I need to replicate this behavior for ${root_user}. - Fixed

@NICHOLAS85
Copy link
Contributor Author

Placed on hold until a per-user infrastructure is developed for pmm/Bedrock as pmm will have to manage non-root per-user package caches as well.

@NICHOLAS85 NICHOLAS85 changed the title pmm: Add auto_escalate[] logic [PAUSED] pmm: Add auto_escalate[] logic Nov 15, 2020
@NICHOLAS85 NICHOLAS85 marked this pull request as draft May 2, 2021 20:53
@paradigm paradigm force-pushed the master branch 2 times, most recently from 12e1061 to 524beae Compare March 18, 2024 01:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant