cloudstack-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (CLOUDSTACK-9458) Some VMs are being stopped when agent is reconnecting
Date Wed, 17 Aug 2016 15:39:20 GMT


ASF GitHub Bot commented on CLOUDSTACK-9458:

Github user jburwell commented on a diff in the pull request:
    --- Diff: server/src/com/cloud/ha/ ---
    @@ -497,7 +498,7 @@ protected Long restart(final HaWorkVO work) {
                     boolean fenced = false;
                     if (alive == null) {
    -                    s_logger.debug("Fencing off VM that we don't know the state of");
    +                    s_logger.debug("Fencing off VM " + vm + " that we don't know the
state of");
    --- End diff --
    Please wrap in a isDebugEnabled check to avoid expensive string concatenation when DEBUG
is off.  Also, should this message be a WARN instead of a DEBUG?

> Some VMs are being stopped when agent is reconnecting
> -----------------------------------------------------
>                 Key: CLOUDSTACK-9458
>                 URL:
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the default.) 
>            Reporter: Marc-Aurèle Brothier
>            Assignee: Marc-Aurèle Brothier
> If you loose the communication between the management server and one of the agent for
a few minutes, even though HA mode is not active the HighAvailibilityManager kicks in and
start to schedule vm restart. Those tasks are being inserted as async job in the DB and if
the agent comes back online during the time the jobs are still in the async table, they are
pushed to the agent and shuts down the VMs. Then since HA is not active, the VM are not restarted.
> The expected behavior in my opinion is that the VM should not be restarted at all if
HA mode is not active on them, and let the agent update the VM state with the power report.
> The bug lies in {{HighAvailibilityManagerImpl.scheduleRestartForVmsOnHost(final HostVO
host, boolean investigate)}}, PR will follow.

This message was sent by Atlassian JIRA

View raw message