[jboss-jira] [JBoss JIRA] (WFCORE-1715) HostProcessReloadHandler does not reset the HostRunningModeControl's restartMode

[Brian Stansberry (JIRA)]

    [ https://issues.jboss.org/browse/WFCORE-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13279320#comment-13279320 ] 

Brian Stansberry commented on WFCORE-1715:
------------------------------------------

Note that if we go into the "shutdownServers" logic we need to consider what the correct default timeout in the HostRunningModeControl should be. I believe zero is fine (shutdown immediately).

> HostProcessReloadHandler does not reset the HostRunningModeControl's restartMode
> --------------------------------------------------------------------------------
>
>                 Key: WFCORE-1715
>                 URL: https://issues.jboss.org/browse/WFCORE-1715
>             Project: WildFly Core
>          Issue Type: Bug
>          Components: Domain Management
>    Affects Versions: 2.2.0.Final, 3.0.0.Alpha5
>            Reporter: Yeray Santana Borges
>             Fix For: 3.0.0.Alpha6
>
>
> The ReloadContext created by HostProcessReloadHandler sets the HostRunningModeControl's restartMode but then it never gets restored to the default value. The doReload() method of the ReloadContext should restore it. This ensures the ServerInventoryService.stop() only uses the value set by HostProcessReloadHandler once, for that one reload.
> A concern here is that ServerInventoryService only goes into its "shutdownServers" logic if the restartMode == RestartMode.SERVERS. Which, due to this bug, it will be following any HC reload. But if we restore the default value of restartMode, that is null, and null != RestartMode.SERVERS. So the ""shutdownServers" logic will no longer kick in. But should it? Should the default value of restartMode be "null"? Or should it be RestartMode.SERVERS? If we change the default from null, then the behavior when no reload has happened will change.
> Basically we need to decide whether the "shutdownServers" logic should happen by default.



--
This message was sent by Atlassian JIRA
(v6.4.11#64026)