Skip to content

Commit 689e529

Browse files
committed
Merge release branch 4.13 to master
* 4.13: Fixed guest vlan range going missing when using zone wizzard (#4042) Volume migration (#4043)
2 parents 8e4be6d + 7b7caf5 commit 689e529

2 files changed

Lines changed: 7 additions & 6 deletions

File tree

engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/AncientDataMotionStrategy.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -348,8 +348,9 @@ protected Answer copyVolumeBetweenPools(DataObject srcData, DataObject destData)
348348
// directly to s3
349349
ImageStoreEntity imageStore = (ImageStoreEntity)dataStoreMgr.getImageStoreWithFreeCapacity(destScope.getScopeId());
350350
if (imageStore == null || !imageStore.getProtocol().equalsIgnoreCase("nfs") && !imageStore.getProtocol().equalsIgnoreCase("cifs")) {
351-
s_logger.debug("can't find a nfs (or cifs) image store to satisfy the need for a staging store");
352-
return null;
351+
String errMsg = "can't find a nfs (or cifs) image store to satisfy the need for a staging store";
352+
Answer answer = new Answer(null, false, errMsg);
353+
return answer;
353354
}
354355

355356
DataObject objOnImageStore = imageStore.create(srcData);

ui/scripts/zoneWizard.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4173,7 +4173,7 @@
41734173
} else if (args.data.returnedZone.networktype == "Advanced") { //update VLAN in physical network(s) in advanced zone
41744174
var physicalNetworksHavingGuestIncludingVlan = [];
41754175
$(args.data.physicalNetworks).each(function() {
4176-
if (this.guestConfiguration != null && this.guestConfiguration.vlanRangeStart != null && this.guestConfiguration.vlanRangeStart.length > 0) {
4176+
if (args.data.guestTraffic != null && args.data.guestTraffic.vlanRangeStart != null && args.data.guestTraffic.vlanRangeStart.length > 0) {
41774177
physicalNetworksHavingGuestIncludingVlan.push(this);
41784178
}
41794179
});
@@ -4186,10 +4186,10 @@
41864186
var updatedCount = 0;
41874187
$(physicalNetworksHavingGuestIncludingVlan).each(function() {
41884188
var vlan;
4189-
if (this.guestConfiguration.vlanRangeEnd == null || this.guestConfiguration.vlanRangeEnd.length == 0)
4190-
vlan = this.guestConfiguration.vlanRangeStart;
4189+
if (args.data.guestTraffic.vlanRangeEnd == null || args.data.guestTraffic.vlanRangeEnd.length == 0)
4190+
vlan = args.data.guestTraffic.vlanRangeStart;
41914191
else
4192-
vlan = this.guestConfiguration.vlanRangeStart + "-" + this.guestConfiguration.vlanRangeEnd;
4192+
vlan = args.data.guestTraffic.vlanRangeStart + "-" + args.data.guestTraffic.vlanRangeEnd;
41934193

41944194
var originalId = this.id;
41954195
var returnedId;

0 commit comments

Comments
 (0)