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

Workaround broken OS patch KB2918614 #101

Merged
merged 1 commit into from
Oct 4, 2014
Merged

Conversation

adamedx
Copy link

@adamedx adamedx commented Oct 4, 2014

This OS patch is causing the chef-client msi installation on Windows to fail when bootstrapping over winrm. The issue is apparently a regression in Windows Installer:
http://support.microsoft.com/kb/2918614
http://answers.microsoft.com/en-us/windows/forum/windows8_1-windows_install/kb2918614-breaks-windows-installer-service/3d75a1c2-114a-4241-a527-35b536edc158

The workaround here is to detect if there's a failure to install, then use task scheduler to install the msi. Because task scheduler is asynchronous, we use the waitfor command to send a signal when the installation is done, and the bootstrap script waits for it before proceeding to try to run Chef. A downside of this approach is that it is difficult to obtain the return value of the task execution.

@adamedx adamedx force-pushed the adamedx/os-patch-workaround branch 2 times, most recently from d0a2fe0 to a1206b3 Compare October 4, 2014 18:56
Remove puts

Removing incorrect quote character

Explain quoting issues in task retries
@adamedx adamedx force-pushed the adamedx/os-patch-workaround branch from a1206b3 to 14c8e1b Compare October 4, 2014 19:49
adamedx pushed a commit that referenced this pull request Oct 4, 2014
Workaround broken OS patch KB2918614
@adamedx adamedx merged commit 41cc8a2 into master Oct 4, 2014
@adamedx adamedx deleted the adamedx/os-patch-workaround branch October 4, 2014 20:01
adamedx pushed a commit that referenced this pull request Oct 4, 2014
Workaround broken OS patch KB2918614
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.

1 participant