Programs should be installed in the Program Files directory instead of the Root of C:
. We need to modify the installer to allow Salt to be installed anywhere, but default to Program files.
This would mean modifying the way the installer works:
C:\Program Files
)win_verify_env
will have to be more robust, not just locking down the salt
and pki
directories, but allowing the user to move those where they wish and have them be locked down. (currently, if they are defined in the minion config and are moved outside of the salt structure, they are not locked down.)I think anything that needs to be locked down or cached probably needs to be separated out and default to being in %PROGRAMDATA%
. This also fixes the problem of the cache being lost on upgrade (#38590) and would be the logical place to write log files and store keys (i.e. 'Program Files' is meant to be static).
Agreed.
@morganwillcock the msi installer does not lose the cache on upgrade, but I agree that the cache should not be placed under 'Program Files'
@morganwillcock The cache was being deleted specifically by the installer... I removed that line this PR.
just wondering if there were any updates on this?
Upvote.
@twangboy @dwoz any chance this is going to be implemented in the near future? thank you!
Hello, anyone here? Same question.
It was a dealbreaker for me so a few weeks ago I customised the code that builds the Salt.exe and fixed a couple of other little issues along the way. So now I can install Salt to any directory and it seems to work fine on Windows 10 and 2016. Still need to test it on older versions which I will do soon.
When I have some time in the new year was planning to figure out how to submit a pull request to submit the changes back.
If you want to test out the .exe I could post a link to it here (only version 2019.2.2 64 bit) - though not sure if the Salt devs would be okay with that or not?
Most helpful comment
I think anything that needs to be locked down or cached probably needs to be separated out and default to being in
%PROGRAMDATA%
. This also fixes the problem of the cache being lost on upgrade (#38590) and would be the logical place to write log files and store keys (i.e. 'Program Files' is meant to be static).