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.

Sign up for GitHub

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

Jump to bottom

[BUG] Win_wua fails to load if CryptSvc set to manual #62993

Closed
twangboy opened this issue Nov 1, 2022 · 0 comments · Fixed by #62994
Closed

[BUG] Win_wua fails to load if CryptSvc set to manual #62993

twangboy opened this issue Nov 1, 2022 · 0 comments · Fixed by #62994
Assignees
@twangboy
Labels
Bug broken, incorrect, or confusing behavior Execution-Module Sulfur v3006.0 release code name and version Windows
Milestone
Sulfur v3006.0

Comments

@twangboy
Copy link
Contributor

twangboy commented Nov 1, 2022 •
edited
Loading

Description
From @lukasraska
Is there any specific reason why the win_wua module checks if CryptSvc is set to Auto? https://github.com/saltstack/salt/blob/master/salt/modules/win_wua.py#L111
For me the check for Disabled would make more sense, based on the error message, or checking whether it's running in the first place might even be the best... we've come across one server where it's set to Manual, so win_wua fails to load, so I'm just wondering why it is done this way. Thanks
@twangboy twangboy added Bug broken, incorrect, or confusing behavior needs-triage labels Nov 1, 2022
@twangboy twangboy self-assigned this Nov 1, 2022
@twangboy twangboy added Execution-Module Windows Sulfur v3006.0 release code name and version and removed needs-triage labels Nov 1, 2022
@twangboy twangboy added this to the Sulphur v3006.0 milestone Nov 1, 2022
@twangboy twangboy mentioned this issue Nov 1, 2022
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@twangboy twangboy

Labels
Bug broken, incorrect, or confusing behavior Execution-Module Sulfur v3006.0 release code name and version Windows
Projects
None yet
Milestone
Sulfur v3006.0
Development

Successfully merging a pull request may close this issue.

Allow win_wua to load if CryptSvc set to Manual Start twangboy/salt
1 participant
@twangboy