aboutsummaryrefslogtreecommitdiffstats
path: root/sfc/lib
AgeCommit message (Collapse)AuthorFilesLines
2017-03-31Fix checking for optional cmd line args in cleanupdanube.2.0danube.1.RC1danube.1.0George Paraskevopoulos1-1/+1
Change-Id: I5305e7c6426e0a563499f0aeefd910ce03e5ceae Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com> (cherry picked from commit 6d6d321145befb9f823a9d8af40e5dce83f0dbc7)
2017-03-24Revert "Change from functest logger to python logging"George Paraskevopoulos3-10/+6
This reverts commit cb311aad19c732fa006b50795ab66687e01b95a6. Change-Id: I123c30acd765df692483eb5f91fecd1478219201 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com> (cherry picked from commit e011a3d122888444c36fba6ee74b27b955356480)
2017-03-23Limit topologies to the ones workingManuel Buil1-2/+4
Due to the classifier rules only deployed where VNFs are, some use cases will fail because the client traffic never gets classified Change-Id: I77b10231b7a34497ed8ecd42d72e08e0b4481bfa Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-03-23Delete ACL when deleting a classifierGeorge Paraskevopoulos1-0/+8
Change-Id: I8e54892450e903aa51f888229940e6371a0710f7 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com> (cherry picked from commit d41124f9c8cd14cfab61d4848a70aa227e80eefb)
2017-03-22Added more logs to classification checkManuel Buil1-2/+18
The classification check is not working in CI and we don't understand why. I added a lot more logs to check Change-Id: I81bcc8874c5e5e843e0c301e1d66d5d11de092f3 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-03-21Bug fix: Eth header build with wrong dest. macManuel Buil1-1/+2
JIRA: SFC-89 The second test fetches the destination mac of the previous test because an entry with that IP is in the arp table. However, that table is outdated. We must flush the arp table before starting the new test. Change-Id: Ie00345bba13793dc68e8442421861a2555e5ba7d Signed-off-by: Manuel Buil <mbuil@suse.com> (cherry picked from commit 4d07f0f7e2f03486c8d78a24282c5e4e7fc59253)
2017-03-20Add all topologies to the testManuel Buil1-3/+1
We were limiting the topologies to the ones working but we believe all should work now Change-Id: I11921916105a149c585cd934150de2898286b142 Signed-off-by: Manuel Buil <mbuil@suse.com> (cherry picked from commit 1b9b0ad28243362b7953db12ebd072d9c6a99aff)
2017-03-20Cleanup IETF-ACL in OpenDaylightGeorge Paraskevopoulos2-0/+36
Change-Id: I6914a5fbe7eb30b2d76b89d8f38b7d3d16e7b44a Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com> (cherry picked from commit 3097fef23e4ae1c89b535c1d890e44710b33eea4)
2017-03-20VNFDs were not deletedManuel Buil1-0/+1
Added vnfd_delete in the cleanup library Change-Id: Iaa3e9ecb33a15bf882585786271fef206a54ffa0 Signed-off-by: Manuel Buil <mbuil@suse.com> (cherry picked from commit 5342d88d799359ce9f92b0cef26382d26fce9dfb)
2017-03-20Give time to start the firewall processManuel Buil1-0/+1
Sometimes test sfc_one is failing because the firewall is not started when the test begins. Waiting some seconds after starting the process corrects the problem Change-Id: Iebef292fa66338c3e27a325333cc5f1993facac5 Signed-off-by: Manuel Buil <mbuil@suse.com> (cherry picked from commit 891d42491fdabea0b3907cba8cbf220c507398b8)
2017-03-18Change from functest logger to python loggingGeorge Paraskevopoulos3-6/+10
Change-Id: Ie7b2a63c301819a79768d1b286ee00693cd1129b Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com> (cherry picked from commit cb311aad19c732fa006b50795ab66687e01b95a6)
2017-03-18Add checks for cleanup utilitiesGeorge Paraskevopoulos2-6/+26
Change-Id: I1facd54b11ef6a090e39dd5977eb5cf7183147af Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com> (cherry picked from commit 98bb8ed7510d118a680079973aa8f7dce406be37)
2017-03-18Add ODL cleanup utilitiesGeorge Paraskevopoulos2-3/+71
ODL does not clean all resources properly after tacker deletions, so we add this as a post processing step Change-Id: Ic485ee32b195f72235922ebe613af6c010cb57db Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com> (cherry picked from commit e48b950bc77e37fbda611415577a7388e19c1ff6)
2017-03-17Implement delete.sh as a python moduleGeorge Paraskevopoulos1-0/+71
We need to call delete.sh after each test is run. The script is implemented as a python module and called inside run_tests.py Change-Id: I23a794be522b4713a44cc1d56587e0aadc6f1210 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com> (cherry picked from commit 8869d175363dfec69cc70fd8cb9bef56e8ebc115)
2017-03-17Remove SFF_filter“Manuel Buil”1-15/+0
After further investigation we found out that the classifier is deployed in all computes. When deploying several tests, ODL starts to function very slowly and sometimes the classifier takes a long time to be deployed in one compute. That tricked us and made us think that it was not deployed everywhere Change-Id: I474a05f77a6342be741d14ffbdb674ef7f970908 Signed-off-by: “Manuel Buil” <mbuil@suse.com> (cherry picked from commit 70445ba93cb07164933ed0c5aac8db32f6f652d4)
2017-03-16Bug fix: rsps and first_RSPs must be restarted“Manuel Buil”1-1/+5
JIRA: SFC-88 Not a blocking bug but when the classifier is implemented in several computes, we loop all computes. By the time the second compute is inspected, more than 10 seconds passed and thus, the classification rules are already there. first_RSP should not be evaluated and rsps must be restarted Change-Id: I70858d6b423eebd8088a79aebebb4950f031878d Signed-off-by: “Manuel Buil” <mbuil@suse.com> (cherry picked from commit 189642fff7428457d3992c365eab9a9f5c0bea90)
2017-03-16Bug fix: disable non-working topologies“Manuel Buil”1-10/+13
JIRA: SFC-87 Due to ODL bugs, some topologies do not work Change-Id: Ia2fcfe5948c308ce2348b52c01aa3ce58ac318d6 Signed-off-by: “Manuel Buil” <mbuil@suse.com> (cherry picked from commit 184f787f3a8a57695fa3b9b8f17c65b9aeee1904)
2017-03-15Remove security group exists error logsGeorge Paraskevopoulos1-8/+56
We copied the create_secgroup_rule function from functest.openstack_utils and removed the logger.exception("Security group exists") line. This is very ugly and we should investigate an upstream fix after D release. Change-Id: I1674a77ebc3b3e3132b2b9e1f16a84106b2350eb Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com> (cherry picked from commit 4585d8fe2a4b40f62b96bdd38dc1a14f32510c93)
2017-03-15Bug fix: Classification check failing“Manuel Buil”1-2/+17
If both SFs are in one host, the classification rules are not implemented in all the hosts Change-Id: Ifea131a86f49d33fdc6110a13d5fdc6c2796f7fa Signed-off-by: Manuel Buil <mbuil@suse.com> (cherry picked from commit da6d1d2e018e358f71ad734766580a7343829c29)
2017-03-14Bug fix: classif. check not working for one chainManuel Buil1-11/+24
In the test case with only one chain the wait_classification function was not working Change-Id: I4cff1e61130b6fc68d736108a0da048909dad750 Signed-off-by: Manuel Buil <mbuil@suse.com> (cherry picked from commit 4b082d08b46941da06630926a117054881a8aa1b)
2017-03-01Remove get_floating_ip_2 from sfc_symmetric_chainJuan Vidal1-32/+24
Using the new functions in functest to retrieve the nova ID for a VNF instance, it is possible to use a generic solution at the problem of getting floating ips deterministcally to the instances Change-Id: Ie77820b0f466dcbf8ac771f12b5aee55ef74ead5 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-23Reduce logging level of run_cmdJuan Vidal1-9/+11
run_cmd does not have enough information to determine if something should be logged with ERROR level. For example, you might want to repeat a command until it succeeds, and you would expect it to fail the first times, so it would not be an error in that context. Let's keep the debug logging only, which should be informative for troubleshooting. Change-Id: I72bc40e58d19188b83ad289fe7fa2b4453d5ad56 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-23Merge "Improve readability of ping function"Brady Johnson1-23/+6
2017-02-22Improve readability of ping functionJuan Vidal1-23/+6
The function now uses return codes instead of parsing the output of the command. Unused options have been removed as well. Added a default retry_timeout for the command to avoid ping hangs. Change-Id: I8c1f0f03d8b1e3092743c8745399a08fa7df8e40 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-21Merge "Get vnfd parameter file location from config.yaml"Manuel Buil2-4/+6
2017-02-21Wait for flows to appear in all compute nodesGeorge Paraskevopoulos1-16/+18
Right now we check the classification rules only on the first compute node. This leads to race conditions when running the testcases on multiple computes (ie the test fails when a rule is installed in the first compute but not the other[s]) Change-Id: I13cf26201aae185a0a5d1220b38e061d8bd3c766 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-02-20Add baseline topologyGeorge Paraskevopoulos1-8/+16
Add a topology where everything (endpoints, vnfs) are booted on the same host. This is useful, as we'll have the simplest possible topology to compare against as a baseline Change-Id: I6248fbca0687286da49f6332e01571937b501090 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-02-20Get vnfd parameter file location from config.yamlGeorge Paraskevopoulos2-4/+6
Change-Id: Ide425e860ad219ec24765905666994390858cfdb Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-02-20Use __name__ on Logger instantiation instead of custom stringsJuan Vidal3-3/+3
Change-Id: Ic68256410b29cf3ba39a10a92d243979ea475220 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-17Introduce new test case: symmetric service chainJuan Vidal1-0/+32
JIRA: SFC-53 The goal is to verify that the traffic traverses the service chain, not only in the uplink (client --> server) direction, but also in the downlink (client <-- server) direction, which is the part that is not currently being covered by other test cases. Change-Id: I198a53e5afe3da1b397cee95627834856498b674 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-17Run sfc_two_chains in multiple topologiesGeorge Paraskevopoulos2-54/+47
JIRA: SFC-63 The test will run in a number of configurations as described in SFC-63. We expect to get decent coverage with these 5 topologies: 1. CLIENT_VNF_SAME_HOST 2. CLIENT_SERVER_SAME_HOST 3. SERVER_VNF_SAME_HOST 4. CLIENT_SERVER_SAME_HOST_SPLIT_VNF 5. CLIENT_SERVER_DIFFERENT_HOST_SPLIT_VNF Change-Id: Iaca75bf180c33e3d22759e4da1ca4897072f68a6 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-02-17Fix topology bugsGeorge Paraskevopoulos2-5/+15
- Fix getting the length of available topologies - Format a correct vnf parameter file Change-Id: I6c3534a67b44be6fcfb09d99856e5c27bcbd2ed1 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-02-17Fix check_ssh to allow any number of SFs to be checkedJuan Vidal1-1/+1
Change-Id: Ida31e1e7f1b638f1ba5e851cbdfd0379c0f546d4 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-17Merge "Add source port option for netcat utility"Manuel Buil1-10/+22
2017-02-17Merge "Add topology_shuffler"Manuel Buil1-0/+151
2017-02-16Add source port option for netcat utilityJuan Vidal1-10/+22
Change-Id: I3dedc06f0b288684a907df14874f277449c33146 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-16Add topology_shufflerGeorge Paraskevopoulos1-0/+151
JIRA: SFC-63 This module will be used to configure the assignment of endpoints and VNFs to compute hosts dynamically Change-Id: Ie778b52796c76d829ccb3d49e1c9bc7f5e7c7b47 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-02-16Merge "Bug fix in sfc utils lib"Manuel Buil1-1/+1
2017-02-16Bug fix in sfc utils lib“Manuel Buil”1-1/+1
The command to add route to the VM was wrong Change-Id: If45425a5bdc8ba9d2ecb37dd9914628b6963ed28 Signed-off-by: “Manuel Buil” <mbuil@suse.com>
2017-02-16Merge "Pass availability zone as param on vnf create"Manuel Buil1-0/+18
2017-02-16Fix is_ssh_blocked and is_http_blockedJuan Vidal1-21/+17
This patch fixes JIRA SFC-79 Previously, those functions used regex matching against the command output. Now, the return code from netcat is used. It should be easier and more relieable. Change-Id: I639c64b3fe0a0e3c0b59f73e5615b8737e676849 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-15Pass availability zone as param on vnf createGeorge Paraskevopoulos1-0/+18
JIRA: SFC-63 JIRA: SFC-73 Instead of changing the vnfd-template file in runtime we can pass the availability zone as a parameter when the vnf is created This enables also the use of topology_shuffler for SFC-63 Change-Id: Ie650e3347e40aee030005b6a2272cae64968d303 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-02-15Modify run_cmd to return exit codes as well as stdout and stderrJuan Vidal1-12/+14
The old implementation of run_cmd returned different things in different situations, so the simplified new version broke some functions relying on boolean values instead of stdout. Change-Id: Ie94e529d2abfad171b9707df169925c6f5f47dfc Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-15Refactor utils using installer adaptersGeorge Paraskevopoulos2-77/+36
JIRA: SFC-65 JIRA: SFC-71 Use the new library in releng to talk to fuel. This allows for correct configuration of the compute hosts to resolve SFC-71 Also since https://gerrit.opnfv.org/gerrit/#/c/28281/ was merged it allows to run in multienv deployments so it resolves SFC-65 Change-Id: I5e6beb90d9c5108c21acb8d898f07cd2f4ae34f3 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-02-14Extend the logging in the utils lib“Manuel Buil”1-0/+2
To learn about some of the bugs I saw, I added these lines to know what is going on. It was very useful to me and I thought that perhaps we can add it to the code to help in the future Change-Id: I070077749d2086489a536c9a7b7ecca4a05bb0a1 Signed-off-by: “Manuel Buil” <mbuil@suse.com>
2017-02-13Fix bugs when setting up computes“Manuel Buil”1-2/+3
JIRA: SFC-71 1 - dev br-int was missing when adding the route 2 - grep exits with 1 when nothing is found triggering an ERROR log even though there was not error Change-Id: Ib65d74d4b01021dcabb5af9068451b97c0170f75 Signed-off-by: “Manuel Buil” <mbuil@suse.com>
2017-02-08Merge "Avoid looking for VMs from tenants different from the one running the ↵Manuel Buil1-1/+1
testcase"
2017-02-08Merge "Simplify run_cmd"Manuel Buil1-11/+5
2017-02-08Merge "Modification of the ofctl_time counter"Manuel Buil1-2/+17
2017-02-07Simplify run_cmdGeorge Paraskevopoulos1-11/+5
sfc.lib.utils.run_cmd has a lot of unnecessary options which are never used. This patch simplifies this function and also fixes the returncode checking (commands have terminate with errors on returncode != 0 not on return code < 0) Also change the return value on error to None instead of False Change-Id: I5f1068ffd1bc6091e866a002b1afcb2507d30981 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>