sureshanaparti commented on code in PR #5797:
URL: https://github.com/apache/cloudstack/pull/5797#discussion_r847073231
##########
server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java:
##########
@@ -7311,6 +7319,82 @@ public String getConfigComponentName() {
};
}
+ @Override
+ public String getConfigurationType(final String configName) {
+ final ConfigurationVO cfg = _configDao.findByName(configName);
+ if (cfg == null) {
+ s_logger.error("Configuration " + configName + " not found");
+ return Configuration.ValueType.String.name();
+ }
+
+ if (weightBasedParametersForValidation.contains(configName)) {
+ return Configuration.ValueType.Range.name();
+ }
+
+ Class<?> type = null;
+ final Config c = Config.getConfig(configName);
+ if (c == null) {
+ s_logger.warn("Configuration " + configName + " no found. Perhaps
moved to ConfigDepot");
+ final ConfigKey<?> configKey = _configDepot.get(configName);
+ if (configKey == null) {
+ s_logger.warn("Couldn't find configuration " + configName + "
in ConfigDepot too.");
+ return null;
Review Comment:
default is String, update the same.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
