summaryrefslogtreecommitdiffstats
path: root/test/functest/utils.py
AgeCommit message (Collapse)AuthorFilesLines
2016-12-02Move logging instance creation to create_instanceRomanos Skiadas1-0/+3
Change-Id: I299f2be3b127aefdd7036fa68fb934b09d278692 Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
2016-11-18Refactor config global variablesGeorge Paraskevopoulos1-8/+3
- Create config.py module to handle configuration parsing - Remove globals from testcases Change-Id: I16ed7bb138fde2d5d3c69703e3c2e569d49f0b37 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2016-11-10Encapsulate sleeps in functionGeorge Paraskevopoulos1-0/+6
We have some sleeps into the testcase code. We can encapsulate them in a utility function to refactor them out of the code until we figure out the best way to remove them completely Change-Id: I44c32d8329c2e319660518a8a46debc22f02c753 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2016-11-07Refactor common code across testsRomanos Skiadas1-0/+17
- Move collecting test results in a method in Results class - Create a util function for asserting the environment has enough compute nodes Change-Id: Ib7b9dfab7d2e9f50d5f352d3eadba4d4717ee750 Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
2016-11-02Fix network assoc tests in BoronRomanos Skiadas1-0/+28
Network assoc in Boron with netvirt does not work for subnets that have a router due to an inherent limitation in the implementation. See the mailing list and related bug: https://bugs.opendaylight.org/show_bug.cgi?id=6962 https://lists.opendaylight.org/pipermail/netvirt-dev/2016-October/001815.html Also, if a subnet does not have a route distinguisher, it is not added to the FIB, so association does not make traffic move between subnets. This is intentional and an error is logged when a subnet is associated to a bvpvpn without a route distinguisher. This commit fixes the net assoc case and works around these issues by: - Removing the routers from the subnets in testcase_1 - Adding a route distinguisher to the bgpvpn JIRA: SDNVPN-74 Change-Id: I6b57eab89839d9e9122cd24b0f05737467439dd9 Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
2016-09-15RefactoringRomanos Skiadas1-4/+135
- Wrap common methods that use globals in a Class `get_ping_status` and `add_to_summary` depended on globals for statefulness. The easiest way to refactor them was to convert them into methods and the globals they used into instance variables. - Move `generate_userdata_common` and `generate_userdata_with_ssh` to the utils module. - Make flavor a kwarg in create_instance. Add a defaults section in config.yaml with flavor. This value is used if flavor is not passed via kwargs. - Substitute some sleeps with `wait_for_instances_up` which will wait until a dhcp lease success message appears in the instance's log. - Add functions that wait until router and network association is completed. These do not replace the sleeps but make the tests more deterministic. - Remove unused ping_timeout from config.yaml Change-Id: Ia68ebf47aaa5a58c4e20267f1476db626f015a7c Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com> Co-Authored-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2016-09-09Refactor some common methods into utilsRomanos Skiadas1-0/+112
- generete_ping_userdata(): This method was exactly the same in two tests - create_instance(): The various implementations of this method had almost the same signature, but not quite. This was fixed by adding extra arguments as needed. Also a global variable was used in the function, which was converted into a parameter. - create_network(): The three implementations of create_network had three different signatures and returned different variables. They were all unified in a single create_network class and the calls to create_network were adjusted accordingly. Revision 1: Fix flake8 violations Revision 2: Rename logger in utils Change-Id: Ib6cfe798dc561a69eb50c901e3aa71c19d465821 Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>