Skip to content

Commit 0a50359

Browse files
KvmAutoConvergence configkey backported from master (#12)
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
1 parent 8d4c836 commit 0a50359

File tree

4 files changed

+10
-15
lines changed

4 files changed

+10
-15
lines changed

engine/components-api/src/com/cloud/storage/StorageManager.java

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,14 @@ public interface StorageManager extends StorageService {
5858
true,
5959
ConfigKey.Scope.Global,
6060
null);
61-
61+
ConfigKey<Boolean> KvmAutoConvergence = new ConfigKey<>(Boolean.class,
62+
"kvm.auto.convergence",
63+
"Storage",
64+
"false",
65+
"Setting this to 'true' allows KVM to use auto convergence to complete VM migration (libvirt version 1.2.3+ and QEMU version 1.6+)",
66+
true,
67+
ConfigKey.Scope.Global,
68+
null);
6269
/**
6370
* Returns a comma separated list of tags for the specified storage pool
6471
* @param poolId

engine/storage/datamotion/src/org/apache/cloudstack/storage/motion/StorageSystemDataMotionStrategy.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1138,11 +1138,7 @@ public void copyAsync(Map<VolumeInfo, DataStore> volumeDataStoreMap, VirtualMach
11381138
migrateCommand.setWait(StorageManager.KvmStorageOnlineMigrationWait.value());
11391139
migrateCommand.setMigrateStorage(migrateStorage);
11401140
migrateCommand.setMigrateStorageManaged(managedStorageDestination);
1141-
1142-
String autoConvergence = _configDao.getValue(Config.KvmAutoConvergence.toString());
1143-
boolean kvmAutoConvergence = Boolean.parseBoolean(autoConvergence);
1144-
1145-
migrateCommand.setAutoConvergence(kvmAutoConvergence);
1141+
migrateCommand.setAutoConvergence(StorageManager.KvmAutoConvergence.value());
11461142

11471143
MigrateAnswer migrateAnswer = (MigrateAnswer)_agentMgr.send(srcHost.getId(), migrateCommand);
11481144

server/src/com/cloud/configuration/Config.java

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -914,14 +914,6 @@ public enum Config {
914914
"0",
915915
"Default disk I/O writerate in bytes per second allowed in User vm's disk.",
916916
null),
917-
KvmAutoConvergence(
918-
"Advanced",
919-
ManagementServer.class,
920-
Boolean.class,
921-
"kvm.auto.convergence",
922-
"false",
923-
"Setting this to 'true' allows KVM to use auto convergence to complete VM migration (libvirt version 1.2.3+ and QEMU version 1.6+)",
924-
null),
925917
ControlCidr(
926918
"Advanced",
927919
ManagementServer.class,

server/src/com/cloud/storage/StorageManagerImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2390,7 +2390,7 @@ public String getConfigComponentName() {
23902390
@Override
23912391
public ConfigKey<?>[] getConfigKeys() {
23922392
return new ConfigKey<?>[] {StorageCleanupInterval, StorageCleanupDelay, StorageCleanupEnabled,
2393-
KvmStorageOnlineMigrationWait};
2393+
KvmStorageOnlineMigrationWait, KvmAutoConvergence};
23942394
}
23952395

23962396
@Override

0 commit comments

Comments
 (0)