Powershell install doesn't trigger reboot

[palfrey]

Powershell install doesn't trigger reboot

[juliandunn]

Hmm... I've had it just reboot the box by itself after it installs. Did that not happen for you?

[palfrey]

Which version of powershell is that for? Also, were you installing anything else? AFAIK, there's no direct support for reboot in this cookbook, but something else may be doing it accidentally...

[christrotter]

Running into the same issue. 2008R2 machine w. PSv2.0, using this to bring it to PSv4.0. Works fine after a reboot, but ideally the cookbook would handle it.

[siddheshwar-more]

We have 'powershell::windows_reboot recipe'. Please check, it might help you. Currently we used this in PS4 recipe.
https://github.com/chef-windows/powershell/blob/master/recipes/powershell4.rb#L34

[palfrey]

Really, it should just do the windows_reboot notify all the time, and then leave it up to the user to do the allow_pending_reboots/allow_reboot_on_failure attributes if they want it to trigger

[christrotter]

Ah, I'm getting my cookbooks via 'knife cookbook site install powershell' - that version doesn't have the reboot code. Is there a better way to do this? (I understand Berkshelf is recommended, but haven't had a chance to investigate yet)

[NimishaS]

@palfrey, Thanks for pointing this out! Right now user can set 'allow_pending_reboot' and 'allow_reboot_on_failure' attributes only by modifying windows_reboot recipe itself.

We have powershell::dsc recipe which sets up environment ( i.e install PS > 4.0) for windows DSC. And we implemented powershell::windows_reboot recipe for doing reboot after required environment is set-up. We will add windows_reboot support for other PS version as well.

[tjnicholas]

With reference to @christrotter's comment, it looks like master is 60 commits ahead of the 3.0.7 tag.

Can we get a new version released/tagged? Lots of good stuff in those 60 commits... @NimishaS

[NimishaS]

@tjnicholas , yes we'll release the changes.

[NimishaS]

@palfrey , @christrotter Reboot code has been added to all powershell cookbooks in the master branch. It's not released yet. Please use master branch code till the release.

[tas50]

Closing this out since the commit was merged. We'll try to get a release out shortly.