diff --git a/engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/AncientDataMotionStrategy.java b/engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/AncientDataMotionStrategy.java index 9c8f4df3b6e..7c930fb34c9 100644 --- a/engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/AncientDataMotionStrategy.java +++ b/engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/AncientDataMotionStrategy.java @@ -348,8 +348,9 @@ public class AncientDataMotionStrategy implements DataMotionStrategy { // directly to s3 ImageStoreEntity imageStore = (ImageStoreEntity)dataStoreMgr.getImageStoreWithFreeCapacity(destScope.getScopeId()); if (imageStore == null || !imageStore.getProtocol().equalsIgnoreCase("nfs") && !imageStore.getProtocol().equalsIgnoreCase("cifs")) { - s_logger.debug("can't find a nfs (or cifs) image store to satisfy the need for a staging store"); - return null; + String errMsg = "can't find a nfs (or cifs) image store to satisfy the need for a staging store"; + Answer answer = new Answer(null, false, errMsg); + return answer; } DataObject objOnImageStore = imageStore.create(srcData); diff --git a/ui/scripts/zoneWizard.js b/ui/scripts/zoneWizard.js index 5b7c25f87ea..785f3884765 100755 --- a/ui/scripts/zoneWizard.js +++ b/ui/scripts/zoneWizard.js @@ -4173,7 +4173,7 @@ } else if (args.data.returnedZone.networktype == "Advanced") { //update VLAN in physical network(s) in advanced zone var physicalNetworksHavingGuestIncludingVlan = []; $(args.data.physicalNetworks).each(function() { - if (this.guestConfiguration != null && this.guestConfiguration.vlanRangeStart != null && this.guestConfiguration.vlanRangeStart.length > 0) { + if (args.data.guestTraffic != null && args.data.guestTraffic.vlanRangeStart != null && args.data.guestTraffic.vlanRangeStart.length > 0) { physicalNetworksHavingGuestIncludingVlan.push(this); } }); @@ -4186,10 +4186,10 @@ var updatedCount = 0; $(physicalNetworksHavingGuestIncludingVlan).each(function() { var vlan; - if (this.guestConfiguration.vlanRangeEnd == null || this.guestConfiguration.vlanRangeEnd.length == 0) - vlan = this.guestConfiguration.vlanRangeStart; + if (args.data.guestTraffic.vlanRangeEnd == null || args.data.guestTraffic.vlanRangeEnd.length == 0) + vlan = args.data.guestTraffic.vlanRangeStart; else - vlan = this.guestConfiguration.vlanRangeStart + "-" + this.guestConfiguration.vlanRangeEnd; + vlan = args.data.guestTraffic.vlanRangeStart + "-" + args.data.guestTraffic.vlanRangeEnd; var originalId = this.id; var returnedId;