Age | Commit message (Collapse) | Author | Files | Lines |
|
Depends-On: https://gerrit.opnfv.org/gerrit/#/c/59177/
Change-Id: I0d56aa4d5f494a25db92e61415a24b4257c71a04
Signed-off-by: Manuel Buil <mbuil@suse.com>
(cherry picked from commit a52c05a07b2a01b8c911537d29c995fb42f66972)
|
|
In order to collect logs from SFC test scenarios running through
functest framework (mainly for troubleshooting purposes),
the logging mechanism has been updated accordingly.
JIRA: SFC-125
Change-Id: Ide7c32519afa6ece299d0ac1dcb168609a2e3945
Signed-off-by: Panagiotis Karalis <pkaralis@intracom-telecom.com>
|
|
Change-Id: I31506971cf57d761f966841029e3a878d2a0d284
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
|
|
Apex deployments are currently failing and it could be because it uses the
wrong config file. We are currently not logging what file it is fetched and
that's what this patch tries to fix
Change-Id: I441beee9d3ad5d918ed4b8dc0164456b39e53b56
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
vnffgd and vnfd are being parsed in different ways, we should parse them
in the same way. We could do it directly with safe_load, however, the returned
object is not easy to read. If we use read() instead, the resturned object is
easy to read/log. Therefore, this patch combines both options
Change-Id: I3416933801569a6f57962cd9906726229923136d
Signed-off-by: Manuel Buil <mbuil@suse.com>
(cherry picked from commit e122e00cb02adc65870cbb05f394dfc9a0cf55b2)
|
|
It is hard to understand what is going on in the SFC executions
Change-Id: I5c9688e68b03c59d6d39290faee8683513a60440
Signed-off-by: Manuel Buil <mbuil@suse.com>
(cherry picked from commit 5f8eb6bf01ca3ef240cfcdee297327b21b33f35a)
|
|
JIRA: SFC-125
If we don't use the opnfv.logger from functest, logs are not printed when
exeucting tests through the functest framework
Change-Id: I6d58ff3977fd94221720eea978e0efde07a576a2
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
Selecting another tag than the official ones (opnfv-xxx) breaks all
Functest builds because pbr requirements are unmet [1].
[1] https://build.opnfv.org/ci/job/functest-features-docker-build-amd64-master/219/console
Change-Id: I8a6a603c9f4e976c8bcdcce865850b46f6e69453
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
Change-Id: Ie53eb812c9111a2f75342470b10ef3376e43b5fb
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
One patch was merged without rebasing and it reverted some accepted changes
Change-Id: Ia64cef60df1ceb72d8789d122cf1bbb820819aa2
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
Change-Id: Ia2112794ef542ff91af1a547d814a0f0c737dd54
Signed-off-by: Manuel Buil <mbuil@suse.com>
(cherry picked from commit 10065e1a4fc60ab31ac166745a11cb08b47ac8db)
|
|
We can read the openstack version from the env variables and then use
a different config. The reason we use this is because openstack API
changed from Pike to Queens and Pike is the official version for
OPNFV Fraser release
Change-Id: I8a94e71224d2ba0aa75a6438f3d7b03929acbbef
Signed-off-by: Manuel Buil <mbuil@suse.com>
(cherry picked from commit a459dee88ba00d6f8283232ce2159594ae23d22a)
|
|
|
|
Update the symmetric chain test case aligning it with updates available
from ODL Oxygen:
- ODL networking-sfc implementation now supports symmetric paths. It is
no longer needed to configure the ACL or chain directly through the
ODL rest api. It can now be fully relied on tacker for this.
- ODL networking-sfc implementation now supports different ingress and
egress ports for port pairs. The test case exercises this by setting
up a VNF with two distinct ports. Note that this requires an updated
VNF image that supports two nics and bundles a vxlan_tool capable of
using them.
Change-Id: Ie711abb93109943dcbf699ece7b2b570500a3711
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
(cherry picked from commit 8e85daee26c523cd181f5884858646b7589c50af)
|
|
Starting Oxygen, ODL networking-sfc translator supports symmetric
paths and maps flow classifiers to ACLs with SFP redirect action
instead of RSP redirect action. A SFP may have two RSPs associated
if the path is symmetric, one otherwise. One of the two RSPs will
be flagged as the reverse path and the other is the forward path.
Traffic from the logical-source-port will be classified to the forward
path, while traffic from the logical-destination-port will be
classified to the reverse path.
This patch updates the ACL flow check logic to account for the above.
Change-Id: I2d8b9e6c2a1033d469668db29b9c18f525e89370
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
(cherry picked from commit f66a8164cfb2ef172ee3ea274992b9c07306f08c)
|
|
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>
|
|
Change-Id: I35960abdab146ab2f80a1ad160489ee4877d42e4
Signed-off-by: Fatih Degirmenci <fdegir@gmail.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>
|
|
|
|
|
|
JIRA: SFC-122
Add support for Compass installer in SFC repo which will make
Compass SFC scenarios be verified by SFC testcase daily.
Change-Id: I8129ebe837d54cdcd163158990fccf40451b98ad
Signed-off-by: Yifei Xue <xueyifei@huawei.com>
|
|
Removing it was a mistake because without it is harder to debug (e.g. using
break points does not work when triggering tests from functest)
Change-Id: I3a958da0add0fb57946da333f75fb526c2fe6c33
Signed-off-by: Manuel Buil <mbuil@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>
|
|
|
|
|
|
|
|
It is no longer called via BashFeature.
Change-Id: I8ae6b24d5e02f269056c1d4615b436c1a2a3a903
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.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>
|
|
|
|
Change-Id: I0b0aabffb2a5995364051fa34d218b2ebc649d4e
Signed-off-by: Manuel Buil <mbuil@suse.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>
|
|
|
|
We are now aligned with master so we don't need a-r-r file anymore or the
OpenStack versions file
Change-Id: Ida0b8f0414870d5fa3fecdc79564def345187b2e
Signed-off-by: Manuel Buil <mbuil@suse.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>
|
|
|
|
Change-Id: Ic1dfe50cbf8f1567a055aa8780e671333f67e880
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.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>
|
|
The following tacker patch:
https://review.openstack.org/#/c/532203/
changed the syntax of the vnffgd templates
Change-Id: Ifbf788b02a95c8f45eb97230d3d2da117de158d6
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>
|