Mice, no need to revert. I also had similar issues that the NIC was present even after reboot. May be we need to look into the difference in behavior. But for now, I guess at-least the add api must be allowed to pass. Your thoughts?
Thanks, Saksham On Thursday 09 May 2013 02:37 PM, Mice Xia wrote: > Saksham, > > Really sorry I didn’t notice it's in progress until I pushed the commit.. I > can revert it if necessary. > > Yes, without vmtool, a reboot of the VM can bring up the added NIC, but > removing NIC fails in my environment, the nic is still there even after a > reboot, can you confirm it in your env? > > Regards > Mice > > -----Original Message----- > From: Saksham Srivastava [mailto:saksham.srivast...@citrix.com] > Sent: Thursday, May 09, 2013 4:37 PM > To: Mice Xia; dev@cloudstack.apache.org > Subject: Fix for CLOUDSTACK-2277 > > Hi Mice , > > I just saw your commit for > https://issues.apache.org/jira/browse/CLOUDSTACK-2277 > So I recently picked this bug, marked it as "in progress" as I was working > on it and was about to submit the patch for the same, but then I saw that you > already fixed it :) > > A quick comment : > The fix makes sure that if vm-tools are not running on the vm, add/remove nic > fails. > What I have observed is that if vm-tools are not running, a reboot of the vm > will actually show up the added NIC. IMO we should allow the api to pass, but > provide a log message that reboot of vm will be required for added NIC to > show up, rather than completely failing the api. > > Your thoughts ? > > Thanks, > Saksham