aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2018-06-04Provide support for local imagesManuel Buil1-2/+13
There might be cases where we want to test local images and the image location in config.yml is locally Change-Id: I29f115de5c80db187ecc2b260c386b377fafd6d2 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-06-01Fix the logging to be compatible with functestPanagiotis Karalis13-38/+38
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-23Update the release notesManuel Buil1-3/+3
Change-Id: I31506971cf57d761f966841029e3a878d2a0d284 Signed-off-by: Manuel Buil <mbuil@suse.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-21Merge "Revert "Add iteritems() when parsing dict in Python 2.x""Manuel Buil1-1/+1
2018-05-21Increase loggingManuel Buil1-0/+1
Apex deployments are currently failing and it could be because it uses the wrong config file. We are currently not logging what file it is fetched and that's what this patch tries to fix Change-Id: I441beee9d3ad5d918ed4b8dc0164456b39e53b56 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-05-21Improve logging and unify how to read from yamlManuel Buil1-4/+6
vnffgd and vnfd are being parsed in different ways, we should parse them in the same way. We could do it directly with safe_load, however, the returned object is not easy to read. If we use read() instead, the resturned object is easy to read/log. Therefore, this patch combines both options Change-Id: I3416933801569a6f57962cd9906726229923136d Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-05-21Revert "Add iteritems() when parsing dict in Python 2.x"Manuel Buil1-1/+1
This reverts commit e29fe40e0c98d6c3b619de7f99f0a8fb9e4a9d7b. acl_types_names is not a dictionary but a list of tuples: (Pdb) pp acl_types_names [(u'ietf-access-control-list:ipv4-acl', u'04a5ce7c-6aac-4f26-b3f9-a19749d814f3'), (u'ietf-access-control-list:ipv4-acl', u'75737a58-c894-46d9-9319-8d2b4785b0f3'), (u'ietf-access-control-list:ipv4-acl', u'bb422318-c3a3-4356-a67c-f44e8874edf6')] (Pdb) type(acl_types_names) <type 'list'> (Pdb) type(acl_types_names[0]) <type 'tuple'> Therefore, this patch introduces an error Change-Id: I7f7e69fb1855c7915b2c17d80a5b7ea76aaa8694 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-05-18Merge "Clean up Test Case code"Manuel Buil7-1019/+870
2018-05-16Merge "Add iteritems() when parsing dict in Python 2.x"Brady Johnson1-1/+1
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-14Add latest_release as default6.0.1Manuel Buil1-1/+1
If nothing is specified, let's deploy with the latest release which we know works correctly Change-Id: Ia38e4b5e5c32b886dfdd5ab76654241b495d6863 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-05-10Merge "Lookup ODL_VERSION variable"Manuel Buil2-0/+7
2018-05-09Add iteritems() when parsing dict in Python 2.xDimitrios Markou1-1/+1
The SFC code still runs with Python 2.x version and that means that when a dictionary is getting parsed it should use the iteritems() function. Change-Id: I44f1c9cb916e7b76b277e376c8f524a99aa01f26 Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
2018-05-09Add more logs to the codeManuel Buil5-6/+16
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-08Lookup ODL_VERSION variableManuel Buil2-0/+7
We require that variable to select different ODL versions Change-Id: I37aca357a7a5e0772529dac70ec8ee81b52f3cf8 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-05-03Bug fix: Use opnfv.logger objectManuel Buil12-59/+39
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-05-02Fix version in setup.cfgCédric Ollivier1-1/+0
Selecting another tag than the official ones (opnfv-xxx) breaks all Functest builds because pbr requirements are unmet [1]. [1] https://build.opnfv.org/ci/job/functest-features-docker-build-amd64-master/219/console Change-Id: I8a6a603c9f4e976c8bcdcce865850b46f6e69453 Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2018-04-25Update docs for FraserManuel Buil7-247/+61
Change-Id: Ie53eb812c9111a2f75342470b10ef3376e43b5fb Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-04-18Track the indirectly created security groupsManuel Buil2-1/+18
networking-odl creates a security group when creating a router. This security group is not tracked by SNAPs and thus it never gets removed when running the clean up. We need this workaround until the following bug is fixed: https://bugs.launchpad.net/networking-odl/+bug/1763705 Change-Id: I15b939609e815f24ef0c1dcc0aeb9f4471f351f0 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-04-16Merge "Provide support for different ODL versions"Dimitrios Markou7-4/+40
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-13Merge "Updated from global requirements"Manuel Buil1-3/+3
2018-04-12Changing README6.0.0Manuel Buil1-55/+4
Change-Id: Ia2112794ef542ff91af1a547d814a0f0c737dd54 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-04-12Provide support for different ODL versionsManuel Buil7-4/+40
We should be able to deploy with different ODL versions if the user defines the ODL_VERSION variable when deploying XCI. Change-Id: Ied20959116fed19bf4445f1544883ed0dc579dfb Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-04-12Updated from global requirementsCédric Ollivier1-3/+3
Change-Id: I532d83d1e381b2ef0559fdcc27e1bf455002f8ce Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2018-04-11Merge "Auto Generated INFO.yaml file."Aric Gardner1-0/+67
2018-04-09Use different config files depending on versionManuel Buil11-52/+150
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 Ruiz8-94/+360
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 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>
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>