Skip to content

Add the ability to remove a KB from the DISM module - #62615

Merged
Ch3LL merged 4 commits into
saltstack:masterfrom
twangboy:fix_62366
Oct 4, 2022
Merged

Add the ability to remove a KB from the DISM module#62615
Ch3LL merged 4 commits into
saltstack:masterfrom
twangboy:fix_62366

Conversation

@twangboy

@twangboy twangboy commented Sep 2, 2022

Copy link
Copy Markdown
Contributor

What does this PR do?

Adds the ability to remove a KB using the DISM module by passing just the KB number
Adds a state for doing the same thing

What issues does this PR fix or reference?

Fixes: #62366

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes

@twangboy
twangboy requested a review from a team as a code owner September 2, 2022 17:42
@twangboy
twangboy requested review from MKLeb and removed request for a team September 2, 2022 17:42
@twangboy
twangboy force-pushed the fix_62366 branch 2 times, most recently from 2065e75 to ef4ce41 Compare September 2, 2022 17:52
@twangboy
twangboy force-pushed the fix_62366 branch 4 times, most recently from 22f22ea to 75051e5 Compare September 27, 2022 15:17
@twangboy twangboy added the Sulfur v3006.0 release code name and version label Sep 27, 2022
@twangboy twangboy added this to the Sulphur v3006.0 milestone Sep 27, 2022
@Ch3LL

Ch3LL commented Sep 29, 2022

Copy link
Copy Markdown
Contributor

looks like there are still related failing tests

Comment thread salt/modules/win_dism.py Outdated
Comment thread salt/modules/win_dism.py Outdated
Comment thread salt/states/win_dism.py Outdated
Adds the ability to remove a KB using the DISM module by passing
just the KB number
Adds a state for doing the same thing
@Ch3LL
Ch3LL merged commit 79b07bb into saltstack:master Oct 4, 2022
@twangboy
twangboy deleted the fix_62366 branch March 23, 2023 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Sulfur v3006.0 release code name and version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] win_wusa.uninstall does not work, /quiet /kb combined no longer supported

2 participants