Skip to content

Commit 140cd7c

Browse files
author
Jayapal
committed
CLOUDSTACK-8324: Fixed config key in multiple places
1 parent 14d5363 commit 140cd7c

4 files changed

Lines changed: 5 additions & 8 deletions

File tree

api/src/com/cloud/vm/UserVmService.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,12 +55,9 @@
5555
import com.cloud.user.Account;
5656
import com.cloud.uservm.UserVm;
5757
import com.cloud.utils.exception.ExecutionException;
58-
import org.apache.cloudstack.framework.config.ConfigKey;
5958

6059
public interface UserVmService {
6160

62-
static final ConfigKey<String> VmConfigDriveLabel = new ConfigKey<String>("Hidden", String.class, "vm.configdrive.label", "config",
63-
"The default lable name for the config drive", false);
6461

6562
/**
6663
* Destroys one virtual machine

engine/api/src/com/cloud/vm/VirtualMachineManager.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,9 @@ public interface VirtualMachineManager extends Manager {
5252
"If set to true, StartCommand, StopCommand, CopyCommand, MigrateCommand will be synchronized on the agent side."
5353
+ " If set to false, these commands become asynchronous. Default value is false.", false);
5454

55+
static final ConfigKey<String> VmConfigDriveLabel = new ConfigKey<String>("Hidden", String.class, "vm.configdrive.label", "config",
56+
"The default lable name for the config drive", false);
57+
5558
public interface Topics {
5659
public static final String VM_POWER_STATE = "vm.powerstate";
5760
}

engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -361,9 +361,6 @@ public void setHostAllocators(final List<HostAllocator> hostAllocators) {
361361
Integer.class, "vm.job.report.interval", "60",
362362
"Interval to send application level pings to make sure the connection is still working", false);
363363

364-
static final ConfigKey<String> VmConfigDriveLabel = new ConfigKey<String>("Hidden", String.class, "vm.configdrive.label", "config",
365-
"The default lable name for the config drive", false);
366-
367364
ScheduledExecutorService _executor = null;
368365

369366
protected long _nodeId;

server/src/com/cloud/vm/UserVmManagerImpl.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3517,7 +3517,7 @@ public boolean finalizeVirtualMachineProfile(VirtualMachineProfile profile, Depl
35173517
String configDriveIsoRootFolder = "/tmp";
35183518
String isoFile = configDriveIsoRootFolder + "/" + vmName + "/configDrive/" + vmName + ".iso";
35193519
profile.setVmData(vmData);
3520-
profile.setConfigDriveLabel(VmConfigDriveLabel.value());
3520+
profile.setConfigDriveLabel(VirtualMachineManager.VmConfigDriveLabel.value());
35213521
profile.setConfigDriveIsoRootFolder(configDriveIsoRootFolder);
35223522
profile.setConfigDriveIsoFile(isoFile);
35233523
}
@@ -5292,7 +5292,7 @@ public String getConfigComponentName() {
52925292

52935293
@Override
52945294
public ConfigKey<?>[] getConfigKeys() {
5295-
return new ConfigKey<?>[] {EnableDynamicallyScaleVm, VmIpFetchWaitInterval, VmIpFetchTrialMax, VmIpFetchThreadPoolMax, VmConfigDriveLabel};
5295+
return new ConfigKey<?>[] {EnableDynamicallyScaleVm, VmIpFetchWaitInterval, VmIpFetchTrialMax, VmIpFetchThreadPoolMax};
52965296
}
52975297

52985298
@Override

0 commit comments

Comments
 (0)