You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by bh...@apache.org on 2016/11/02 17:28:12 UTC
[2/2] git commit: updated refs/heads/4.8 to 68f22e2

Merge pull request #1694 from shapeblue/kvm-no-storage-failfast

CLOUDSTACK-9509: Host Connects Without StorageKVM hosts on shared storage failure was accepted by mgmt server with the
host state as Up, even though there was no primary/shared storage available on
it. This patch offers a quick fix by throwing an exception in the storage monitor
which connects storage pool on host. The failure is trapped by agent manager
that disconnects the agent without any investigation.

Based on Lab tests, KVM agent may take upto 2 minutes to attempt NFS mount when
the storage is inaccessible (firewalled, or shutdown) before returning back with
an error. It is safe to assume that this won't add pressure on mgmt server due to
several reconnection attempts, and KVM agent would retry reconnection every 2
minutes.

For such KVM hosts, where failure happens due to storage issues; they will be
briefly put in Alert state but will be mostly be in Connecting state during which
the KVM host attempts to mount/reconfigure NFS storage pool.

/cc @jburwell @karuturi
@blueorangutan package

* pr/1694:
  CLOUDSTACK-9509: Host Connects Without Storage

Signed-off-by: Rohit Yadav <ro...@shapeblue.com>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/68f22e2a
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/68f22e2a
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/68f22e2a

Branch: refs/heads/4.8
Commit: 68f22e2a438c5ec1b7a390632cb94256bfdb6938
Parents: 84e2825 32a397a
Author: Rohit Yadav <ro...@shapeblue.com>
Authored: Wed Nov 2 22:57:43 2016 +0530
Committer: Rohit Yadav <ro...@shapeblue.com>
Committed: Wed Nov 2 22:57:51 2016 +0530

----------------------------------------------------------------------
 .../agent/manager/AgentManagerImplTest.java     | 86 ++++++++++++++++++++
 .../storage/listener/StoragePoolMonitor.java    |  6 +-
 .../listener/StoragePoolMonitorTest.java        | 80 ++++++++++++++++++
 3 files changed, 170 insertions(+), 2 deletions(-)
----------------------------------------------------------------------