You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2023/01/16 13:21:52 UTC

[GitHub] [cloudstack] GutoVeronezi commented on pull request #7094: Handle console session in multiple management servers

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

   > except for the name part 'allowed', which seems to be immplied in any case, looks good. Also I am still not comfortable approving logs that are
   > 
   > 1. debug
   > 2. expanding parameters
   > 3. not having a guard with isDebugEnabled()
   >    You have an PR in the making that will deal with that, can you share something about that?
   
   @DaanHoogland, regarding the PR, we are working to open it today. As soon as it is opened, we ping you.


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org