aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2018-10-26Add a bit more data to the logging messageManuel Buil2-8/+8
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>
2018-10-23Merge "Create new scenario: os-odl-sfc_osm"Manuel Buil18-0/+1288
2018-10-23Create new scenario: os-odl-sfc_osmVenkata Harshavardhan Reddy Allu18-0/+1288
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>
2018-10-22Cleanup our ansible roleManuel Buil6-15/+4
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>
2018-10-19Use changed_when instead of the when conditionManuel Buil1-2/+2
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>
2018-10-18Remove all Pike configManuel Buil18-1693/+0
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>
2018-10-18Add post-provision playbook to pass ml2_conf.iniManuel Buil1-0/+17
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>
2018-10-06Updated from global requirementsCédric Ollivier2-13/+13
Change-Id: Id0eaf823a81196a4f8920576762d801cca0d93ea Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2018-10-04Add neutron_dnsmasq_force_metadata: TrueManuel Buil6-0/+15
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>
2018-10-02Add ceilometer to the scenarioManuel Buil3-0/+19
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>
2018-09-18Add new variables for ODL FluorineManuel Buil3-3/+27
Change-Id: I80429ebcfd9e4a9c347b1b1865f4584b9bce1a66 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-09-18Refactor unit tests in 'test_openstack_utils.py'Venkata Harshavardhan Reddy Allu1-119/+227
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>
2018-09-17Refactor unit tests in 'test_test_utils.py'Venkata Harshavardhan Reddy Allu1-26/+41
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>
2018-09-17Merge "Refactor unit tests in 'test_odl_utils.py'"Manuel Buil1-57/+115
2018-09-15Merge "Fix links to remove references to submodules"Sam Hague3-20/+12
2018-09-14Fix links to remove references to submodulesAric Gardner3-20/+12
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>
2018-09-11Use block=True to avoid race conditionsManuel Buil1-1/+1
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>
2018-09-11Refactor unit tests in 'test_odl_utils.py'Venkata Harshavardhan Reddy Allu1-57/+115
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>
2018-09-03Update entry_point to conform with XtestingCédric Ollivier1-0/+2
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>
2018-09-03Remove port security from vnfs when no-manoManuel Buil3-12/+80
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>
2018-08-31Decouple tacker from testsManuel Buil15-236/+688
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>
2018-08-30Merge "Bug fix: Bad usage of positional argument"Manuel Buil1-2/+2
2018-08-30Merge "Improve readability of the tests"Manuel Buil5-11/+11
2018-08-29Bug fix: Deletion testcase only uses one VNFManuel Buil1-1/+0
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>
2018-08-29Improve readability of the testsManuel Buil5-11/+11
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>
2018-08-29Bug fix: Bad usage of positional argumentManuel Buil1-2/+2
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>
2018-08-29Flake8 fixesManuel Buil3-14/+3
When running flake8 I am getting several errors. This patch fixes them Change-Id: I4d32b4b214ca61e98da6fc28a4de44201c4c1cdc Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-08-29Change the test result comparison in 'test_odl_utils.py' #L103Venkata Harshavardhan Reddy Allu1-1/+1
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>
2018-08-22Added unit tests for 'odl_utils.py' moduleVenkata Harshavardhan Reddy Allu1-0/+775
JIRA: SFC-123 Change-Id: I49b8dae157816ef04a45dea188cc7dbe3aa72d4a Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
2018-08-13Added unit tests for 'openstack_utils.py' moduleVenkata Harshavardhan Reddy Allu1-0/+1715
JIRA: SFC-123 Change-Id: Ia41ec596d1c2247b2c86d88672a07e50021b3bc0 Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
2018-08-08Fix UnboundLocalError in 'odl_utils.py'#L420Venkata Harshavardhan Reddy Allu1-0/+1
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>
2018-08-05formatting the log message in odl_utils.py #413Venkata Harshavardhan Reddy Allu1-1/+1
Change-Id: I32f826fbb2c0a339264b8b316666ddf7f241570c Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
2018-07-19Added parentheses in 'openstack_utils.py' line#476Venkata Harshavardhan Reddy Allu1-1/+1
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>
2018-07-02Merge "Fix Update README.tests"Dimitrios Markou1-8/+29
2018-06-28Merge "Added unit tests for 'test_utils.py' module"Manuel Buil1-0/+528
2018-06-28Fix Update README.testsspiritus721-8/+29
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>
2018-06-27Update release notes for Fraser 6.2Manuel Buil1-3/+3
Change-Id: I0d56aa4d5f494a25db92e61415a24b4257c71a04 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-06-27Added unit tests for 'test_utils.py' moduleVenkata Harshavardhan Reddy Allu1-0/+528
Jira: SFC-123 Change-Id: Ib8e3e748f3992d9030db1308a728a2576e7957ed Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
2018-06-26Merge "Move the yamllint check to tox"Manuel Buil2-1/+10
2018-06-25Remove side_effect method in 'test_cleanup.py' moduleVenkata Harshavardhan Reddy Allu1-2/+0
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>
2018-06-21Move the yamllint check to toxDimitrios Markou2-1/+10
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>
2018-06-21Create unit tests for 'cleanup.py' moduleDimitrios Markou7-1/+394
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>
2018-06-18[Bug fix] Fix broken urlsDimitrios Markou3-4/+4
Change-Id: Ide4d6f8a2a54f70522a01a7467bddcd69daf8531 Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
2018-06-15Fix "Remove the annoying WARNINGS when using XCI"Bill Morrisson5-1/+25
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>
2018-06-12Bug fix: pass variables in the class constructorManuel Buil1-2/+2
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>
2018-06-12Remove the pike specific filesManuel Buil10-315/+21
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>
2018-06-12Rethink the clean up processPanagiotis Karalis6-18/+58
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>
2018-06-12Merge "Enable local documentation builds"Dimitrios Markou8-0/+77
2018-06-07Provide support for VNFsManuel Buil3-0/+26
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>
2018-06-06Enable local documentation buildsTrevor Bramwell8-0/+77
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>