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 2020/12/02 12:55:02 UTC

[GitHub] [cloudstack] DaanHoogland commented on pull request #4507: Fix failure in validating IP address in case of multiple Management Servers

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


   > > Though I see the need of this fix, i am a bit wary of it. It must be implemented somewhere for the kvm agent as well, so why is that code not called here. It seems like we are doubling/overlooking something.
   > 
   > This situation does not arise in case of KVM agent @DaanHoogland
   
   I think I wasn't clear , @Pearl1594 . The kvm agent must have code to handle the case of multiple ip addresses for the MS, why isn't the same code used for both? The ip selection process during connecting is the same for all agent so the code should be the same bit as well.


----------------------------------------------------------------
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:
users@infra.apache.org