sureshanaparti commented on a change in pull request #4701: URL: https://github.com/apache/cloudstack/pull/4701#discussion_r591248773
########## File path: plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java ########## @@ -5732,16 +5733,24 @@ protected Answer execute(UnregisterNicCommand cmd) { } public void cleanupNetwork(HostMO hostMo, NetworkDetails netDetails) { - // we will no longer cleanup VLAN networks in order to support native VMware HA Review comment: As per the comment here, native VMware HA not supported when the setting "vmware.cleanup.port.groups" is true, right? If so, may be you can add this as a WARNING in the config description. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org