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

Review request for cloudstack, Chip Childers, edison su, Kelven Yang, and 
Sateesh Chodapuneedi.


Bugs: CLOUDSTACK-3311


Repository: cloudstack-git


Description
-------

An incorrect type check for an existing promiscuous pvlan id was causing 
Virtual Router reboot failure. Fixing the same. Also detected incomplete 
checking of existing secondary pvlan id on a VMWare DVS, so fixing that as well.


Diffs
-----

  vmware-base/src/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java 
dc1486a 

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


Testing
-------

Created a pvlan setup in vmware. Created guest instances successfully. 
Restarted the virtual router of the pvlan shared network. The VR rebooted 
successfully without erroring out. Created multiple pvlan networks and 
associated guest VMs and restarted the VRs, all rebooted successfully. Created 
guest VMs with multiple pvlan networks in each and they also worked as expected.


Thanks,

Venkata Siva Vijayendra Bhamidipati

Reply via email to