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