From 1c688f40e134e2f65fe11bb37946e550feb896ca Mon Sep 17 00:00:00 2001 From: Anshul Gangwar Date: Thu, 7 Aug 2014 11:36:45 +0530 Subject: [PATCH] CLOUDSTACK-7277: [Hyper-V] For vmsync first VMs are listed and their power state is determined. To exclude hosted system we filter the result on Caption='Virtual Machine' but this string is locale dependent so it may not not work properly for locales other than english. To overcome this now we started using ProcessId >= 0 filter --- .../HypervResource/HypervResourceController.cs | 5 ++++- .../DotNet/ServerResource/HypervResource/WmiCallsV2.cs | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs b/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs index 62f5355c658..6c6ec56f424 100644 --- a/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs +++ b/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs @@ -2434,7 +2434,10 @@ namespace HypervResource logger.Error(details, sysEx); } - return JArray.FromObject(hostVmStateReport); + var answer = JArray.FromObject(hostVmStateReport); + logger.Info(String.Format("{0}: {1}",CloudStackTypes.HostVmStateReportCommand, answer.ToString())); + + return answer; } } diff --git a/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs b/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs index b2a7b454f12..f9b6e343965 100644 --- a/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs +++ b/plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs @@ -2315,7 +2315,7 @@ namespace HypervResource public ComputerSystem.ComputerSystemCollection GetComputerSystemCollection() { - var wmiQuery = String.Format("Caption=\"Virtual Machine\""); + var wmiQuery = String.Format("ProcessId >= 0"); return ComputerSystem.GetInstances(wmiQuery); }