Skip to content

Commit bca8556

Browse files
author
Rakesh Venkatesh
committed
rename variables
1 parent 50f4a8a commit bca8556

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -442,9 +442,9 @@ public class ConfigurationManagerImpl extends ManagerBase implements Configurati
442442
public static final ConfigKey<Boolean> SET_HOST_DOWN_TO_MAINTENANCE = new ConfigKey<Boolean>(Boolean.class, "set.host.down.to.maintenance", "Advanced", "false",
443443
"Indicates whether the host in down state can be put into maintenance state so thats its not enabled after it comes back.",
444444
true, ConfigKey.Scope.Zone, null);
445-
public static final ConfigKey<Boolean> EnableAccountSettingsForDomain = new ConfigKey<Boolean>(Boolean.class, "enable.account.settings.for.domain", "Advanced", "false",
445+
public static final ConfigKey<Boolean> ENABLE_ACCOUNT_SETTINGS_FOR_DOMAIN = new ConfigKey<Boolean>(Boolean.class, "enable.account.settings.for.domain", "Advanced", "false",
446446
"Indicates whether to add account settings for domain. If true, account settings will be added to domain settings, all accounts in the domain will inherit the domain setting if account setting is not set.", true, ConfigKey.Scope.Global, null);
447-
public static final ConfigKey<Boolean> EnableDomainSettingsForChildDomain = new ConfigKey<Boolean>(Boolean.class, "enable.domain.settings.for.child.domain", "Advanced", "false",
447+
public static final ConfigKey<Boolean> ENABLE_DOMAIN_SETTINGS_FOR_CHILD_DOMAIN = new ConfigKey<Boolean>(Boolean.class, "enable.domain.settings.for.child.domain", "Advanced", "false",
448448
"Indicates whether the settings of parent domain should be applied for child domain. If true, the child domain will get value from parent domain if its not configured in child domain else global value is taken.",
449449
true, ConfigKey.Scope.Global, null);
450450

@@ -889,7 +889,7 @@ private String validateConfigurationValue(final String name, String value, final
889889
final String configScope = cfg.getScope();
890890
if (scope != null) {
891891
if (!configScope.contains(scope) &&
892-
!(EnableAccountSettingsForDomain.value() && configScope.contains(ConfigKey.Scope.Account.toString()) &&
892+
!(ENABLE_ACCOUNT_SETTINGS_FOR_DOMAIN.value() && configScope.contains(ConfigKey.Scope.Account.toString()) &&
893893
scope.equals(ConfigKey.Scope.Domain.toString()))) {
894894
s_logger.error("Invalid scope id provided for the parameter " + name);
895895
return "Invalid scope id provided for the parameter " + name;
@@ -6875,7 +6875,7 @@ public ConfigKey<?>[] getConfigKeys() {
68756875
return new ConfigKey<?>[] {SystemVMUseLocalStorage, IOPS_MAX_READ_LENGTH, IOPS_MAX_WRITE_LENGTH,
68766876
BYTES_MAX_READ_LENGTH, BYTES_MAX_WRITE_LENGTH, ADD_HOST_ON_SERVICE_RESTART_KVM, SET_HOST_DOWN_TO_MAINTENANCE, VM_SERVICE_OFFERING_MAX_CPU_CORES,
68776877
VM_SERVICE_OFFERING_MAX_RAM_SIZE, VM_USERDATA_MAX_LENGTH, MIGRATE_VM_ACROSS_CLUSTERS,
6878-
EnableAccountSettingsForDomain, EnableDomainSettingsForChildDomain
6878+
ENABLE_ACCOUNT_SETTINGS_FOR_DOMAIN, ENABLE_DOMAIN_SETTINGS_FOR_CHILD_DOMAIN
68796879
};
68806880
}
68816881
}

server/src/main/java/com/cloud/server/ManagementServerImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2041,7 +2041,7 @@ public Pair<List<? extends Configuration>, Integer> searchForConfigurations(fina
20412041

20422042
if (scope != null && !scope.isEmpty()) {
20432043
// getting the list of parameters at requested scope
2044-
if (ConfigurationManagerImpl.EnableAccountSettingsForDomain.value()
2044+
if (ConfigurationManagerImpl.ENABLE_ACCOUNT_SETTINGS_FOR_DOMAIN.value()
20452045
&& scope.equals(ConfigKey.Scope.Domain.toString())) {
20462046
sc.addAnd("scope", SearchCriteria.Op.IN, ConfigKey.Scope.Domain.toString(), ConfigKey.Scope.Account.toString());
20472047
} else {

0 commit comments

Comments
 (0)