This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git
The following commit(s) were added to refs/heads/master by this push: new beb0422 fix incorrect iscsi path stat for managed storage (#3181) beb0422 is described below commit beb0422dd54bb288b12b07c9f7d9689df38fb599 Author: skattoju3 <45469763+skatto...@users.noreply.github.com> AuthorDate: Fri Mar 29 05:25:01 2019 -0400 fix incorrect iscsi path stat for managed storage (#3181) ## Description The issue was that an Incorrect iscsi path was being passed for managed storage pools when collecting volume stats. Storage pools normally have a UUID based path while managed storage pools require an IQN based path --- server/src/main/java/com/cloud/server/StatsCollector.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/server/src/main/java/com/cloud/server/StatsCollector.java b/server/src/main/java/com/cloud/server/StatsCollector.java index 8e2bc7e..80b745e 100644 --- a/server/src/main/java/com/cloud/server/StatsCollector.java +++ b/server/src/main/java/com/cloud/server/StatsCollector.java @@ -928,7 +928,11 @@ public class StatsCollector extends ManagerBase implements ComponentMethodInterc List<String> volumeLocators = new ArrayList<String>(); for (VolumeVO volume : volumes) { if (volume.getFormat() == ImageFormat.QCOW2) { - volumeLocators.add(volume.getUuid()); + if (pool.isManaged()) { + volumeLocators.add(volume.getPath()); + } else { + volumeLocators.add(volume.getUuid()); + } } else if (volume.getFormat() == ImageFormat.VHD) { volumeLocators.add(volume.getPath()); } else if (volume.getFormat() == ImageFormat.OVA) {