DaanHoogland commented on pull request #4215:
URL: https://github.com/apache/cloudstack/pull/4215#issuecomment-665526708


   This quite a functional change @ravening .
   Did you test settings for domains linked to ldap?
   I read in your description that you are (kind of) mixing account level 
settings with domain level settings. some settings do not apply to domains and 
others do not apply to accounts.
   
   I like your idea that the setting system should be hierarchical and agree, 
but it just isn't designed this way. the same issue exists in 
zone-pod-cluster-host.


----------------------------------------------------------------
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:
[email protected]


Reply via email to