vishesh92 commented on code in PR #10212:
URL: https://github.com/apache/cloudstack/pull/10212#discussion_r2300292294


##########
server/src/main/java/com/cloud/vm/UserVmManager.java:
##########
@@ -59,6 +59,12 @@ public interface UserVmManager extends UserVmService {
             "Destroys the VM's root volume when the VM is destroyed.",
             true, ConfigKey.Scope.Domain);
 
+    ConfigKey<String> VmDistinctHostNameScope = new ConfigKey<>(String.class, 
"vm.distinct.hostname.scope", ConfigKey.CATEGORY_ADVANCED,
+            "network",
+            "Scope of resources to check while checking if the hostname is 
unique. Possible values are global, domain, subdomain, account, network.",
+            true, ConfigKey.Scope.Global, null, "VM distinct hostname scope", 
null, null, null, ConfigKey.Kind.Select,
+            "global,domain,subdomain,account,network");
+

Review Comment:
   For project, we specify account_id as the project id. So, this is covered by 
the account scope. Adding project will make the check complicated.
   For networks in a VPC, the network domain is same for all.



-- 
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