Github user aledsage commented on a diff in the pull request:

    https://github.com/apache/brooklyn-docs/pull/66#discussion_r63842330
  
    --- Diff: guide/yaml/winrm/index.md ---
    @@ -271,6 +271,16 @@ config like `pre.install.reboot.required` and 
`install.reboot.required`. If requ
     installation commands can be split between the pre-install, install and 
post-install phases
     in order to do a reboot at the appropriate point of the VM setup.
     
    +We Strongly recommend to **write blueprints in a way that they do NOT 
restart automatically windows**
    +
    +Using install commands that restart the Operating System automatically 
through a forked process
    +will make the blueprint to behave inconsistently and report wrongly 
failure.
    +
    +To avoid such cases manually call the command `shutdown /r` or the 
powershell one `Restart-Computer`.
    --- End diff --
    
    See the config option "install.reboot.required". That handles reboot better 
than executing this command directly.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to