-
Notifications
You must be signed in to change notification settings - Fork 173
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
Promote the win_hotfix Module from community #701
Promote the win_hotfix Module from community #701
Conversation
…ble-collections#65) * removing chocolatey as they have moved to chocolatey.chocolatey * more chocolatey removals * fix sanity error * fixing one last time I hope
* Fix up docs after migration * Fix up sanity errors
* fix up sanity ignores * Bump ansible-windows dep * Fix bad change for win_region
* Rebalance the test targets * Make sure IIS test removes the service so our httptester works
* Update win_hotfix.ps1 * Fix up docs and return values Co-authored-by: Jordan Borean <jborean93@gmail.com>
Looks like the PR needs a rebase with the latest changes in the main branch. |
dc12ee3
to
4fa0434
Compare
branch rebased and changes added |
module: win_hotfix | ||
short_description: Install and uninstalls Windows hotfixes | ||
description: | ||
- Install, uninstall a Windows hotfix. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs the version added entry.
plugins/modules/win_hotfix.py
Outdated
community.windows.win_hotfix: | ||
source: C:\temp\windows8.1-kb3172729-x64_e8003822a7ef4705cbb65623b72fd3cec73fe222.msu | ||
state: present | ||
register: hotfix_install | ||
|
||
- ansible.windows.win_reboot: | ||
when: hotfix_install.reboot_required | ||
|
||
- name: Install hotfix validating KB | ||
community.windows.win_hotfix: | ||
hotfix_kb: KB3172729 | ||
source: C:\temp\windows8.1-kb3172729-x64_e8003822a7ef4705cbb65623b72fd3cec73fe222.msu | ||
state: present | ||
register: hotfix_install | ||
|
||
- ansible.windows.win_reboot: | ||
when: hotfix_install.reboot_required | ||
|
||
- name: Install hotfix validating Identifier | ||
community.windows.win_hotfix: | ||
hotfix_identifier: Package_for_KB3172729~31bf3856ad364e35~amd64~~6.3.1.0 | ||
source: C:\temp\windows8.1-kb3172729-x64_e8003822a7ef4705cbb65623b72fd3cec73fe222.msu | ||
state: present | ||
register: hotfix_install | ||
|
||
- ansible.windows.win_reboot: | ||
when: hotfix_install.reboot_required | ||
|
||
- name: Uninstall hotfix with Identifier | ||
community.windows.win_hotfix: | ||
hotfix_identifier: Package_for_KB3172729~31bf3856ad364e35~amd64~~6.3.1.0 | ||
state: absent | ||
register: hotfix_uninstall | ||
|
||
- ansible.windows.win_reboot: | ||
when: hotfix_uninstall.reboot_required | ||
|
||
- name: Uninstall hotfix with KB (not recommended) | ||
community.windows.win_hotfix: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs to change from community.windows.win_hotfix
to ansible.windows.win_hotfix
.
4fa0434
to
eb3b479
Compare
Changes were added again after being lost while rebasing |
SUMMARY
Promote the win_hotfix Module from community
Updating documentation examples.
ISSUE TYPE
COMPONENT NAME
plugins/modules/win_hotfix.ps1
plugins/modules/win_hotfix.py
tests/integration/targets/win_hotfix/aliases
tests/integration/targets/win_hotfix/defaults/main.yml
tests/integration/targets/win_hotfix/tasks/main.yml
tests/integration/targets/win_hotfix/tasks/tests.yml
tests/integration/targets/win_hotfix/tasks/tests_2012R2.yml