aboutsummaryrefslogtreecommitdiffstats
path: root/sfc/tests
AgeCommit message (Collapse)AuthorFilesLines
2019-03-18Remove set_scripts directoryManuel Buil6-149/+0
The contents of this directory are not used anymore Change-Id: I1dff86aba72288c618b4909ffa6197de6b241e49 Signed-off-by: Manuel Buil <mbuil@suse.com>
2019-01-21Simplify deletion test and fix unittestManuel Buil2-22/+20
Removing the port groups when deleting the first chain simplifies things because we don't need port_groups to be a variable of the SfcCommonTestCase class There were also some missing unittest Change-Id: I6ae780bdc72116907a9e6037571c29f28a6ba334 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-11-26Migrate SNAPS to Openstack-SDKNikos Mimigiannis6-78/+54
JIRA: SFC-137 This patch replaces SNAPS library with openstack_SDK library for TCs with mano component: no-mano Change-Id: I99b9a5f54a273f61ecff05a11616440d5e85e55b Signed-off-by: Nikos Mimigiannis <nmimi@intracom-telecom.com>
2018-11-14Fix bug in the symmetric test caseManuel Buil5-24/+28
The symmetric testcase requires the neutron port of the server to check that the classification rules were written in the flow tables. This patch adds that. Change-Id: I7ce930b102409f2772771a9c5b0e6de99bb293a0 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-09-03Remove port security from vnfs when no-manoManuel Buil1-1/+2
If the ports have security enabled, the traffic is dropped due to the anti-spoofing Change-Id: I6fbc63a3d959d9d55dfd1d64e12e1d0e7ec48bcb Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-08-31Decouple tacker from testsManuel Buil7-121/+220
Support n-sfc too and abstract a bit the MANO layer so that other MANO components can be inserted into the test Change-Id: I3fb59fbf40b4207bf1721092cd8ff0559e1d9d90 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-08-30Merge "Bug fix: Bad usage of positional argument"Manuel Buil1-2/+2
2018-08-30Merge "Improve readability of the tests"Manuel Buil5-11/+11
2018-08-29Bug fix: Deletion testcase only uses one VNFManuel Buil1-1/+0
We are only using one vnf but there are two defined in the config for that testcase Change-Id: Ifa86c2e48cb60edcd1b173eee14715c3787a9889 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-08-29Improve readability of the testsManuel Buil5-11/+11
The method name "create_custom_av" does not clearly say what it is doing. Changing it to "create_vnf" clarifies the code better Change-Id: Ic3f874bdb22891d348e5609130b8e4f0f3f8c3e8 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-08-29Bug fix: Bad usage of positional argumentManuel Buil1-2/+2
av_zone is passed as a positional argument, however, the create_instance method does not define av_zone as positional: https://github.com/opnfv/sfc/blob/master/sfc/lib/openstack_utils.py#L136-L137 so we should not pass it as a positional argument Change-Id: I035012ef3ffce3318e0bece712cb6bdad8e8cdd0 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-08-29Flake8 fixesManuel Buil2-13/+2
When running flake8 I am getting several errors. This patch fixes them Change-Id: I4d32b4b214ca61e98da6fc28a4de44201c4c1cdc Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-06-28Fix Update README.testsspiritus721-8/+29
When looking at the different testcases in the functest directory TEST SYMMETRIC is outdated and TEST DELETION is missing in the README. This patch is an update to README.tests for the latest changes that have been done the to those two testcases. Change-Id: I54fceac85f45212ed45efca1a6f30d4ba092e0c1 Signed-off-by: spiritus72 <billmorrissonjr@gmail.com>
2018-06-15Fix "Remove the annoying WARNINGS when using XCI"Bill Morrisson5-1/+25
When executing the SFC tests in XCI, we constantly get a whole bunch of annoying "InsecureRequestWarning" messages which does not allow to follow what is going on in the test. This patch filters all those errors leaving the output of the functest results clean. Change-Id: I3bd9519676222d00ee4a2c8d47084737cd9768aa Signed-off-by: Bill Morrisson <billmorrissonjr@gmail.com>
2018-06-12Bug fix: pass variables in the class constructorManuel Buil1-2/+2
The Testcase class requires parameters in the constructor. We should pass them or otherwise we get an error: https://github.com/opnfv/functest-xtesting/blob/master/xtesting/core/testcase.py#L48 ERROR: TypeError: "__init__() got an unexpected keyword argument 'project_name'" Change-Id: I6f4b62b7ff201bf6878004e3fb452778c6b55f0e Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-06-12Remove the pike specific filesManuel Buil9-299/+20
We are already preparing for Gambia release and that means Openstack Queens for the traditional release track. Therefore, we can remove the specific configs for Openstack Pike Change-Id: Ib49a53e73ae1020aaead8a897dff8733147d87ac Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-06-12Rethink the clean up processPanagiotis Karalis6-18/+58
Using the '--nocleanup' parameter as part of command, the cleanup mechanism could be disable. Otherwise, the cleanup method is executed even if a exception has been raised. Change-Id: I9638369900f599793393feb5184c30b74f1b7c7e Signed-off-by: Panagiotis Karalis <pkaralis@intracom-telecom.com>
2018-06-07Provide support for VNFsManuel Buil2-0/+13
Currently it is not possible to define different images for client/server and VNFs. This functionality is important to have in order to test different VNFs Change-Id: Ibc3b3f1102d52529701748d37f795cdf05269410 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-06-01Fix the logging to be compatible with functestPanagiotis Karalis6-15/+15
In order to collect logs from SFC test scenarios running through functest framework (mainly for troubleshooting purposes), the logging mechanism has been updated accordingly. JIRA: SFC-125 Change-Id: Ide7c32519afa6ece299d0ac1dcb168609a2e3945 Signed-off-by: Panagiotis Karalis <pkaralis@intracom-telecom.com>
2018-05-22Fix symmetric test case result summaryJaime Caamaño Ruiz2-6/+10
Correctly print in test summary whether we are testing HTTP uplink or HTTP uplink. Change-Id: Ib8d4cc4a5e794384720d7023032c5340039e2ff1 Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
2018-05-15Clean up Test Case codePanagiotis Karalis7-1019/+870
Too much of duplicated code has been obeserved in test cases. With this patch a small refactoring is necessery to our test cases code. So they can be more generic and more flexible. To achieve that the addition of new parent class is essential, hence all the rest test case classes will inherit from the aforementioned parent class. Change-Id: I6a50d8330a06321742b3bb03cc999682d4a4e6a1 Signed-off-by: Panagiotis Karalis <pkaralis@intracom-telecom.com>
2018-05-09Add more logs to the codeManuel Buil4-5/+5
It is hard to understand what is going on in the SFC executions Change-Id: I5c9688e68b03c59d6d39290faee8683513a60440 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-05-03Bug fix: Use opnfv.logger objectManuel Buil5-30/+20
JIRA: SFC-125 If we don't use the opnfv.logger from functest, logs are not printed when exeucting tests through the functest framework Change-Id: I6d58ff3977fd94221720eea978e0efde07a576a2 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-04-13Fix symmetric testcase configManuel Buil2-4/+4
One patch was merged without rebasing and it reverted some accepted changes Change-Id: Ia64cef60df1ceb72d8789d122cf1bbb820819aa2 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-04-09Use different config files depending on versionManuel Buil10-51/+135
We can read the openstack version from the env variables and then use a different config. The reason we use this is because openstack API changed from Pike to Queens and Pike is the official version for OPNFV Fraser release Change-Id: I8a94e71224d2ba0aa75a6438f3d7b03929acbbef Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-04-04Update symmetric chain test caseJaime Caamaño Ruiz5-56/+262
Update the symmetric chain test case aligning it with updates available from ODL Oxygen: - ODL networking-sfc implementation now supports symmetric paths. It is no longer needed to configure the ACL or chain directly through the ODL rest api. It can now be fully relied on tacker for this. - ODL networking-sfc implementation now supports different ingress and egress ports for port pairs. The test case exercises this by setting up a VNF with two distinct ports. Note that this requires an updated VNF image that supports two nics and bundles a vxlan_tool capable of using them. Change-Id: Ie711abb93109943dcbf699ece7b2b570500a3711 Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
2018-04-04Support sfp redirect action on acl flow checkJaime Caamaño Ruiz3-5/+11
Starting Oxygen, ODL networking-sfc translator supports symmetric paths and maps flow classifiers to ACLs with SFP redirect action instead of RSP redirect action. A SFP may have two RSPs associated if the path is symmetric, one otherwise. One of the two RSPs will be flagged as the reverse path and the other is the forward path. Traffic from the logical-source-port will be classified to the forward path, while traffic from the logical-destination-port will be classified to the reverse path. This patch updates the ACL flow check logic to account for the above. Change-Id: I2d8b9e6c2a1033d469668db29b9c18f525e89370 Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
2018-04-03Fix flake8 problemsManuel Buil2-2/+3
There was a bug in releng and pylint was not ran, some issues got merged in master because of that Change-Id: Icf3fa882f4f2b0593fa0b85edce46e49dcc303c9 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-03-19Merge "Add support for Compass installer"Manuel Buil5-4/+7
2018-03-16Add support for Compass installerYifei Xue5-4/+7
JIRA: SFC-122 Add support for Compass installer in SFC repo which will make Compass SFC scenarios be verified by SFC testcase daily. Change-Id: I8129ebe837d54cdcd163158990fccf40451b98ad Signed-off-by: Yifei Xue <xueyifei@huawei.com>
2018-03-15Allow executing tests from run_sfc_testsManuel Buil1-0/+6
Removing it was a mistake because without it is harder to debug (e.g. using break points does not work when triggering tests from functest) Change-Id: I3a958da0add0fb57946da333f75fb526c2fe6c33 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-03-05Remove run_sfc_tests.py from scriptsCédric Ollivier1-8/+0
It is no longer called via BashFeature. Change-Id: I8ae6b24d5e02f269056c1d4615b436c1a2a3a903 Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2018-03-05Import xtesting moduleLinda Wang1-1/+1
Change-Id: Id524fbf1677844b5172f52c857cf8bed8d075ff3 Signed-off-by: Linda Wang <wangwulin@huawei.com>
2018-02-28Bug fix: Problem with SNAPs integrationManuel Buil3-3/+6
SNAPs changed the API: https://github.com/opnfv/snaps/commit/c711acf8ae3e4ad6f746500747857bcc9fd6f7be#diff-0b922a3e087121238d5a8b08ef5b894aR480 Now we require to pass keystone client to get the neutron port Change-Id: I93fa4b127255c1a078aa3644773b7c8b886cea6d Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-02-16Switch from CONST to env.get()Cédric Ollivier4-4/+4
env.get() ensures that default values are reused if unset. It also avoids desynchronization between os.environ and CONST. It follows the change "Get properly env vars or their default values" [1] applied in Functest. It also fixes minor issues on shebangs and takes several pylint rules into account in this module: - http://pylint-messages.wikidot.com/messages:w0702 - https://www.python.org/dev/peps/pep-0282/ [1] https://gerrit.opnfv.org/gerrit/#/c/52221/ Change-Id: Ia7ad168b2cbbade8ea3890fb95dc01bcdfc50468 Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2018-02-05Update the vnffg templatesManuel Buil2-15/+28
The following tacker patch: https://review.openstack.org/#/c/532203/ changed the syntax of the vnffgd templates Change-Id: Ifbf788b02a95c8f45eb97230d3d2da117de158d6 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-01-31Test the deletion of chainManuel Buil4-0/+353
This is the flow of the test: 1 - Create a vnffg in tacker (i.e. chain + classifier) 2 - Remove the vnffg 3 - Check that the RSPs are removed from operation DB in ODL and the classifier rules are not present in the flows 4 - Create a new vnffg 5 - Test the new chain: 5.1 - Check HTTP traffic works 5.2 - Block HTTP traffic in VNF and check that it does not work anymore Change-Id: Ia077e58c5cbce4258a2f8a36f6b961eb923ff9c5 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-01-31Use the new vnf image for FraserManuel Buil5-16/+16
Change-Id: I11c37ed6898c03b2a54a0e3b11422c654023333f Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-01-29Fix problem to fetch the compute name of client VMManuel Buil2-3/+6
When migrating to SNAPs a bug was introduced and could be seen in logs as: Error when waiting for classification rules: 'module' object has no attribute 'get_compute_client' The cause is that get_compute_client is not a global function anymore but a method inside a class. Besides, due to SNAPs, we need to fetch the compute name in another way Change-Id: I32007c83894281ba9514a121b7939b5bfe52f4bc Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-01-25Merge "Adapt the vim-config to new tacker"Brady Johnson2-1/+21
2018-01-17Adapt the vim-config to new tackerManuel Buil2-1/+21
Tacker is changing the config and adds cert_verify Change-Id: Ia99bf2ad4ea0d6986ecd523bbe76e0e2b4a8a0ce Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-01-15Create vnffgd templates with the new syntaxManuel Buil3-0/+117
For Queens, tacker uses a different template syntax for the vnffgds Change-Id: I3c3bd497f09fcf6af2adc0eaf0f046dad88d88f2 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-12-21Migrate to SNAPs openstack libraryManuel Buil6-199/+139
Functest is deprecating its openstack library which we were using extensively We need then to move to an alternative and the best appears to be SNAPs Change-Id: Icaa0b9f1ec580545b9d0faa88be2080a310deaf7 Signed-off-by: Manuel Buil<mbuil@suse.com>
2017-12-19Move to functest TestCase superclassManuel Buil1-1/+1
Due to this: https://gerrit.opnfv.org/gerrit/#/c/49243/ We need to change the class Change-Id: I17dc65fcfd3e7ce3f6ebe5ec5176e5d89ef2e0e2 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-12-19Documentation update and some clean upManuel Buil13-72/+56
Change-Id: I1fca51b9b6afff5a596ceee2e1401127fe23231d Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-12-12Clean up our utils.pyManuel Buil3-91/+93
Utils.py was getting messy. This patch divides it into three different files: * test_utils.py * odl_utils.py * openstack_utils.py The tacker library is integrated into openstack_utils.py Change-Id: I310949d1cee49b6aa1c9b3396bf6d6ca458cbaac Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-11-24Update CMD in gating DockerfileCédric Ollivier1-1/+1
It conforms with the change "Remove prepare_env" [1]. [1] https://gerrit.opnfv.org/gerrit/#/c/47267/ Change-Id: I9a80049e002b812d097067dc53798fa3bc532505 Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2017-11-20Add the needed functest filesManuel Buil2-0/+44
As agreed with the functest project, we keep the Dockerfile and testscases.yaml in our repo Change-Id: Idb25d4bfff7e36b42e4981a138775f90b0fbe72e Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-10-17Produce more logs if things failManuel Buil2-0/+4
Apex deployment shows a problem when creating the SFC config in ODL and it is impossible to know what might be going on Change-Id: If8d33d1c60daaf9cd9773a5cb9c7e47f64e1caa9 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-10-13Modify the vnfd to use the new imageManuel Buil5-5/+5
We moved to a new image for SFs but we have not moved the name of the image in the vnfd Change-Id: I8f658e1dfcf0de3117845f615eddb8fd5b1e11af Signed-off-by: Manuel Buil <mbuil@suse.com>