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>
|
|
|
|
Removed some placeholders and todo notes, updated copyright and did a final readthrough.
Change-Id: I1357174c159ae1065c4452128152140bb6b00d11
Signed-off-by: Tim Irnich <tim.irnich@ericsson.com>
|
|
Change-Id: I58889c0ab1117a51a59b9a4f83ade494286d5517
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
|
|
Change-Id: I909119b0184e02c169cbb694d4cc46bf5a9ed543
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
|
|
|
|
|
|
Change-Id: I6439f1cf4ef3f7ec14fa9bbfd1d5ad17e0fcc1db
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.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>
|
|
Change-Id: I47108a141390499570677253a5cdad78ebbe9240
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.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>
|
|
Change-Id: Ia4abb1e64ef8643e81953b00ae7e3d5a87640cf4
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.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: I065d96cf9692155c336476ec3ad4f4bc92287740
Signed-off-by: Tim Irnich <tim.irnich@ericsson.com>
|
|
Change-Id: I143346d5268ad53eafcb8d2f5420173440f4c2a3
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
|
|
Change-Id: I501845e5200f6df638842dff93ee12346a75ec03
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
|
|
JIRA: DOCS-103, DOCS-134, SDNVPN-40
Moving documentation content into new feature description and user guide template.
Change-Id: I2c1549b44ef43e28961f1a0d2885adc0dd161910
Signed-off-by: Tim Irnich <tim.irnich@ericsson.com>
|
|
|
|
Added a draft of the scenario description template for consideration and review.
This is a draft migrating the existing userguide text into the proposed
feature project userguide format for Colorado. Please review and comment,
correct and merge as needed. (this is the prototype)
Change-Id: Ia82a0c3ba0f654efc2892aac3a3b2d217ed2ad27
Signed-off-by: ChristopherPrice <christopher.price@ericsson.com>
|
|
Change-Id: I067046949d9cd4a764c279c053a2f54b30645fda
Signed-off-by: Nikolas Hermanns <nikolas.hermanns@ericsson.com>
|
|
Change-Id: Ia237ab7347c789761e36bec1cf28372f8fbcefbc
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>
|
|
JIRA: DOCS-134
Adding structure of docs for installation
procedure for virtual deployment.
Introducing a scenario-specific directory level.
Change-Id: I01de7d3b876ba9d5f7b71b91c857842fd3badeef
signed-off-by: Tim Irnich <tim.irnich@ericsson.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>
|
|
Change-Id: Ie9728bafed45dbff444102c85ba10501130521ce
Signed-off-by: ChristopherPrice <christopher.price@ericsson.com>
|
|
|
|
Adding release notes for Brahmaputra.3.0.
Change-Id: I262f4e0ab7ec785088601b48557555e07e03f870
|
|
Change-Id: Ibb7efe2c6a476e205f806e99afdfd041f3740f36
|
|
Some additional content for feature config guide.
Fixing a typo in userguide.
Change-Id: I1f3056535052e74ec893b9f3264a154a145c2350
|
|
Change-Id: Id0d67c804d721baf41828b5aff01990b729f3406
|
|
Change-Id: Iff53422b5e609a21e7165946d776e66fdfdbe431
Signed-off-by: ChristopherPrice <christopher.price@ericsson.com>
|