Age | Commit message (Collapse) | Author | Files | Lines |
|
There was a bug in releng and pylint was not ran, some issues got merged
in master because of that
Change-Id: Icf3fa882f4f2b0593fa0b85edce46e49dcc303c9
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
Looks like using tacker_client.get to request a resource collection
causes connection drops and subsequent requests to fail, probably
because of incomplete pagination handling. Use tacker_client.list
instead.
Change-Id: I79ba39759b61f57774e65b5e49686edb6bf05c13
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
|
|
SNAPs is not supporting get_vm_info() anymore, we should remove it from our lib
https://gerrit.opnfv.org/gerrit/#/c/53263/4
Change-Id: Idad1bb60b86c45256bb0a95b6c554e83e6bfae35
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
|
|
|
|
Change-Id: Id524fbf1677844b5172f52c857cf8bed8d075ff3
Signed-off-by: Linda Wang <wangwulin@huawei.com>
|
|
JIRA:SFC-120
Snaps updated two methods which now require more parameters:
https://github.com/opnfv/snaps/blob/master/snaps/openstack/utils/heat_utils.py#L260
https://github.com/opnfv/snaps/blob/master/snaps/openstack/create_instance.py#L781
Change-Id: I164c51298afd7b80ee88312f3a09abe7f09282a0
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
Change-Id: I44d0dd4fdabd38f09e6113c28668f11a40c6ba86
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
SNAPs changed the API:
https://github.com/opnfv/snaps/commit/c711acf8ae3e4ad6f746500747857bcc9fd6f7be#diff-0b922a3e087121238d5a8b08ef5b894aR480
Now we require to pass keystone client to get the neutron port
Change-Id: I93fa4b127255c1a078aa3644773b7c8b886cea6d
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
It switches from functest_utils.get_functest_config() to
config.CONF(). download_url is now hosted in sfc as it's unused by
Functest.
ft_utils.get_parameter_from_yaml() will be considered as part of
Functest framework (then it must be well covered and documented).
Change-Id: I3014d90dba53422020ae85a2815c5bf8c0dcf34c
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
CONST is not the right Functest object to get config data.
It also updates the way to get constants as proposed by Functest [1].
[1] https://gerrit.opnfv.org/gerrit/#/c/52389/
Depends-On: Ie23c0ef90efc839d60f0f3a9754c58746b1f3a00
Change-Id: I6cfa832466dcefd737314633d807512e46267a69
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
CONST shouldn't be selected to get env vars.
Now it may raise KeyErrors if env is not well configured.
Change-Id: I5e3f525640809778c5349fdc1c3796227894d0cd
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
It follows the next changes (Functest) which renamed the related
config entry [1].
[1] https://gerrit.opnfv.org/gerrit/#/c/51283/5/functest/ci/config_functest.yaml
Change-Id: Ieae5baff13bb70aa35b01c2358ab4865cdb9dfb0
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
env.get() ensures that default values are reused if unset.
It also avoids desynchronization between os.environ and CONST.
It follows the change "Get properly env vars or their default values"
[1] applied in Functest.
It also fixes minor issues on shebangs and takes several pylint rules
into account in this module:
- http://pylint-messages.wikidot.com/messages:w0702
- https://www.python.org/dev/peps/pep-0282/
[1] https://gerrit.opnfv.org/gerrit/#/c/52221/
Change-Id: Ia7ad168b2cbbade8ea3890fb95dc01bcdfc50468
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
Functest removed the function from their repo, so we need to define it in ours:
Change-Id: Ica122e57612c38751d8ff810f3ba61b749e867eb
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
This is the flow of the test:
1 - Create a vnffg in tacker (i.e. chain + classifier)
2 - Remove the vnffg
3 - Check that the RSPs are removed from operation DB in ODL and
the classifier rules are not present in the flows
4 - Create a new vnffg
5 - Test the new chain:
5.1 - Check HTTP traffic works
5.2 - Block HTTP traffic in VNF and check that it does not work anymore
Change-Id: Ia077e58c5cbce4258a2f8a36f6b961eb923ff9c5
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
Change-Id: I11c37ed6898c03b2a54a0e3b11422c654023333f
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
The variable TEST_DB_URL must be read as a env. variable since this patch":
https://gerrit.opnfv.org/gerrit/#/c/49757/
Change-Id: I8f77acbbb2fe949f5fbe8a3f983c49224d893bcc
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
When migrating to SNAPs a bug was introduced and could be seen in logs as:
Error when waiting for classification rules: 'module' object has no attribute
'get_compute_client'
The cause is that get_compute_client is not a global function anymore but a
method inside a class. Besides, due to SNAPs, we need to fetch the compute
name in another way
Change-Id: I32007c83894281ba9514a121b7939b5bfe52f4bc
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
A floating ip remains when executing the clean up function. That floating ip
belongs to the Tacker VNF and thus we should add that creator to the list
Change-Id: Ic15574bf93f30a26f81a4944d742307dda4bfe6d
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
Functest is deprecating its openstack library which we were using extensively
We need then to move to an alternative and the best appears to be SNAPs
Change-Id: Icaa0b9f1ec580545b9d0faa88be2080a310deaf7
Signed-off-by: Manuel Buil<mbuil@suse.com>
|
|
Utils.py was getting messy. This patch divides it into three different files:
* test_utils.py
* odl_utils.py
* openstack_utils.py
The tacker library is integrated into openstack_utils.py
Change-Id: I310949d1cee49b6aa1c9b3396bf6d6ca458cbaac
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
Apex deployment shows a problem when creating the SFC config in ODL and
it is impossible to know what might be going on
Change-Id: If8d33d1c60daaf9cd9773a5cb9c7e47f64e1caa9
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
Sometimes VM starts very quickly and then it takes longer for this service
to come up
Change-Id: Ibc4ea47eb0b4b6655e338d1b85e6ed14e61d0064
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
After testing, there are three topologies which work and three topologies which
do not work because of bugs in OVS and ODL. We remove those two from the
testing in 5.0.0
Change-Id: I576768978bfa763725027239081686c95408eef4
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
JIRA: SFC-117
Change-Id: I20e12e38a3cf2a9954893cf52187ac153b64bdfa
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
|
|
JIRA:SFC-113
We used to check that the ACL and RSP flows were implemented because it took a
lot of time. However, the new ODL Nitrogen reduced this time a lot and thus we
must check that the vxlan_tool is up and the HTTP port is up in the server
before doing any test
Change-Id: I70219071c7965781b17e80631bf3e2efbfcfe4cd
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
JIRA: SFC-110
Sometimes the test is quicker than ODL and when it checks the configured RSP,
that configuration has not yet been submitted in ODL, thus ODL replies with an
empty list. We add an iteration to try again for a couple of times
Change-Id: I21e4ac3fc3a919791daa7e41cb2953f7565c4ebb
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
JIRA: SFC-107
We are submitting the vnffgd with a variable for the neutron_port. Before
creating the vnffg, we must provide a file with the neutron_port value
Change-Id: I9994e2baa53f323df17434448d372598c0d94c72
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
It is currently impossible to obtain the vnfd_id based on the vnfd_name when
we want to create the vnffg
Change-Id: Ib7752156db43d76c24f8c6c2803eb7dbc5bbe1d8
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
JIRA: SFC-106
In sfc_two_chain testcase we need to delete first the blue
and then the red vnffg because if we do it the other way around
the tacker produces a warning that the flow classifier of the vnffg red
is in use and cannot delete the chain.
The aforementioned warning is produced because during the execution
of the two_chain testcase we reassing the classifier from the red vnffg
to the blue vnffg. So if we try to delete first the red vnffg we get the
warning because the classifier is assigned and in use from the blue
vnffg.
Change-Id: I3cb7bef4b9b9e75eef4961fe7f63cbc1755130be
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
|
|
Two things must be done:
1 - Be compatible with the new way of doing SFC in ODL Nitrogen
2 - Be installer independent
Change-Id: Ibbb3057db30a7c00e92f788bb03e770749e2b255
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
|
|
Change-Id: I078e9f26d3040cf04120abd425f03cb4f2e8daef
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
|
|
We are going to use ODL Nitrogen which does not require the NSH hack as
packets reaching the VNF will use ETH+NSH encapsulation
Change-Id: I5a55fde84e8f17860783ec9de8b99242fc38b9a9
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
Change-Id: I710e6f5da92ddfddd9111c6c53274b92d76d7449
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
|
|
Right now the testcases are dependent to only one installer.
So when we are trying to run the testcases to multiple installers
the tests are failing.
This patch make the testcases independent to the installer that they
run.
The testcases are running until the point which they hit the tacker
client commands.They are failing in the tacker client commands
because we need the new library for the upstream tacker API.
The testaces have already tested to two installers (Fuel,Apex-ha).
Also this patch contains the changes which are required for the
adaptation to the new functest.
For more information see the Jira tickets [0],[1],[2].
[0]: https://jira.opnfv.org/browse/SFC-100
[1]: https://jira.opnfv.org/browse/SFC-101
[2]: https://jira.opnfv.org/browse/SFC-102
Change-Id: Id1e5d5c94a65ab8bdea9584fa833bfa0cdec6632
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
|
|
Today I learnt that the use of except Exception, variable is not recommended
anymore and it is not supported after python3. When using >=2.6, the syntax
should be except Exception as variable.
https://www.python.org/dev/peps/pep-3110/
Change-Id: I213c220fc6e92878ffa4737efa16adb76ace1271
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
- Recently there is removal of functest logger utils.
- This uses the default python logging mechanism
Change-Id: I3aba38041bbed8b1e5a3de93bc9f639253f262d8
Signed-off-by: Vijayendra Radhakrishna <vradhakrishna@mvista.com>
|
|
|
|
Move all test that work up so that we can commit a patch to danube branch in
which the seeds can go from 0 to 3, leaving 4 and 5 out of danube. This patch
is separated from the danube patch because I think it is easier to have the
same list in danube and master to avoid confusion
Change-Id: I99f6dad4f1878f776b02fc7eccfc51876057a0fb
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
get_floating_ips() is non-reusable function, with too much logic
into it. By using smaller functions, we can compose the same
functionality and build all tests upon a common set of utilities.
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: Ic7dba908fa6bb343c177fe1a68322d3803ed1707
Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
|
|
Renamed vxlan_firewall and vxlan_tool_stop to start_vxlan_tool and
stop_vxlan_tool respectively. Added some comments to explain their
behavior.
Cleaned both functions to use ".format()" string formatting, which
improves readability.
Modified behavior of start_vxlan_tool: now it does NOT block
traffic by default
Change-Id: I6754b020a474be1e9adf2d83e7c9f5053930b702
Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
|
|
Change-Id: I5305e7c6426e0a563499f0aeefd910ce03e5ceae
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
|
|
In the CI the availability zones are using a different domain. For
example instead of "node-1.domain.tld", we have
"node-1.opnfvericsson.com"
This patch takes the av zones dynamically from nova hypervisor-list.
Change-Id: Ifa26dbbb6b8d415ee30bb1216fc1150f65096a16
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
|
|
This is a candidate replacement for the wait_for_classification_rules
function. The functionality is as follows:
- Query ODL operational datastore for the RSPs for which we should
expect to see flows in the computes (RSPs for which ACL rules exist)
and associate them with a list of vnfs. Also get the tp_dst from the
ACL match rules.
- Use the known topology to associate the RSPs to a set of compute nodes
(at this point we know that ODL promised us that we should see
classification rules for a particular rsp in the nodes C1, C2 ...)
- Gather the installed flows in table=11 in each compute
- Verify that the RSPs are installed as promised by ODL for the correct
tp_dst
Change-Id: I79747ad1df3eb0c67f783167601b5ea99fb43f16
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
|
|
This reverts commit cb311aad19c732fa006b50795ab66687e01b95a6.
Change-Id: I123c30acd765df692483eb5f91fecd1478219201
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
|
|
Change-Id: I8e54892450e903aa51f888229940e6371a0710f7
Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
|
|
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>
|
|
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>
|