cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Saksham Srivastava" <saksham.srivast...@citrix.com>
Subject Re: Review Request 17736: CLOUDSTACK-5999: Virtual Router does not start if Guest VM is rebooted from CloudStack
Date Thu, 13 Mar 2014 07:31:45 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17736/
-----------------------------------------------------------

(Updated March 13, 2014, 7:31 a.m.)


Review request for cloudstack, John Burwell and Murali Reddy.


Bugs: CLOUDSTACK-5999
    https://issues.apache.org/jira/browse/CLOUDSTACK-5999


Repository: cloudstack-git


Description
-------

When a guest is rebooted from CloudStack, if the virtual router managing the guest network
of that guest is down, CloudStack will not start the virtual router.
However the router is started in case the guest vm is stopped and then started.
To mantain similarity between the 2 process it is necessary to start the VR in case it is
not running.
The fix will address the same.


Diffs
-----

  server/src/com/cloud/vm/UserVmManagerImpl.java be00aa8 

Diff: https://reviews.apache.org/r/17736/diff/


Testing
-------

Testing:
1) vm in a single guest network :
               vm Reboot : If the VR is stopped: VR is first started and then the VM reboots.
               vm Reboot : If the VR is running, VM reboots as it used to.
vm Stop/Start continue to work the same.

2)  vm having nics in multi networks :
               vm Reboot : If VR in any/all network is stopped: VRs are first started and
then the VM reboots.
               vm Reboot : If the VRs are running, VM reboots as it used to.
vm Stop/Start continue to work the same.

3) vpc :
               Tested the above scenarios for vpc also, works fine.

Patch applies cleanly.
Build passes succesfully.
FindBug is passed.
Rebased against latest master.


Thanks,

Saksham Srivastava


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message