You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by re...@apache.org on 2016/01/16 21:40:17 UTC

[4/5] git commit: updated refs/heads/master to 1b8c464

Merge pull request #1293 from miguelaferreira/nsx-heath-check

Add Health Check Command to NSX pluginThe NSX plugin does not support the HeathCheckCommand. Instead it fakes a PingCommand as a call tot he control cluster status API.
However, we have seen in production that the management server will sometimes find the NSX controller to be behind on ping and that will trigger a HealthCheckCommand which will return with an unsupported command answer.
Once this happens the controller is put into Alert state and will not recover until the management sever is restarted.

In addition, during the investigation, there will be a null pointer exception due tot he fact that the NSX controllers do not live in a pod.

This PR tries to address those two issues.

* pr/1293:
  Implement CheckHealthCommand for NSX controllers
  Fix log message that refers to agent, not host
  Prevent NullPointerException when host does not belong to a pod

Signed-off-by: Remi Bergsma <gi...@remi.nl>


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

Branch: refs/heads/master
Commit: a767407fd285543dff5a99ec922ae00aa6ccb785
Parents: 4dabd13 47a23c2
Author: Remi Bergsma <gi...@remi.nl>
Authored: Sat Jan 16 20:39:43 2016 +0100
Committer: Remi Bergsma <gi...@remi.nl>
Committed: Sat Jan 16 20:39:43 2016 +0100

----------------------------------------------------------------------
 .../cloud/agent/manager/AgentManagerImpl.java   | 142 +++++++++----------
 .../NiciraCheckHealthCommandWrapper.java        |  63 ++++++++
 .../NiciraCheckHealthCommandWrapperTest.java    |  80 +++++++++++
 3 files changed, 214 insertions(+), 71 deletions(-)
----------------------------------------------------------------------