Ritu, Would be good to add automated tests for the procedure you have outlined. Marvin has already similar tests and you should be able to reuse them.
Thanks /sudha >> Testing >> ------- >> >> * 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 >> -----Original Message----- From: Trippie [mailto:trip...@gmail.com] On Behalf Of Hugo Trippaers Sent: Wednesday, July 23, 2014 2:15 AM To: <dev@cloudstack.apache.org> Cc: Ritu Sabharwal Subject: Re: Review Request 22863: CLOUDSTACK-6823 : First code drop for Brocade Network plugin to orchestrate Brocade VDX switches for L2 connectivity. Hey all, Just pushed the brocade VDX code into master. * fing bugs is not showing any issues * decent unit test coverage * includes functional test procedure * majority of the functional code is contained in a plugin, minimal changes to core Cheers, Hugo On 23 jul. 2014, at 11:12, Hugo Trippaers <htrippa...@schubergphilis.com> wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22863/#review48487 > ----------------------------------------------------------- > > Ship it! > > > commit 628d8e66f77053de9819436739325720710175ed > Author: Ritu Sabharwal <rsabh...@brocade.com> > Date: Wed Jul 23 08:51:20 2014 +0200 > > CLOUDSTACK-6823 : First code drop for Brocade Network plugin to > orchestrate Brocade VDX switches for L2 connectivity > > Signed-off-by: Hugo Trippaers <htrippa...@schubergphilis.com> > > > - Hugo Trippaers > > > On July 22, 2014, 9:44 p.m., Ritu Sabharwal wrote: >> >> ----------------------------------------------------------- >> This is an automatically generated e-mail. To reply, visit: >> https://reviews.apache.org/r/22863/ >> ----------------------------------------------------------- >> >> (Updated July 22, 2014, 9:44 p.m.) >> >> >> Review request for cloudstack and Hugo Trippaers. >> >> >> 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 >> ----- >> >> api/src/com/cloud/network/Network.java 0a08f28 >> api/src/com/cloud/network/Networks.java 1ad3350 >> api/src/com/cloud/network/PhysicalNetwork.java 024b3ce >> api/src/org/apache/cloudstack/network/ExternalNetworkDeviceManager.ja >> va e73f526 client/WEB-INF/classes/resources/messages.properties >> bb75b08 client/WEB-INF/classes/resources/messages_zh_CN.properties >> d7a0ca9 client/pom.xml 410cb19 >> client/tomcatconf/commands.properties.in aa03949 >> plugins/network-elements/brocade-vcs/pom.xml PRE-CREATION >> plugins/network-elements/brocade-vcs/resources/BrocadeInterfaceSchema >> .xsd PRE-CREATION >> plugins/network-elements/brocade-vcs/resources/BrocadePortProfileSche >> ma.xsd PRE-CREATION >> plugins/network-elements/brocade-vcs/resources/BrocadeShowVcsSchema.x >> sd PRE-CREATION >> plugins/network-elements/brocade-vcs/resources/META-INF/cloudstack/vc >> s/module.properties PRE-CREATION >> plugins/network-elements/brocade-vcs/resources/META-INF/cloudstack/vc >> s/spring-vcs-context.xml PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/Associat >> eMacToNetworkAnswer.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/Associat >> eMacToNetworkCommand.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/CreateNe >> tworkAnswer.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/CreateNe >> tworkCommand.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/DeleteNe >> tworkAnswer.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/DeleteNe >> tworkCommand.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/Disassoc >> iateMacFromNetworkAnswer.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/Disassoc >> iateMacFromNetworkCommand.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/agent/api/StartupB >> rocadeVcsCommand.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/api/commands/AddBr >> ocadeVcsDeviceCmd.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/api/commands/Delet >> eBrocadeVcsDeviceCmd.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/api/commands/ListB >> rocadeVcsDeviceNetworksCmd.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/api/commands/ListB >> rocadeVcsDevicesCmd.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/api/response/Broca >> deVcsDeviceResponse.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/network/BrocadeVcs >> DeviceVO.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/network/BrocadeVcs >> NetworkVlanMappingVO.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/network/brocade/Br >> ocadeVcsApi.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/network/brocade/Br >> ocadeVcsApiException.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/network/brocade/Co >> nstants.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/network/dao/Brocad >> eVcsDao.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/network/dao/Brocad >> eVcsDaoImpl.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/network/dao/Brocad >> eVcsNetworkVlanMappingDao.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/network/dao/Brocad >> eVcsNetworkVlanMappingDaoImpl.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/network/element/Br >> ocadeVcsElement.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/network/element/Br >> ocadeVcsElementService.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/network/guru/Broca >> deVcsGuestNetworkGuru.java PRE-CREATION >> plugins/network-elements/brocade-vcs/src/com/cloud/network/resource/B >> rocadeVcsResource.java PRE-CREATION >> plugins/network-elements/brocade-vcs/test/com/cloud/network/brocade/B >> rocadeVcsApiTest.java PRE-CREATION >> plugins/network-elements/brocade-vcs/test/com/cloud/network/guru/Broc >> adeVcsGuestNetworkGuruTest.java PRE-CREATION >> plugins/network-elements/brocade-vcs/test/com/cloud/network/resource/ >> BrocadeVcsResourceTest.java PRE-CREATION plugins/pom.xml eff5a01 >> setup/db/db/schema-440to450.sql 6cb81c6 >> test/integration/component/test_brocade_vcs.py PRE-CREATION >> tools/apidoc/gen_toc.py bcc8417 ui/dictionary.jsp c84d0dd >> ui/scripts/system.js 502007f ui/scripts/ui-custom/zoneWizard.js >> 645c2b1 >> >> Diff: https://reviews.apache.org/r/22863/diff/ >> >> >> Testing >> ------- >> >> * 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-7 >> a21-4714-98f3-6514cf54ba84__diff Patch file for Brocade functionality >> code >> https://reviews.apache.org/media/uploaded/files/2014/06/26/92bb0014-a >> 7b7-4f0b-97c9-018d615b658a__brocade-vcs.patch >> >> >> Thanks, >> >> Ritu Sabharwal >> >> >