summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2018-04-09Support sfp redirect action on acl flow checkJaime Caamaño Ruiz4-64/+150
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> (cherry picked from commit f66a8164cfb2ef172ee3ea274992b9c07306f08c)
2018-04-03Fix flake8 problemsManuel Buil3-3/+4
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-23xci: Lowercase ansible variablesFatih Degirmenci4-25/+25
Change-Id: I35960abdab146ab2f80a1ad160489ee4877d42e4 Signed-off-by: Fatih Degirmenci <fdegir@gmail.com>
2018-03-22Fix tacker resource collection requestJaime Caamaño Ruiz1-6/+6
Looks like using tacker_client.get to request a resource collection causes connection drops and subsequent requests to fail, probably because of incomplete pagination handling. Use tacker_client.list instead. Change-Id: I79ba39759b61f57774e65b5e49686edb6bf05c13 Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
2018-03-19Merge "Add support for Compass installer"Manuel Buil5-4/+7
2018-03-16Merge "Allow executing tests from run_sfc_tests"Dimitrios Markou2-0/+10
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 Buil2-0/+10
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-11Remove SNAPs deprecated methodManuel Buil1-4/+3
SNAPs is not supporting get_vm_info() anymore, we should remove it from our lib https://gerrit.opnfv.org/gerrit/#/c/53263/4 Change-Id: Idad1bb60b86c45256bb0a95b6c554e83e6bfae35 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-03-08Merge "Update SNAPs usage of library"Dimitrios Markou1-2/+7
2018-03-05Merge "Remove run_sfc_tests.py from scripts"Manuel Buil2-10/+0
2018-03-05Merge "Import xtesting module"Manuel Buil3-2/+2
2018-03-05Remove run_sfc_tests.py from scriptsCédric Ollivier2-10/+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 Wang3-2/+2
Change-Id: Id524fbf1677844b5172f52c857cf8bed8d075ff3 Signed-off-by: Linda Wang <wangwulin@huawei.com>
2018-03-05Update SNAPs usage of libraryManuel Buil1-2/+7
JIRA:SFC-120 Snaps updated two methods which now require more parameters: https://github.com/opnfv/snaps/blob/master/snaps/openstack/utils/heat_utils.py#L260 https://github.com/opnfv/snaps/blob/master/snaps/openstack/create_instance.py#L781 Change-Id: I164c51298afd7b80ee88312f3a09abe7f09282a0 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-03-02Remove glance_utils as we don't use it anymoreManuel Buil1-2/+1
Change-Id: I44d0dd4fdabd38f09e6113c28668f11a40c6ba86 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-02-28Bug fix: Problem with SNAPs integrationManuel Buil4-7/+16
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-26Stop relying on internal Functest utilsCédric Ollivier2-4/+20
It switches from functest_utils.get_functest_config() to config.CONF(). download_url is now hosted in sfc as it's unused by Functest. ft_utils.get_parameter_from_yaml() will be considered as part of Functest framework (then it must be well covered and documented). Change-Id: I3014d90dba53422020ae85a2815c5bf8c0dcf34c Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2018-02-23Merge "Switch from CONST to CONF"Manuel Buil2-5/+5
2018-02-22Bump SHAs for PikeManuel Buil2-68/+68
Change-Id: I0b0aabffb2a5995364051fa34d218b2ebc649d4e Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-02-22Switch from CONST to CONFCédric Ollivier2-5/+5
CONST is not the right Functest object to get config data. It also updates the way to get constants as proposed by Functest [1]. [1] https://gerrit.opnfv.org/gerrit/#/c/52389/ Depends-On: Ie23c0ef90efc839d60f0f3a9754c58746b1f3a00 Change-Id: I6cfa832466dcefd737314633d807512e46267a69 Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2018-02-22Merge "Switch to os.environ for getting OpenStack env vars"Manuel Buil1-4/+4
2018-02-21Update XCI deployment filesManuel Buil6-491/+0
We are now aligned with master so we don't need a-r-r file anymore or the OpenStack versions file Change-Id: Ida0b8f0414870d5fa3fecdc79564def345187b2e Signed-off-by: Manuel Buil <mbuil@suse.com>
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-21Merge "Switch from CONST to env.get()"Manuel Buil5-17/+20
2018-02-16Bump os_keystone SHA for Pike deploymentDimitrios Markou1-1/+1
Change-Id: Ic1dfe50cbf8f1567a055aa8780e671333f67e880 Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.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-25Merge "Create vnffgd templates with the new syntax"Brady Johnson3-0/+117
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>
2018-01-15Adapt to the new ODL neutron portManuel Buil6-6/+6
Change-Id: I65a2fa8074d56b53ee400c604d08103ea9abf390 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-12-21Merge "Add the xci_overrides file"Dimitrios Markou1-0/+5
2017-12-21Migrate to SNAPs openstack libraryManuel Buil11-433/+376
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-21Add the xci_overrides fileManuel Buil1-0/+5
We can override variables by default using this file. One of those variables is the VM_MEMORY_SIZE Change-Id: I19f12151a65e90399e1ffcfa2465256bc5376abe Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-12-20Pin os_heat role to an older SHA to solve policy problemsDimitrios Markou1-1/+1
This one follows the logic of this patch [0] [0]: https://gerrit.opnfv.org/gerrit/#/c/49327/ Change-Id: I1b71b9be6db60aa84d20ac14588b70a8e4c52736 Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.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 Buil14-88/+79
Change-Id: I1fca51b9b6afff5a596ceee2e1401127fe23231d Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-12-13Fix bug in sfc_user variables file for pikeManuel Buil4-19/+4
Change-Id: I6cd9a6fd2b3688ae784b724cedd63094e5bc6f5d Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-12-13Provide support for xci-masterManuel Buil14-107/+969
Change-Id: Id4e80be218c51f56324cb43aa27f580ac8c92c47 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-12-11Bump SHAs in OSA roles for Pike releaseDimitrios Markou1-22/+22
Getting alligned with this patch [0] [0]: https://review.openstack.org/#/c/525471/ Change-Id: I0ecc9c1ddb8f4e32ca660e7daeb1b998555b6a12 Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>