Sudha, I think those tests are in test/integration/component/test_brocade_vcs.py that Ritu included.
Cheers, Hugo On 23 jul. 2014, at 14:07, Sudha Ponnaganti <sudha.ponnaga...@citrix.com> wrote: > 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 >>> >>> >> >