-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
env_vars in pip_state vs. env in cmd #45348
Comments
I think this would be a good improvement. We would just need to make sure we properly deprecate the argument we do not choose. |
Any news on this? |
this is not currently assigned out to anyone due to other higher priority issues. Please feel free to try to send in a PR for this if you feel so inclined :) |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue. |
Yep. |
Thank you for updating this issue. It is no longer marked as stale. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue. |
And again. |
Thank you for updating this issue. It is no longer marked as stale. |
Description of Issue/Question
Why is there a difference in the way environment variables are specified in pip.installed and cmd.run state methods?
Examples from the documentation:
vs.
Differences are:
Could this be streamlined, please, so that there's only one unique way of specifying environment variables (and everywhere else where I didn't spot it, of course)?
See also:
https://docs.saltstack.com/en/latest/ref/states/all/salt.states.cmd.html#salt.states.cmd.run
https://docs.saltstack.com/en/latest/ref/states/all/salt.states.pip_state.html#salt.states.pip_state.installed
The text was updated successfully, but these errors were encountered: