Skip to content

Windows Ansible playbook optimization - #576

Closed
piccoloaiutante wants to merge 2 commits into
nodejs:masterfrom
piccoloaiutante:windows-provisioning-optimization
Closed

piccoloaiutante wants to merge 2 commits into
nodejs:masterfrom
piccoloaiutante:windows-provisioning-optimization

Conversation

@piccoloaiutante

Copy link
Copy Markdown
Member

Starting from suggestions for a good first contribution that I received (#495) I've added few extra steps to Windows Ansible scripts:

  • Force Windows to do update
  • Reboot machine if necessary after update installation
  • Reboot machine after Visual Studio installation

As per #574 @joaocgreis can you test it on Windows 2008 Server R2 ? I already tested it on windows 10 and Windows Server 2012.

@jbergstroem

Copy link
Copy Markdown
Member

Interesting! Question: does the reboot stuff really work? I've been fighting with ansible about waiting at reboots (for the host to come back up).

@piccoloaiutante

piccoloaiutante commented Dec 20, 2016 •

Copy link
Copy Markdown
Member Author

@jbergstroem yes it worked on the two operating systems that I tested but I want to have a definitive response for @joaocgreis about Windows 2008 Server.

Here is the doc for the windows version. Have you tried to expand the reboot_timeout_sec?
https://docs.ansible.com/ansible/win_reboot_module.html

@jbergstroem

Copy link
Copy Markdown
Member

@piccoloaiutante no, i haven't tried windows rebooting at all -- its just a nightmare for !windows :\

joaocgreis pushed a commit that referenced this pull request Jan 7, 2017
Added update installation and reboot as first step and an extra reboot
after visual studio installation

PR-URL: #576
Reviewed-By: João Reis <reis@janeasystems.com>
@joaocgreis

Copy link
Copy Markdown
Member

Landed in 5cd0f41

I took the liberty to clean up a bit and add a small header (<50char) to the commit message, hope it's ok. I tested in Windows 2008R2, the reboot works beautifully. Thanks @piccoloaiutante !

@joaocgreis joaocgreis closed this Jan 7, 2017
@piccoloaiutante

Copy link
Copy Markdown
Member Author

Excellent @joaocgreis now moving on generating .rdp and/or .remmina files.

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.

3 participants