-
Notifications
You must be signed in to change notification settings - Fork 45
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
Remote script execution for snaps #209
Conversation
mcw-work
commented
Jan 31, 2024
- Enable remote script execution in the snap.
- When a snap execute all scripts as root (confined by the snap)
- Add LD_LIBRARY and PYTHONPATH env variables to script execution environment.
* When a snap execute all scripts as root (confined by the snap) * Add LD_LIBRARY and PYTHONPATH env variables to script execution environment.
Restructure IS_SNAP tests
Update test to match script env_vars
updated tests to match changes to script execution. |
Added unit tests for when script is called from a snap |
Tests all passed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests pass, nice! Have you (or someone else) ensured that script execution works with this version both on core and classic, then?
Yeap - I have installed it on a x86 classic machine and a pi core device and they both work fine. The scripts that are saved in the snap-management-demo account on the staging server are also tested and verified as working - that includes one as a heredoc, one using REST API directly and one using Snap-HTTP. Stephen and Yanisa are having a look at what we need to document as well so we don't fall too far behind with that. |