Age | Commit message (Collapse) | Author | Files | Lines |
|
|
|
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>
|
|
- 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>
|
|
As per I6b57eab89839d9e9122cd24b0f05737467439dd9:
- Add a route distinguisher
- Remove the router from the network that is network associated
Change-Id: I1399bb51c787df881529279ef7a1946a3c5c5591
JIRA: SDNVPN-66
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
|
|
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>
|
|
Change-Id: Ib0b1a8c1b82c57d76d97520cbc27f043fede1352
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
|
|
- 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>
|
|
The latest functest enforces using `get_parameter_from_yaml` to only
get test specific parameters from the config file.
`get_functest_config` is given to get functest configuration parameters.
Change-Id: Icb6ec2ed4a9d6c278bf1c403d4d0c4f4b21e51b7
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
|
|
- 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>
|
|
Due to a small refactor of some functions in Functest,
the calls to those are outdated and fail.
Change-Id: I1fdce5d343fc55ecd1b25a837608c7172047f22e
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
|
|
JIRA: SDNVPN-49
This functional test is a variant of testcase 1. Instead of associating
Network 1 and Network 2 to the BGP VPN, we connect Network 1 to a
router and associate that router and Network 2 to the VPN.
The testcase is disabled for now.
Test setup procedure:
- Set up VM1 and VM2 on Node1 and VM3 on Node2, all having ports in the
same Neutron Network N1 and having 10.10.10/24 addresses in subnet
SN1. N1/SN1 are connected to router R1.
- Set up VM4 on Node1 and VM5 on Node2, both having ports in Neutron
Network N2 and having 10.10.11/24 addresses in subnet SN2.
Test execution:
1. Create VPN1 with eRT<>iRT and associate R1 to it
- Ping from VM1 to VM2 should work
- Ping from VM1 to VM3 should work
- Ping from VM1 to VM4 should not work
2. Associate SN2 to VPN1
- Ping from VM4 to VM5 should work
- Ping from VM1 to VM4 should not work
- Ping from VM1 to VM5 should not work
3. Change VPN1 so that iRT=eRT
- Ping from VM1 to VM4 should work
- Ping from VM1 to VM5 should work
Jira task in SDNVPN: https://jira.opnfv.org/browse/SDNVPN-49
Change-Id: I0e805b042eb645e923f1639179c4c2c37be17835
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
|
|
Change-Id: I19c3e8b979376a38b8a5838b7a41a3915f90b57f
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
|
|
Change-Id: Iebaa6f15b16f91e451662e13e0542f863f93da52
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
|
|
The reporting Dashboard looks for the test case called
'bgpvpn'. Now, we report the overall status of the tests
into that test case and also each one separatelly.
http://testresults.opnfv.org/test/api/v1/results?case=bgpvpn
Also, fix the 3 flake8 violations
Change-Id: I5b0d816894bf16d3d11085437f1556b5d0446d81
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
|
|
JIRA: FUNCTEST-369
Change-Id: Ib8bd3dca7c5653e282f7c58023db6bb743db9c6d
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
|
|
JIRA: SDNVPN-19
The test scripts now return a json object with
the following format:
{"status": criteria, "details": results}
where
criteria: "PASS"|"FAILED" (result of the test case)
details: free output that will be printed to the console
and pushed to the DB
Change-Id: I9e5e0c3b6cb5f4b060929b71a06f6e4f95f814fb
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
|
|
|
|
The tempest.conf is copied to bgpvpn_tempest.conf to be updated
with the needed parameters, in this case, only adding
bgpvpn=True.
The way to execute this module only is with the option -C.
For example:
run_tempest.sh -C bgpvpn_tempest.conf -t
-N -- networking_bgpvpn_tempest
Change-Id: I465961bc619f59355be964eb94f916cb877b2273
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
|
|
JIRA: SDNVPN-31
It seems due to the merge conflict, some of the lines got
reordered and the script failed.
Also, added the private key for the instances.
Change-Id: I58c3dfe5ba2d43e0749ca5cd08b3a87221a68b66
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
|
|
Change-Id: Ibd97de79eb8d824120eb2a1b80017ff1f9f99f14
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
|
|
JIRA: SDNVPN-31
Test case 2 checks tenant IP address space separation.
Now done with this change.
Change-Id: I07ef04de4195b2892c837ac87e53146d133755c6
Signed-off-by: Tim Irnich <tim.irnich@ericsson.com>
|
|
JIRA:SDNVPN-31
Change-Id: I0e15afa737e4cc65fb6235ed4400ea6b1523a327
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
|
|
JIRA: SDNVPN-30
Change-Id: I4d31e9c4dc683c031d6a3d44e0d2bf4d80332281
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
|
|
JIRA: SDNVPN-19
Change-Id: I358efcb377211182a93b2db43f248ca82c14fb36
Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
|