DaanHoogland commented on PR #14027:
URL: https://github.com/apache/cloudstack/pull/14027#issuecomment-5568852974

   > please ignore my previous approval
   > 
   > * We have never encountered an NPE caused by this line of code.
   > * I suggest throwing an exception if the ipVO object is null.
   > 
   > cc @DaanHoogland
   
   @nagaboinaramgopal , can you agree with @weizhouapache , or do you have a 
scenario where you want to skip checking the network services and continue?


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