nvazquez commented on code in PR #7692:
URL: https://github.com/apache/cloudstack/pull/7692#discussion_r1282556509


##########
server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java:
##########
@@ -853,6 +855,15 @@ public String updateConfiguration(final long userId, final 
String name, final St
             } catch (final Throwable e) {
                 throw new CloudRuntimeException("Failed to clean up download 
URLs in template_store_ref or volume_store_ref due to exception ", e);
             }
+        } else if 
(HypervisorGuru.HypervisorCustomDisplayName.key().equals(name)) {
+            String hypervisorListConfigName = Config.HypervisorList.key();
+            String hypervisors = _configDao.getValue(hypervisorListConfigName);
+            if (Arrays.asList(hypervisors.split(",")).contains(previousValue)) 
{
+                hypervisors = hypervisors.replace(previousValue, value);
+                s_logger.info(String.format("Updating the hypervisor list 
configuration '%s' " +
+                        "to match the new custom hypervisor display name", 
hypervisorListConfigName));
+                _configDao.update(hypervisorListConfigName, hypervisors);
+            }

Review Comment:
   Done



-- 
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: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to