JoaoJandre commented on PR #7446:
URL: https://github.com/apache/cloudstack/pull/7446#issuecomment-1840614363

   > @JoaoJandre @DaanHoogland any consensus here? Are all issues/comments 
raised by @rajujith already addressed? [#7446 
(review)](https://github.com/apache/cloudstack/pull/7446#pullrequestreview-1724439749)
   
   @shwstppr yes, all the comments raised by @rajujith were addressed.
   
   
   
   > I recently encoutered a similar issue and I am actually -1 on this without 
my sugestion because of that. I think you even agreed with me on that ticket, 
@JoaoJandre. we need to allow users to minimise logging by using a lesser 
level. cc @shwstppr
   > 
   > ad. #8278
   
   @DaanHoogland I thought a little more about it and in this instance I don't 
see a problem with this approach. But I don't think that this should always be 
the case (only have stacktraces on debug or lower). I'll make a post on the 
discussion thread later.


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