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 <[email protected]> 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/4.7 Commit: a767407fd285543dff5a99ec922ae00aa6ccb785 Parents: 4dabd13 47a23c2 Author: Remi Bergsma <[email protected]> Authored: Sat Jan 16 20:39:43 2016 +0100 Committer: Remi Bergsma <[email protected]> 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(-) ----------------------------------------------------------------------
