aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2019-04-19Updated from global requirementsstable/hunterCédric Ollivier1-1/+1
Change-Id: I0e3d26b44bd44a443eb61f59e99e4315ecdf264e Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2019-01-25Fix Two Chains Test and Enabled all Testcasesgvrangan9-35/+335
- Method added to support Port Chain update - Used the new method to modify the test as follows - Create two Port Chains (one VNF per chain) - Block ssh in one vnf and http in the other - Test communication - Swap the flow classifiers in the chains so that ssh packets are sent to vnf where http is blocked and vice versa - Fix extracting odl username/password from ml2_conf - Checking flow classifiers are implemented - Fixed odl cleanup Change-Id: I1f0f3a3b829d6c73d1bb1a774ebf3484912b84b7 Signed-off-by: gvrangan <venkatrangang@hcl.com>
2019-01-23Update release notesManuel Buil1-4/+4
Change-Id: I480506dcbea94ddc4cd35ea0f0c042cc4b958a3e Signed-off-by: Manuel Buil <mbuil@suse.com>
2019-01-21Simplify deletion test and fix unittestManuel Buil4-24/+56
Removing the port groups when deleting the first chain simplifies things because we don't need port_groups to be a variable of the SfcCommonTestCase class There were also some missing unittest Change-Id: I6ae780bdc72116907a9e6037571c29f28a6ba334 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-11-29Merge "Migrate SNAPS to Openstack-SDK"Manuel Buil10-626/+942
2018-11-29Merge "Introduce the Barbican in XCI for SFC-HA scenario"Manuel Buil6-0/+44
2018-11-26Migrate SNAPS to Openstack-SDKNikos Mimigiannis10-626/+942
JIRA: SFC-137 This patch replaces SNAPS library with openstack_SDK library for TCs with mano component: no-mano Change-Id: I99b9a5f54a273f61ecff05a11616440d5e85e55b Signed-off-by: Nikos Mimigiannis <nmimi@intracom-telecom.com>
2018-11-14Fix bug in the symmetric test caseManuel Buil5-24/+28
The symmetric testcase requires the neutron port of the server to check that the classification rules were written in the flow tables. This patch adds that. Change-Id: I7ce930b102409f2772771a9c5b0e6de99bb293a0 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-11-13Correct the links to releng-xci and compassManuel Buil2-4/+4
Change-Id: I2285118eafc3059ffa7d1c7e1fba2e3981613c77 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-11-07Include "how to deploy scenario" sectionManuel Buil2-0/+26
According to the new documentation rules, we should include a section that describes how to deploy de scenario Change-Id: I7f5897e4e1af225ee91defafd65aa99e69684d93 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-11-06Introduce the Barbican in XCI for SFC-HA scenarioPanagiotis Karalis6-0/+44
The barbican OS component is introduced for SFC HA deployed scenario. The reason behind is that In HA scenarios we need Openstack Barbican to gather and store the fernet keys so Tacker can access them and be able to register new VIMs. deploy-scenario:os-odl-sfc installer-type:osa JIRA: SFC-131 Change-Id: Ifa4a8bab0547c18c9e2ad654b0d048aff161c56f Signed-off-by: Panagiotis Karalis <pkaralis@intracom-telecom.com>
2018-11-06Merge "Bug fix: Correct the ips in the config"Brady Johnson2-66/+66
2018-11-06Merge "Avoid race conditions"Brady Johnson2-0/+51
2018-11-06Avoid race conditionsManuel Buil2-0/+51
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>
2018-11-06Update documentationManuel Buil7-185/+46
Change-Id: Ic7af81de3d93462f8ee626576cd4f007d5b95f83 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-11-05Bug fix: Correct the ips in the configManuel Buil2-66/+66
When doing ha, controllers get the .11, .14 and .15 whereas computes get the .12 and .13. The current config was wrong deploy-scenario:os-odl-sfc installer-type:osa Change-Id: Icedfba9d8b557a926340d66618461d2ec19fe744 Signed-off-by: Manuel Buil <mbuil@suse.com>
2018-10-30Improve ansible tasks and fix license headersVenkata Harshavardhan Reddy Allu9-32/+32
This patch does two things: 1. improve the ansible tasks: - register Openstack as VIM - create osmrc file - install OSM (master) 2. fix license headers in unit tests installer-type:osa deploy-scenario:os-odl-sfc_osm Change-Id: I02297dae2b0108397541cedeb2f47183f1df99fc Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
2018-10-26Merge "Adapt regex to new flows"Brady Johnson1-5/+6
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-26Adapt regex to new flowsManuel Buil1-5/+6
ODL Fluorine changes the nsh syntax, so we should adapt it Change-Id: I63537f9b880f14b40be2658d3a7ed2e315e86b11 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>