summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2017-03-22Added more logs to classification checkManuel Buil3-2/+24
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-22Perform cmd argument parsing when called from cliGeorge Paraskevopoulos1-11/+12
To be able to import run_tests as a module we must not try to parse command line arguments in global scope Also import the testcase using the full module path Change-Id: I1696800a16301d446bf9c926bdaf2b545435a2e7 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com> (cherry picked from commit 08aa86e1e7cb874aac72c831e239c003070e5ca3)
2017-03-21Bug fix in CI: security groups not workingManuel Buil2-1/+6
CI tests create a lot of security groups with the name default. Our SFs are assigned to a default security group but there are so many that the result is not predictable and most likely wrong. We add it now to example-sg sec. group Change-Id: I64984aa7b6114747c5bab5fb2e4b8d8d32865bb0 Signed-off-by: Manuel Buil <mbuil@suse.com> (cherry picked from commit 3c6e0efc2f4575ec2cf2e1304bdc1b66cdc08b06)
2017-03-21Bug fix in CI: security groupsManuel Buil1-0/+5
CI tests create a lot of security groups with the name default. Our SFs are assigned to a default security group but there are so many that the result is not predictable and most likely wrong. We add it now to example-sg sec. group Change-Id: I120765a33e55e47b78ef3c8664c82fb147ce9df0 Signed-off-by: Manuel Buil <mbuil@suse.com> (cherry picked from commit 6282c5cce71ae994aaece374375234cf207992f8)
2017-03-21Fix run_testsManuel Buil1-14/+16
It was impossible to get a status='PASS' Change-Id: I016346c1acf38ae1a130afbfd132166b72d5e1af Signed-off-by: Manuel Buil <mbuil@suse.com> (cherry picked from commit d63f9fc8f18f42cd80a32e3b93aed19091beccee)
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-20Disable resource_finder_cache in controllersGeorge Paraskevopoulos1-0/+24
Heat caching somehow messes our tests so we disable it before the tests start. For instance, if we do: 1. create example-net with NET_ID1 2. create VNFD1 with example-net 3. delete VNFD1 4. delete NET_ID1 5. create example-net with NET_ID2 6. create VNFD2 with example-net heat has NET_ID1 cached as the id of example-net and tries to use that instead of NET_ID2 to create VNFD2. This off course leads to a crash since NET_ID1 does not exist anymore. In the future we should investigate further why heat cache does not play well for us (probably bc of tacker). Change-Id: Iac07d78641e2d1ff0c2d67e46752d551833f19fd Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com> (cherry picked from commit d0fec46322915ef8b49e1f7ab9c485ea2cdd399a)
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-18Run tests in orderGeorge Paraskevopoulos2-5/+12
Change-Id: I3cbbcd17182bf6f1f6558cb199577d989ac62ae6 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com> (cherry picked from commit d3b5ba3c16b0918060d1fc800bd47e688146f2c4)
2017-03-18Change from functest logger to python loggingGeorge Paraskevopoulos6-12/+19
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 Paraskevopoulos3-5/+76
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-17Add more logs to the CI“Manuel Buil”1-0/+11
We want to double check if the CI is using the correct tacker plugin version Change-Id: Ib6082014988daee316553d980c5e7f88d316fe8c Signed-off-by: “Manuel Buil” <mbuil@suse.com> (cherry picked from commit 268fc4edfd7560bcf5b5e1248df05face402074a)
2017-03-17Implement delete.sh as a python moduleGeorge Paraskevopoulos2-0/+73
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”3-23/+3
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-17Run all testcases even if one throws an exception“Manuel Buil”1-1/+5
If the first testcase throws an exception, the rest were not being executed Change-Id: I5fdc521da840ffdbc7a0bde120d7bfe7907b881c Signed-off-by: “Manuel Buil” <mbuil@suse.com> (cherry picked from commit 84f3ea23ccc9f6e19a86748519deb600b2ac95e3)
2017-03-17Log credentialsGeorge Paraskevopoulos1-2/+8
We have to make sure that we source the correct credentials in the CI Change-Id: I4d988943588bbc99a3118cb89eabfea37c41daf6 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com> (cherry picked from commit 1d72729b1513e419ab1d97f294410a5cfdd1c62f)
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”3-5/+27
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 Buil3-13/+29
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-14Bug fix: client and server not using shuffler“Manuel Buil”1-2/+2
Client and server were not being created based on the topology shuffler and thus the topology was not followed Change-Id: I1d7228d2d75e4ecf57ff8eb1c4226b963b2b9dd4 Signed-off-by: “Manuel Buil” <mbuil@suse.com>
2017-03-10Using installer adapters for second test case“Manuel Buil”1-104/+70
The second test case can also benefit from installer adapters Change-Id: Ie2ce8c9112582fa39428e5410f1c7ae6274161ea Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-03-10Disable symmetric test case to not test it in CIManuel Buil1-1/+1
symmetric test case doe snot work in multicompute env. We want to avoid that it is executed in CI because it will always fail in the Danube release Change-Id: I2a12d9aea31df37c689d96ae7eb323339b8ffb89 Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-03-08Adding LabelsShubhamRathi4-0/+8
Labels serve as anchors/ refs when referenced from other documents This patch adds labels to docs/*/*/index.rst Change-Id: I22964ee60952047ca20d8d4c54db790b9c01ee5a Signed-off-by: ShubhamRathi <shubhamiiitbackup@gmail.com>
2017-03-03Fix path where we get tackerc from controllerGeorge Paraskevopoulos1-1/+1
SFTP does not expand ~ to $HOME dir Change-Id: I2d710c577986152a5cc703ca3668ae7b7767a07e Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-03-02Merge "Add marks to specify that symmetric_service_chain test is not ok"Manuel Buil2-0/+6
2017-03-02Add marks to specify that symmetric_service_chain test is not okJuan Vidal2-0/+6
Due to JIRA SFC-86, this test is not working (althouth it will pass with a topology where all the VMs are deployed in a single compute node). Change-Id: I98231d22a0b340c305ba83ee78634c83fa4dc7e7 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-03-01Remove get_floating_ip_2 from sfc_symmetric_chainJuan Vidal2-36/+41
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-24Merge "Create downstream classifier in symmetric-chain test"Manuel Buil2-7/+32
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-23Create downstream classifier in symmetric-chain testJuan Vidal2-7/+32
Also, fix some wrong parameters, and add a script to delete the artifacts created by the test case Change-Id: I354007e75da62c706e79c793a9e6a422a6a50bf1 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-23Merge "Improve readability of ping function"Brady Johnson1-23/+6
2017-02-23Use baseline topology on symmetric chain testcaseJuan Vidal1-5/+17
The topology for this test is fixed for the Danube release. When it is stable for some time, we can try changing topologies. Change-Id: I43f06ff973c74fbaea3aa9161f7a177b3ee94791 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
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 Buil5-6/+24
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 Paraskevopoulos5-6/+24
Change-Id: Ide425e860ad219ec24765905666994390858cfdb Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-02-20Use __name__ on Logger instantiation instead of custom stringsJuan Vidal8-10/+8
Change-Id: Ic68256410b29cf3ba39a10a92d243979ea475220 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-17Introduce new test case: symmetric service chainJuan Vidal5-0/+292
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 Paraskevopoulos3-58/+70
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 "Check that VNFs boot properly"Manuel Buil2-8/+6