harikrishna-patnala commented on PR #11811:
URL: https://github.com/apache/cloudstack/pull/11811#issuecomment-3388729149

   > > clgtm. I did very basic testing by importing the VM from the shared 
disk. Earlier an IP was assigned to the imported VM (stopped state). After 
these changes, no IP is assigned to the imported VM. When the VM is started, an 
IP will be allocated. This should allow the user to import more VMs than the 
available IPs. But the start VM will probably fail due to unavailability of IPs.
   > 
   > thanks @vishesh92 for the testing
   > 
   > I think it would be better to allocate an IP when VM is imported (not 
started). what's your opinions ? @nvazquez @harikrishna-patnala @rajujith
   
   yes I do agree @weizhouapache but with this PR it is supposed to get an IP 
during import itself right, except for L2 network. Do we need more changes then 
?


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