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

Promote the win_hotfix Module from community #701

Merged
merged 11 commits into from
Dec 8, 2024

Conversation

Yaish25491
Copy link
Contributor

SUMMARY

Promote the win_hotfix Module from community
Updating documentation examples.

ISSUE TYPE
  • New Module Pull Request
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

jborean93 and others added 10 commits March 6, 2020 07:14
…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>
tests/integration/targets/win_hotfix/aliases Outdated Show resolved Hide resolved
plugins/modules/win_hotfix.py Outdated Show resolved Hide resolved
plugins/modules/win_hotfix.py Outdated Show resolved Hide resolved
@jborean93
Copy link
Collaborator

Looks like the PR needs a rebase with the latest changes in the main branch.

@Yaish25491 Yaish25491 force-pushed the aap-36369-hotfix branch 2 times, most recently from dc12ee3 to 4fa0434 Compare December 5, 2024 14:43
@Yaish25491
Copy link
Contributor Author

branch rebased and changes added

module: win_hotfix
short_description: Install and uninstalls Windows hotfixes
description:
- Install, uninstall a Windows hotfix.
Copy link
Collaborator

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.

Comment on lines 70 to 108
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:
Copy link
Collaborator

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.

@Yaish25491
Copy link
Contributor Author

Changes were added again after being lost while rebasing

@jborean93 jborean93 merged commit 25ebfa6 into ansible-collections:main Dec 8, 2024
39 checks passed
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.

4 participants