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

[2017.7] Back-port PR 50333 to 2017.7 #50590

Merged
merged 1 commit into from
Nov 21, 2018

Conversation

garethgreenaway
Copy link
Contributor

@garethgreenaway garethgreenaway commented Nov 20, 2018

Back-port PR #50333 to 2017.7
#50311

…rue. This was due to a change in saltstack#48426 that swapped out sending the pkgs variable for the desired variable instead.  This caused problems with pkg.hold because desired and sources are always populated, and pkg.hold can only include one or the other.  This change just includes desired in the call to pkg.hold since desired has the same value for sources.
@ghost ghost self-requested a review November 20, 2018 20:27
@garethgreenaway garethgreenaway requested a review from a team November 20, 2018 20:27
@garethgreenaway garethgreenaway changed the title [2017.7] Back-port PR 50333 to 2018.3 [2017.7] Back-port PR 50333 to 2017.7 Nov 20, 2018
@garethgreenaway garethgreenaway merged commit 6679242 into saltstack:2017.7 Nov 21, 2018
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.

3 participants