cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Venkata Siva Vijayendra Bhamidipati" <vijayendra.bhamidip...@citrix.com>
Subject Re: Review Request 12517: Fix checks for already existing primary and secondary pvlan id on vmware DVS in pvlan setup
Date Mon, 15 Jul 2013 22:41:14 GMT

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

(Updated July 15, 2013, 10:41 p.m.)


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


Changes
-------

+Sheng


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


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