GutoVeronezi commented on PR #6809:
URL: https://github.com/apache/cloudstack/pull/6809#issuecomment-1366678641

   > Exactly, which should be good enough. Editing the global level should set 
the default on a scoped level. It might be that a scoped level that has not set 
this setting, keeps using the default value from the global level though, 
instead of the edited global value. I have not tested that and if it does that 
would be a bug.
   > 
   > a work around for this would be to edit the global default.
   
   It tries to retrieve the value from the scope:
   
   
https://github.com/apache/cloudstack/blob/ca7e1ac1abbe78513bfdd57b990d58e018f8d65d/framework/config/src/main/java/org/apache/cloudstack/framework/config/ConfigKey.java#L153-L164
   
   If it returns null (meaning the value is not defined for the scope), it will 
retrieve the value of the global configuration:
   
   
https://github.com/apache/cloudstack/blob/ca7e1ac1abbe78513bfdd57b990d58e018f8d65d/framework/config/src/main/java/org/apache/cloudstack/framework/config/ConfigKey.java#L143-L151
   
   It will only use the default value if the global configuration is null 
(which should not happen) or its value is null.
   
   


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

Reply via email to