-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22863/
-----------------------------------------------------------
(Updated July 17, 2014, 11:52 p.m.)
Review request for cloudstack and Hugo Trippaers.
Changes
-------
Removed the cfg file from patch. Only for testing.
Bugs: CLOUDSTACK-6823
https://issues.apache.org/jira/browse/CLOUDSTACK-6823
Repository: cloudstack-git
Description
-------
First code drop for Brocade Network plugin to orchestrate Brocade VDX switches
for L2 connectivity. Please create a new branch for Brocade plugin.
Diffs (updated)
-----
api/src/com/cloud/network/Network.java 885bffe
api/src/com/cloud/network/Networks.java 1e4d229
api/src/com/cloud/network/PhysicalNetwork.java 8cc214e
api/src/org/apache/cloudstack/network/ExternalNetworkDeviceManager.java
e73f526
client/WEB-INF/classes/resources/messages.properties b504a18
client/WEB-INF/classes/resources/messages_zh_CN.properties 1ec4e95
client/pom.xml 29fef4f
client/tomcatconf/commands.properties.in d247aa0
plugins/pom.xml b5e6a61
setup/db/db/schema-440to450.sql 77445a9
test/integration/component/test_brocade_vcs.py PRE-CREATION
tools/apidoc/gen_toc.py 827d6bf
ui/dictionary.jsp 9026a36
ui/scripts/system.js 9a98a5c
ui/scripts/ui-custom/zoneWizard.js 4091c03
Diff: https://reviews.apache.org/r/22863/diff/
Testing (updated)
-------
• Create an isolated network; verify that the port-profile is created on
the Brocade switch.
• Attach a VM to the network; verify that the VMs MAC address is
associated with the port profile of the network on the Brocade switch.
• Delete VMs for an isolated network; verify that the VMs MAC address is
disassociated with the port profile of the network on the Brocade switch.
• Delete the isolated network; verify that the port-profile is deleted
from the Brocade switch.
Integration test result:
Test Brocade Network and VM Creation ... === TestName: test_network_vcs |
Status : SUCCESS ===
ok
----------------------------------------------------------------------
Ran 1 test in 297.497s
OK
File Attachments
----------------
Diff for the existing cloudstack code
https://reviews.apache.org/media/uploaded/files/2014/06/23/8fc3cfb1-7a21-4714-98f3-6514cf54ba84__diff
Patch file for Brocade functionality code
https://reviews.apache.org/media/uploaded/files/2014/06/26/92bb0014-a7b7-4f0b-97c9-018d615b658a__brocade-vcs.patch
Thanks,
Ritu Sabharwal
