search

sous-chefs / powershell

Development repository for the powershell cookbook
https://supermarket.chef.io/cookbooks/powershell
Apache License 2.0
110 stars 88 forks source link

New windows_package 'success_codes' attributes failing #33

Closed jmauro closed 9 years ago

jmauro commented 9 years ago

Hi,

There are some issue with the latest version of windows cookbook (now that it uses "success_codes").

       ==> default: ---- End output of start "" /wait "\var\chef\cache\Windows6.1-KB2819745-x64-MultiPkg.msu" /quiet /norestart & exit %%ERRORLEVEL%% ----
       ==> default: Ran start "" /wait "\var\chef\cache\Windows6.1-KB2819745-x64-MultiPkg.msu" /quiet /norestart & exit %%ERRORLEVEL%% returned 3010
       ==> default: 
       ==> default: ================================================================================
       ==> default: Error executing action `install` on resource 'windows_package[Windows Management Framework Core 4.0]'
       ==> default: ================================================================================
       ==> default: 
       ==> default: Mixlib::ShellOut::ShellCommandFailed
       ==> default: ------------------------------------
       ==> default: Expected process to exit with [0, 42, 127], but received '3010'
       ==> default: ---- Begin output of start "" /wait "\var\chef\cache\Windows6.1-KB2819745-x64-MultiPkg.msu" /quiet /norestart & exit %%ERRORLEVEL%% ----
       ==> default: STDOUT: 
       ==> default: STDERR: 
       ==> default: ---- End output of start "" /wait "\var\chef\cache\Windows6.1-KB2819745-x64-MultiPkg.msu" /quiet /norestart & exit %%ERRORLEVEL%% ----
       ==> default: Ran start "" /wait "\var\chef\cache\Windows6.1-KB2819745-x64-MultiPkg.msu" /quiet /norestart & exit %%ERRORLEVEL%% returned 3010

I have created a PR to fix the issue.

Regards, JM

jmauro commented 9 years ago

Here is the PR: https://github.com/opscode-cookbooks/powershell/pull/34

NimishaS commented 9 years ago

@jmauro , this issue has been resolved and is merged in the master branch. I think we have missed this change in powershell2 cookbook by mistake. All other cookbooks have it.

csharris commented 9 years ago

Hi, I am also currently blocked on this issue. I see that as @NimishaS stated, the fix was missed in powershell2. Is there any plan to make the fix?

NimishaS commented 9 years ago

@csharris , sorry for the delay. Required changes are merged in master branch now.

jaym commented 9 years ago

It looks like this has been fixed in master