-
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
Add disableexcludes to states.pkg #25617
Comments
@seanjnkns, thanks for the report. |
I fail to see how #29707 addresses any part of this issue/feature request. What I'm recommending is something similar to this, assuming say httpd* was excluded in your /etc/yum.conf. I realize doesn't work for all distros: httpd:
And this should just as easily be extended to pkgs, not just pkg, if there's a grouping and you want to ensure that exclusions are disabled for salt runs, but otherwise enforced. This then makes salt's salt.states.pkg.uptodate usable while not blindly updating everything, imo. |
I made a mistake when linking this PR. Sorry about that. |
Since #11415 added this functionality to execution module yumpkg, there is now unexpected feature disparity between the pkg state and the yumpkg execution module. The labels of this issue should include state module, surely? |
+1 When this feature will be added to states.pkg? |
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. |
I think this is still needed. |
This feature is still needed like in Kubernetes installations as aforementioned. |
Per issue #11415, this was added into yumpkg. If doing any yum exclusion enforcement, states will fail when trying to install anything that was excluded. It's preferable to add disableexcludes to the states.pkg vs. having no exclusions and then doing holds on most versions of packages on the system or doing something similar to this:
IMO, disableexcludes should not be set unless specifically included.
The text was updated successfully, but these errors were encountered: