summaryrefslogtreecommitdiffstats
path: root/sfc
AgeCommit message (Collapse)AuthorFilesLines
2018-02-21Switch to os.environ for getting OpenStack env varsCédric Ollivier1-4/+4
CONST shouldn't be selected to get env vars. Now it may raise KeyErrors if env is not well configured. Change-Id: I5e3f525640809778c5349fdc1c3796227894d0cd Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2018-02-21Take into account last Functest config updatesCédric Ollivier1-2/+2
It follows the next changes (Functest) which renamed the related config entry [1]. [1] https://gerrit.opnfv.org/gerrit/#/c/51283/5/functest/ci/config_functest.yaml Change-Id: Ieae5baff13bb70aa35b01c2358ab4865cdb9dfb0 Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2018-02-16Switch from CONST to env.get()Cédric Ollivier5-17/+20
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-06Define the timethis function in the sfc repoManuel Buil1-2/+17
Functest removed the function from their repo, so we need to define it in ours: Change-Id: Ica122e57612c38751d8ff810f3ba61b749e867eb Signed-off-by: Manuel Buil <mbuil@suse.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 Buil5-8/+430
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 Buil6-17/+17
Change-Id: I11c37ed6898c03b2a54a0e3b11422c654023333f Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-01-31Adapt change in functestManuel Buil1-1/+1
The variable TEST_DB_URL must be read as a env. variable since this patch": https://gerrit.opnfv.org/gerrit/#/c/49757/ Change-Id: I8f77acbbb2fe949f5fbe8a3f983c49224d893bcc Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-01-29Fix problem to fetch the compute name of client VMManuel Buil4-10/+17
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-26Fix cleanup functionManuel Buil1-0/+1
A floating ip remains when executing the clean up function. That floating ip belongs to the Tacker VNF and thus we should add that creator to the list Change-Id: Ic15574bf93f30a26f81a4944d742307dda4bfe6d 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 Buil10-433/+375
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 Buil7-842/+850
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 Buil4-0/+42
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>
2017-10-12Move to sfc_nsh_euphrates imageManuel Buil1-2/+2
To create the four SFs we will use the sfc_nsh_euphrates image. This image is like the sfc_nsh_danube but modifying the vxlan_tool which now has the important fix: https://git.opendaylight.org/gerrit/#/c/64116/ Change-Id: I396b5523b7243472b6d05b62d831837938067422 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-10-06If the VM starts quickly, this check might failManuel Buil1-1/+1
Sometimes VM starts very quickly and then it takes longer for this service to come up Change-Id: Ibc4ea47eb0b4b6655e338d1b85e6ed14e61d0064 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-10-04Do not use capital lettersManuel Buil4-4/+4
As agreed with the releng-xci guys, we should not use capital letters to refer to openstack-ansible (osa) Change-Id: I634747b07e72513f751cde3eb33f3b4bad5c176d Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-29Sort config.yml correctly and correct vnfd namesManuel Buil1-11/+12
JIRA: SFC-116 It is confusing that our test-1 is in the second position of config.yaml and that it uses test2-vnfd instead of test-vnfd Change-Id: Iffd22f295630bce0ce854d4876c1478c56690cfa Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-28Modify the list of working topologiesManuel Buil1-14/+13
After testing, there are three topologies which work and three topologies which do not work because of bugs in OVS and ODL. We remove those two from the testing in 5.0.0 Change-Id: I576768978bfa763725027239081686c95408eef4 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-28Disable test 2 for 5.0.0 (Euphrates 1.0)Manuel Buil1-1/+1
Test 2 is continuously failing because of tacker missing functionality Change-Id: I785e79d78df4bfb9f48e20288e261efff741bcd4 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-28Fixing typo in test-vnfdManuel Buil2-4/+3
Change-Id: I64caba2eea3ff93d082c08915f82be7ef701a3f1 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-28Bug fix: Actively disable security on SF portManuel Buil5-1/+5
JIRA: SFC-114 Some cases show that it was not enough with the patch: https://gerrit.opnfv.org/gerrit/#/c/42801/ We should explicitely disable security in the SF ports. That can be done with a the antispoofing parameter in the vnfd Change-Id: Ie20ab961633866f48a4abcc299a62e8c55280088 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-28Bug fix: Delete unnecessary functions which are causing errorsDimitrios Markou1-10/+0
JIRA: SFC-117 Change-Id: I20e12e38a3cf2a9954893cf52187ac153b64bdfa Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
2017-09-25Bug fix: Do not start testing before checkingManuel Buil1-5/+30
JIRA:SFC-113 We used to check that the ACL and RSP flows were implemented because it took a lot of time. However, the new ODL Nitrogen reduced this time a lot and thus we must check that the vxlan_tool is up and the HTTP port is up in the server before doing any test Change-Id: I70219071c7965781b17e80631bf3e2efbfcfe4cd Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-22Remove SFs from security groupsManuel Buil6-8/+0
Having the SFs port in security groups delays packets artifically in the tests Besides, the netvirt guys suggested not to do it because it might be confusing for ODL in some situations and it is not a 'suitable construct' Tests without security groups show indeed a better behaviour Change-Id: I33281215cd2e80804c8f09ca201c046f590249ab Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-20Bug fix: Add iteration to check RSP config in ODLManuel Buil1-1/+8
JIRA: SFC-110 Sometimes the test is quicker than ODL and when it checks the configured RSP, that configuration has not yet been submitted in ODL, thus ODL replies with an empty list. We add an iteration to try again for a couple of times Change-Id: I21e4ac3fc3a919791daa7e41cb2953f7565c4ebb Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-19Create vnffg providing the neutron_portManuel Buil6-15/+70
JIRA: SFC-107 We are submitting the vnffgd with a variable for the neutron_port. Before creating the vnffg, we must provide a file with the neutron_port value Change-Id: I9994e2baa53f323df17434448d372598c0d94c72 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-18Bug fix: allows searching vnfd_id based on nameManuel Buil1-2/+6
It is currently impossible to obtain the vnfd_id based on the vnfd_name when we want to create the vnffg Change-Id: Ib7752156db43d76c24f8c6c2803eb7dbc5bbe1d8 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-18Merge "[Bug Fix] Delete vnffgs in a reverse order"Manuel Buil1-1/+1
2017-09-18Fix vnffgd templatesManuel Buil1-2/+2
JIRA: SFC-105 They have a typo which resulted in errors in Tacker when loading the template Change-Id: I5723c756add7610377815180352140d58fd783a5 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-18[Bug Fix] Delete vnffgs in a reverse orderDimitrios Markou1-1/+1
JIRA: SFC-106 In sfc_two_chain testcase we need to delete first the blue and then the red vnffg because if we do it the other way around the tacker produces a warning that the flow classifier of the vnffg red is in use and cannot delete the chain. The aforementioned warning is produced because during the execution of the two_chain testcase we reassing the classifier from the red vnffg to the blue vnffg. So if we try to delete first the red vnffg we get the warning because the classifier is assigned and in use from the blue vnffg. Change-Id: I3cb7bef4b9b9e75eef4961fe7f63cbc1755130be Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
2017-09-15Sync both test cases and move checks to the startManuel Buil2-14/+34
There are some checks in sfc_two... which do not exist in sfc_one. Those checks should be done at the beginning of the test Change-Id: I44c612c7dfe6bb4329bc49478f3342cda555229c Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-15Update the wait_for_classification_rules functionManuel Buil4-68/+58
Two things must be done: 1 - Be compatible with the new way of doing SFC in ODL Nitrogen 2 - Be installer independent Change-Id: Ibbb3057db30a7c00e92f788bb03e770749e2b255 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-14Added support for OSA installerManuel Buil1-0/+2
Change-Id: I3a34daaca258cd2ce511aee51819cf438110e403 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-13Merge "Increase timeout time"Brady Johnson3-3/+3
2017-09-13Increase timeout timeManuel Buil3-3/+3
My VMs take around 80 seconds to be able to respond to pings. The current timeout is in 50 seconds. I will increase it to 150 to have a buffer Change-Id: Ie684e76799210e3d7239e35df2d4bdd7af507669 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-13Add network_src_port_id to vnffgdManuel Buil1-1/+4
ODL requires that parameter to be present in order to classify Change-Id: Idb8e0578a9c4c22cd35e4615755333b67d4fe658 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-11Merge "Allign the SFC testcases to the new Tacker API"Manuel Buil2-65/+55
2017-09-11Merge "Adapt functionality to the new Tacker API"Manuel Buil16-85/+271
2017-09-07Allign the SFC testcases to the new Tacker APIDimitrios Markou2-65/+55
In this patch [0] we adapt the Tacker library which is located to the SFC repo to the new Tacker API With the current patch we allign the testcases to the aforementioned API [0] https://gerrit.opnfv.org/gerrit/#/c/40839/ Change-Id: I07f8bcefbda7d058e9322465df38077ac0346d83 Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
2017-09-07Adapt functionality to the new Tacker APIDimitrios Markou16-85/+271
Change-Id: I078e9f26d3040cf04120abd425f03cb4f2e8daef Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
2017-09-01Remove setup_compute_node and configure_iptablesManuel Buil4-35/+0
We are going to use ODL Nitrogen which does not require the NSH hack as packets reaching the VNF will use ETH+NSH encapsulation Change-Id: I5a55fde84e8f17860783ec9de8b99242fc38b9a9 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-09-01Add new Tacker libraryDimitrios Markou4-3/+193
Change-Id: I710e6f5da92ddfddd9111c6c53274b92d76d7449 Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>