Age | Commit message (Collapse) | Author | Files | Lines |
|
Change-Id: I492e8fa3336b1e267aaa22f3ebdca8f5de4d1362
Signed-off-by: Sofia Wallin <sofia.wallin@est.tech>
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>
|
|
* Update docs/submodules/sfc from branch 'stable/gambia'
- Avoid race conditions
Sometimes the port pair group gets created before port pair was
committed in openstack. Before creating the port pair group, we check
that the port pair is already listed by openstack
Change-Id: Icfd23521859fc5d4e6bb6745be80f79bf9e31a0c
Signed-off-by: Manuel Buil <mbuil@suse.com>
(cherry picked from commit 52e86db6e0a3343852c9aabba49dffe3d699d1a0)
|
|
* Update docs/submodules/sfc from branch 'stable/gambia'
- Update documentation
Change-Id: Ic7af81de3d93462f8ee626576cd4f007d5b95f83
Signed-off-by: Manuel Buil <mbuil@suse.com>
(cherry picked from commit 0eb8ab44ae9192f08026f659bc40cddb48703afb)
|
|
* Update docs/submodules/sfc from branch 'stable/gambia'
- Merge "Add a bit more data to the logging message" into stable/gambia
- Add a bit more data to the logging message
In case of failures, it is hard to understand for what vnf the port
pair was being created when things went wrong
Change-Id: I1cfa2553318d910535d45910b2a46d75b2f7926a
Signed-off-by: Manuel Buil <mbuil@suse.com>
(cherry picked from commit fe7d96a708927b299ddb939f854b09b3452c5304)
|
|
* Update docs/submodules/sfc from branch 'stable/gambia'
- Adapt regex to new flows
ODL Fluorine changes the nsh syntax, so we should adapt it
Change-Id: I63537f9b880f14b40be2658d3a7ed2e315e86b11
Signed-off-by: Manuel Buil <mbuil@suse.com>
(cherry picked from commit 09cfe2f4b864c9566c0ffcdf2c61e7bfb5cf8454)
- Updated from global requirements
Change-Id: I383e6dbaae7e43a5501df6aa7145bad6daebc071
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Merge "Create new scenario: os-odl-sfc_osm"
- Create new scenario: os-odl-sfc_osm
This change creates os-odl-sfc_osm scenario.
- copy the base scenario, os-odl-sfc
- remove tacker and pike config
- add tasks, vars, defaults and templates from os-nosdn-osm role
https://gerrit.opnfv.org/gerrit/#/c/63507/
Verification of this patch has be done using a change in releng-xci
pointing to this commit/corresponding patch on Gerrit to verify it
works in order to help with the review.
Change-Id: I90148063c6ff9881ef188e97956adea0e7cb5a7d
Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
|
|
* Update docs/submodules/sfc from branch 'master'
- Cleanup our ansible role
Our role had some old comments and confusing names which don't make
sense anymore. This patches cleans it up
Change-Id: Ieb928aad2c1b0bfcfa94b300201e026e11a4c991
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Use changed_when instead of the when condition
The 'when' condition does not make sense and it was added in order to
avoid the ansible-lint error which requires all command or shell modules
to use a condition. This can be avoided with changed_when as explained:
http://willthames.github.io/2016/09/21/using-command-and-shell-in-ansible.html
Change-Id: I36316ea0848cff9763745af6dd8d8defcd993b70
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Remove all Pike config
Due to the lack of developers, we will only support one version of
openstack (master). Therefore, we remove the Pike support which is
anyway not the latest stable anymore
Change-Id: Ib2c81f952b0e35c61a3edeb9b4e389f5cd27e956
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Add post-provision playbook to pass ml2_conf.ini
Functest requires ODL variables to run the ODL healthcheck which is
mandatory for all ODL scenarios
Jenkins CI jobs will fail because it requires this patch to be finished:
https://gerrit.opnfv.org/gerrit/#/c/63175/
which requires this one to work (unfortunately, circular dependency)
Change-Id: I960e3346d840c5e30a2b56284fe109b8f99046e3
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Updated from global requirements
Change-Id: Id0eaf823a81196a4f8920576762d801cca0d93ea
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Add neutron_dnsmasq_force_metadata: True
When deploying with ODL, the static route towards the metadata server is
not set and the only alternative is getting the route pushed to the VM.
To do so, we must add the "force_metadata: True" option to the dnsmasq:
https://github.com/openstack/openstack-ansible-os_neutron/commit/a8c7e0b506a0c347a363dde3f60077aa0cab5641
Change-Id: I43f9cabacf6cc8b82a22f1f0eedb36b9f1f0c31f
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Add ceilometer to the scenario
Due to the n-odl dependency, ceilometer must be part of the scenario.
There is a chicken-egg problem with this patch. There is a XCI patch
which will not get a +1 from jenkins if this patch does not get merged.
However, this patch will not get a +1 from jenkins unless XCI patch
gets merged. This is XCI's patch:
https://gerrit.opnfv.org/gerrit/#/c/62419/
Change-Id: I145ac35dcb94d40f6e21a1a49ae5a1e754434a10
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Add new variables for ODL Fluorine
Change-Id: I80429ebcfd9e4a9c347b1b1865f4584b9bce1a66
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Refactor unit tests in 'test_openstack_utils.py'
JIRA: SFC-126
JIRA: SFC-129
Some of the unit tests I wrote earlier needs to be separated into
individual unit tests, which Mr Dimitrios has mentioned in one of
my previous commits. Unit tests are supposed to be independent &
should only test one test case at a time. This patch fixes them.
Change-Id: Ie4651c0eeede5fe729356664493708d8144ff84a
Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
|
|
* Update docs/submodules/sfc from branch 'master'
- Refactor unit tests in 'test_test_utils.py'
JIRA: SFC-126
JIRA: SFC-129
Some of the unit tests I wrote earlier needs to be separated into
individual unit tests, which Mr Dimitrios has mentioned in one of
my previous commits. Unit tests are supposed to be independent &
should only test one test case at a time. This patch fixes them.
Change-Id: I1ccecf049645fe4996d8b4fa5425d485727e1ae0
Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
|
|
* Update docs/submodules/sfc from branch 'master'
- Merge "Refactor unit tests in 'test_odl_utils.py'"
- Refactor unit tests in 'test_odl_utils.py'
JIRA: SFC-126
JIRA: SFC-129
Some of the unit tests I wrote earlier needs to be separated into
individual unit tests, which Mr Dimitrios has mentioned in one of
my previous commits. Unit tests are supposed to be independent &
should only test one test case at a time. This patch fixes them.
Change-Id: Iaf20e5868491f0627882929a953f3c4a3273f626
Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
|
|
* Update docs/submodules/sfc from branch 'master'
- Merge "Fix links to remove references to submodules"
- Fix links to remove references to submodules
All references to submodules must be removed for docs to
work going forward.
Can be tested once your project has a local tox file
Change-Id: I8953e7bece19cb5ab733ce15733e4668ca8d6bc1
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>
|
|
* Update docs/submodules/sfc from branch 'master'
- Use block=True to avoid race conditions
If we do things asynchronously, the instance might not be ready and the
information we get is not complete (e.g. port missing). Using block=True
fixes the problem
Change-Id: I9f90f52656553b8da8697477250ed631be884196
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Update entry_point to conform with Xtesting
Functest is being updated to Xtesting 0.60 which leverages on
stevedore [1].
[1] https://docs.openstack.org/stevedore/latest/
Change-Id: I09382b458929fabd21b5351b9d84b0d73fe2657e
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Remove port security from vnfs when no-mano
If the ports have security enabled, the traffic is dropped due to the
anti-spoofing
Change-Id: I6fbc63a3d959d9d55dfd1d64e12e1d0e7ec48bcb
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Decouple tacker from tests
Support n-sfc too and abstract a bit the MANO layer so that other MANO
components can be inserted into the test
Change-Id: I3fb59fbf40b4207bf1721092cd8ff0559e1d9d90
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Merge "Bug fix: Bad usage of positional argument"
- Bug fix: Bad usage of positional argument
av_zone is passed as a positional argument, however, the create_instance
method does not define av_zone as positional:
https://github.com/opnfv/sfc/blob/master/sfc/lib/openstack_utils.py#L136-L137
so we should not pass it as a positional argument
Change-Id: I035012ef3ffce3318e0bece712cb6bdad8e8cdd0
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Merge "Improve readability of the tests"
- Improve readability of the tests
The method name "create_custom_av" does not clearly say what it is
doing. Changing it to "create_vnf" clarifies the code better
Change-Id: Ic3f874bdb22891d348e5609130b8e4f0f3f8c3e8
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Bug fix: Deletion testcase only uses one VNF
We are only using one vnf but there are two defined in the config for
that testcase
Change-Id: Ifa86c2e48cb60edcd1b173eee14715c3787a9889
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Flake8 fixes
When running flake8 I am getting several errors. This patch fixes them
Change-Id: I4d32b4b214ca61e98da6fc28a4de44201c4c1cdc
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Change the test result comparison in 'test_odl_utils.py' #L103
When 'get_active_rsps_on_ports' is executed, it returns a list of
unordered dictionaries, sorting the result to verify the return
value would fix the test assertion.
Change-Id: Id39dc8d15f9efb0ef76168f823d7d52333474e16
Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
|
|
* Update docs/submodules/sfc from branch 'master'
- Added unit tests for 'odl_utils.py' module
JIRA: SFC-123
Change-Id: I49b8dae157816ef04a45dea188cc7dbe3aa72d4a
Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
|
|
* Update docs/submodules/sfc from branch 'master'
- Added unit tests for 'openstack_utils.py' module
JIRA: SFC-123
Change-Id: Ia41ec596d1c2247b2c86d88672a07e50021b3bc0
Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
|
|
* Update docs/submodules/sfc from branch 'master'
- Fix UnboundLocalError in 'odl_utils.py'#L420
When an exception is raised from find_compute in #L411, the compute
variable is not created and it raises an error < UnboundLocalError:
local variable 'compute' referenced before assignment > at #L420.
Change-Id: I48cff435dc3a03b30b7a8d24aa605611e30938de
Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
|
|
* Update docs/submodules/sfc from branch 'master'
- formatting the log message in odl_utils.py #413
Change-Id: I32f826fbb2c0a339264b8b316666ddf7f241570c
Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
|
|
* Update docs/submodules/sfc from branch 'master'
- Added parentheses in 'openstack_utils.py' line#476
Parentheses were added around the arguments in line#476
for proper formatting the Exception message in line#475
Change-Id: I3b2233aea10ff5afd23da00a837b976456274a51
Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
|
|
* Update docs/submodules/sfc from branch 'master'
- Merge "Fix Update README.tests"
- Fix Update README.tests
When looking at the different testcases in the functest directory
TEST SYMMETRIC is outdated and TEST DELETION is missing in the README.
This patch is an update to README.tests for the latest changes that
have been done the to those two testcases.
Change-Id: I54fceac85f45212ed45efca1a6f30d4ba092e0c1
Signed-off-by: spiritus72 <billmorrissonjr@gmail.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Merge "Added unit tests for 'test_utils.py' module"
- Added unit tests for 'test_utils.py' module
Jira: SFC-123
Change-Id: Ib8e3e748f3992d9030db1308a728a2576e7957ed
Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
|
|
* Update docs/submodules/sfc from branch 'master'
- Update release notes for Fraser 6.2
Change-Id: I0d56aa4d5f494a25db92e61415a24b4257c71a04
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Merge "Move the yamllint check to tox"
- Move the yamllint check to tox
Until now we were executing the yamllint check through shell
commands contained to a builder in the releng repo [0].
This patch moves that check to the tox framework.
Additional changes should be done to the sfc jobs in the
releng repo.
[0]: https://github.com/opnfv/releng/blob/master/jjb/sfc/sfc-project-jobs.yaml#L73
Change-Id: Ie42113087a8c1494593d5510cdf367abe99a7f5b
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Remove side_effect method in 'test_cleanup.py' module
The side_effect method under test_delete_openstack_objects_exception
method in 'test_cleanup.py' module is not needed. In fact, it won't
be called at all. It should have been detected in [pylint] W0612.
So when we run the nosetest, side_effect from Mock instances of
mock_creator_obj_one and mock_creator_obj_two were called.
Change-Id: Ibad6c8ebe14c320ef31c2133635ecd45fd011835
Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
|
|
* Update docs/submodules/sfc from branch 'master'
- Create unit tests for 'cleanup.py' module
This is the first attempt to add unit testing capabilities
to OPNFV/SFC project.
Also we are following the contraints.txt logic which
helps in the direction of simulating a test env which
will be as much as close to the real env that the SFC
testcases are running.
Jira: SFC-127
Jira: SFC-129
Change-Id: I938d28f9f85614cdefa163949c4ec9f06f5d27b4
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- [Bug fix] Fix broken urls
Change-Id: Ide4d6f8a2a54f70522a01a7467bddcd69daf8531
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Fix "Remove the annoying WARNINGS when using XCI"
When executing the SFC tests in XCI, we constantly get a whole bunch of
annoying "InsecureRequestWarning" messages which does not allow to
follow what is going on in the test.
This patch filters all those errors leaving the output of the functest
results clean.
Change-Id: I3bd9519676222d00ee4a2c8d47084737cd9768aa
Signed-off-by: Bill Morrisson <billmorrissonjr@gmail.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Bug fix: pass variables in the class constructor
The Testcase class requires parameters in the constructor. We should pass
them or otherwise we get an error:
https://github.com/opnfv/functest-xtesting/blob/master/xtesting/core/testcase.py#L48
ERROR:
TypeError: "__init__() got an unexpected keyword argument 'project_name'"
Change-Id: I6f4b62b7ff201bf6878004e3fb452778c6b55f0e
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Remove the pike specific files
We are already preparing for Gambia release and that means Openstack
Queens for the traditional release track. Therefore, we can remove
the specific configs for Openstack Pike
Change-Id: Ib49a53e73ae1020aaead8a897dff8733147d87ac
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Rethink the clean up process
Using the '--nocleanup' parameter as part of command,
the cleanup mechanism could be disable.
Otherwise, the cleanup method is executed even if a exception has been raised.
Change-Id: I9638369900f599793393feb5184c30b74f1b7c7e
Signed-off-by: Panagiotis Karalis <pkaralis@intracom-telecom.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Merge "Enable local documentation builds"
- Enable local documentation builds
Adds a local conf.py and conf.yaml (using lfdocs-conf), to enable local
docs build.
Change-Id: Iaab750983f02476d9bccb6c9bcf2b38544d54d8c
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>
|
|
* Update docs/submodules/sfc from branch 'master'
- Provide support for VNFs
Currently it is not possible to define different images for client/server
and VNFs. This functionality is important to have in order to test
different VNFs
Change-Id: Ibc3b3f1102d52529701748d37f795cdf05269410
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Provide support for local images
There might be cases where we want to test local images and the image
location in config.yml is locally
Change-Id: I29f115de5c80db187ecc2b260c386b377fafd6d2
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Fix the logging to be compatible with functest
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>
|
|
* Update docs/submodules/sfc from branch 'master'
- Update the release notes
Change-Id: I31506971cf57d761f966841029e3a878d2a0d284
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Fix symmetric test case result summary
Correctly print in test summary whether we are testing HTTP uplink or
HTTP uplink.
Change-Id: Ib8d4cc4a5e794384720d7023032c5340039e2ff1
Signed-off-by: Jaime Caamaño Ruiz <jcaamano@suse.com>
|
|
* Update docs/submodules/sfc from branch 'master'
- Merge "Revert "Add iteritems() when parsing dict in Python 2.x""
- Revert "Add iteritems() when parsing dict in Python 2.x"
This reverts commit e29fe40e0c98d6c3b619de7f99f0a8fb9e4a9d7b.
acl_types_names is not a dictionary but a list of tuples:
(Pdb) pp acl_types_names
[(u'ietf-access-control-list:ipv4-acl',
u'04a5ce7c-6aac-4f26-b3f9-a19749d814f3'),
(u'ietf-access-control-list:ipv4-acl',
u'75737a58-c894-46d9-9319-8d2b4785b0f3'),
(u'ietf-access-control-list:ipv4-acl',
u'bb422318-c3a3-4356-a67c-f44e8874edf6')]
(Pdb) type(acl_types_names)
<type 'list'>
(Pdb) type(acl_types_names[0])
<type 'tuple'>
Therefore, this patch introduces an error
Change-Id: I7f7e69fb1855c7915b2c17d80a5b7ea76aaa8694
Signed-off-by: Manuel Buil <mbuil@suse.com>
|