summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2017-02-16Merge "Bug fix in sfc utils lib"Manuel Buil1-1/+1
2017-02-16Bug fix in sfc utils lib“Manuel Buil”1-1/+1
The command to add route to the VM was wrong Change-Id: If45425a5bdc8ba9d2ecb37dd9914628b6963ed28 Signed-off-by: “Manuel Buil” <mbuil@suse.com>
2017-02-16Merge "Pass availability zone as param on vnf create"Manuel Buil7-30/+34
2017-02-16Fix is_ssh_blocked and is_http_blockedJuan Vidal3-27/+23
This patch fixes JIRA SFC-79 Previously, those functions used regex matching against the command output. Now, the return code from netcat is used. It should be easier and more relieable. Change-Id: I639c64b3fe0a0e3c0b59f73e5615b8737e676849 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-15Pass availability zone as param on vnf createGeorge Paraskevopoulos7-30/+34
JIRA: SFC-63 JIRA: SFC-73 Instead of changing the vnfd-template file in runtime we can pass the availability zone as a parameter when the vnf is created This enables also the use of topology_shuffler for SFC-63 Change-Id: Ie650e3347e40aee030005b6a2272cae64968d303 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-02-15Modify run_cmd to return exit codes as well as stdout and stderrJuan Vidal3-17/+19
The old implementation of run_cmd returned different things in different situations, so the simplified new version broke some functions relying on boolean values instead of stdout. Change-Id: Ie94e529d2abfad171b9707df169925c6f5f47dfc Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-15Refactor utils using installer adaptersGeorge Paraskevopoulos5-108/+86
JIRA: SFC-65 JIRA: SFC-71 Use the new library in releng to talk to fuel. This allows for correct configuration of the compute hosts to resolve SFC-71 Also since https://gerrit.opnfv.org/gerrit/#/c/28281/ was merged it allows to run in multienv deployments so it resolves SFC-65 Change-Id: I5e6beb90d9c5108c21acb8d898f07cd2f4ae34f3 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-02-14Merge "Fix image name in vnfd templates and config.yaml"Manuel Buil5-7/+7
2017-02-14Merge "Add a warning to files changed by test cases"Manuel Buil2-0/+4
2017-02-14Merge "Extend the logging in the utils lib"Manuel Buil1-0/+2
2017-02-14Merge "Add some ignores to .gitignore"Manuel Buil1-0/+3
2017-02-14Extend the logging in the utils lib“Manuel Buil”1-0/+2
To learn about some of the bugs I saw, I added these lines to know what is going on. It was very useful to me and I thought that perhaps we can add it to the code to help in the future Change-Id: I070077749d2086489a536c9a7b7ecca4a05bb0a1 Signed-off-by: “Manuel Buil” <mbuil@suse.com>
2017-02-14Fix image name in vnfd templates and config.yamlGeorge Paraskevopoulos5-7/+7
Go back to ubuntu-based VM. Change-Id: I4a2b0da904144f77efba86e4084728102b202eab Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com> Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-13Add some ignores to .gitignoreJuan Vidal1-0/+3
- *pyc is bytecode python - egg-info folders are the result of installation of python packages - ovs-logs is used by tests to dump runtime information None of these files/folders should be added to the repository, so it is better to make git completely ignore them Change-Id: If26038984f82a92e7118f2accc0f93d2592391b7 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-13Add a warning to files changed by test casesJuan Vidal2-0/+4
Test case sfc_one_chain_two_service_functions_two_computes modifies the configuration files when it is run. Since this is not expected (testcases should not change its configuration), it is better to at least have a warning and a reminder to fix it. Change-Id: I00d374742e17e8eed26fb97a0287e1dbd5d42d6b Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-13Fix bugs when setting up computes“Manuel Buil”1-2/+3
JIRA: SFC-71 1 - dev br-int was missing when adding the route 2 - grep exits with 1 when nothing is found triggering an ERROR log even though there was not error Change-Id: Ib65d74d4b01021dcabb5af9068451b97c0170f75 Signed-off-by: “Manuel Buil” <mbuil@suse.com>
2017-02-10Migrate docs to the new Danube dir structureBrady Johnson23-0/+0
Change-Id: I6bdfd21add4d04f0fee988f83888c1edcc488951 Signed-off-by: Brady Johnson <brady.allen.johnson@ericsson.com>
2017-02-08Merge "Avoid looking for VMs from tenants different from the one running the ↵Manuel Buil1-1/+1
testcase"
2017-02-08Merge "Simplify run_cmd"Manuel Buil1-11/+5
2017-02-08Merge "Modify name and filename for VM image to use a sparse and compressed one"Manuel Buil1-3/+3
2017-02-08Merge "Modification of the ofctl_time counter"Manuel Buil1-2/+17
2017-02-07Modify name and filename for VM image to use a sparse and compressed oneJuan Vidal1-3/+3
Change-Id: I83bc0ed150e1e793e0e55d6bd787b13370c7a540 Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-07Simplify run_cmdGeorge Paraskevopoulos1-11/+5
sfc.lib.utils.run_cmd has a lot of unnecessary options which are never used. This patch simplifies this function and also fixes the returncode checking (commands have terminate with errors on returncode != 0 not on return code < 0) Also change the return value on error to None instead of False Change-Id: I5f1068ffd1bc6091e866a002b1afcb2507d30981 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-02-07Merge "Update delete script"Manuel Buil1-1/+2
2017-02-06Avoid looking for VMs from tenants different from the one running the testcaseJuan Vidal1-1/+1
- The testcase should only retrieve servers from its tenant, not from other ones. Trying to access other tenant's VMs makes failures in subsequent parts of the test. Change-Id: I0779f1e159573162db59272142bc8773dcbafa2f Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-06Remove unused variables and collapse some function callsJuan Vidal1-24/+9
- Argument parsing is not used in sfc_two_chains_SSH_and_HTTP test - Collapsed into just one line function calls that fit in 79 characters Change-Id: I630686d109e03dabda0c4e45f946a69d506ccebd Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2017-02-03Merge "Bug fix: prepare all computes not only one"Manuel Buil1-12/+11
2017-02-03Bug fix: prepare all computes not only one“Manuel Buil”1-12/+11
JIRA:SFC-71 Not all compute nodes were prepared with the right config to run SFC-NSH Change-Id: I3187020daecf0a90e5f7314dea3fefbf8fe470be Signed-off-by: “Manuel Buil” <mbuil@suse.com>
2017-02-03Modification of the ofctl_time counter“Manuel Buil”1-2/+17
If ODL is fast and changes the class. flow too quick, this function considers it as coming from an old deployment and thus never detects that the flows changed Change-Id: I0d6d1c27b325fb832d9ed6d4849f830eeea70185 Signed-off-by: “Manuel Buil” <mbuil@suse.com>
2017-02-03sfc-demo bash script for quick debug and testingVijayendra Radhakrishna1-0/+655
- Script is independent of functest, works using openstack commandline - Several commandline options are provided for flexibility Change-Id: Ibb852151a41babdc2cc435bd355bc91bc296fe3e Signed-off-by: Vijayendra Radhakrishna <vradhakrishna@mvista.com>
2017-02-02Update delete script“Manuel Buil”1-1/+2
It removes now the floating ips and does not shot the proble with tackerc Change-Id: Ifb4e640ed549e22a2d0c3d797fd2e4a384cc8b4d Signed-off-by: “Manuel Buil” <mbuil@suse.com>
2017-02-01Rename sfc.pyGeorge Paraskevopoulos2-8/+2
Rename sfc.py to sfc_two_chains_SSH_and_HTTP.py. Renaming the testcase is a functional requirement as well because otherwise it aliases sfc module inside sfc/sfc/tests/ directory Change-Id: Id505c22a8f278260aee8f76254b0e10f5ed7213a Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-01-17Merge "Fix wrong configuration lookup for functest_data folder"Manuel Buil1-1/+1
2017-01-17Merge "Get Fuel environment id from configuration"Manuel Buil3-1/+8
2017-01-12Sanitize sfc.lib.utilsGeorge Paraskevopoulos3-28/+13
- Remove unused (and unnecessary) wrapper function get_compute_nodes - Rename capture_err_logs to descriptive capture_ovs_logs - Rename compute_node argument in create_instance to av_zone - Rename capture_time_logs to more accurate wait_for_classification_rules Change-Id: Iab06eae0aa35b25ef7d5bf41a1e684f38b26430c Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-01-11Get Fuel environment id from configurationGeorge Paraskevopoulos3-1/+8
JIRA: SFC-65 Get the id of the fuel environment from config.yaml instead of an environmental variable Change-Id: I39e61200986b407ae19cc9f7c1384c34c8410c8b Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2017-01-09Fix wrong configuration lookup for functest_data folderJuan Vidal1-1/+1
Change-Id: I08e97a16f040a469ea588d39d65aeebef79081cf Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
2016-12-21Update directory pathjose.lausuch1-3/+3
JIRA: FUNCTEST-673 Change-Id: I91e2e8664112db5fe6474ece96e699f611a01de9 Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
2016-12-19Follow OpenStack guidelines on importsVijayendra Radhakrishna5-9/+12
- Imports are changed as per doc below although it's not a pep8/flake8 violation - http://docs.openstack.org/developer/hacking/#imports Change-Id: I89b1ec8f1eba9145f24507fb0f610e5f74c86ce8 Signed-off-by: Vijayendra Radhakrishna <vradhakrishna@mvista.com>
2016-12-16improper import of config class with recent changeVijayendra Radhakrishna1-1/+1
- Recent change of sfc as installable module moved config class - lib where import has not changed. Change-Id: Ic38d21b63fb8e6189d00f8ecf89ff776a9d7eb51 Signed-off-by: Vijayendra Radhakrishna <vradhakrishna@mvista.com>
2016-12-16Fix create_glance_image callGeorge Paraskevopoulos2-2/+2
Change https://gerrit.opnfv.org/gerrit/#/c/25725/ modified functest API create_glance_image requires the kw arg public to be one of ['public'|'private'] Change-Id: I7f963b0fb20e71c254a1c604021e1ce0b1f049df Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2016-12-16Fix sfc_test_dir pathGeorge Paraskevopoulos1-1/+2
Change-Id: I8caec09aa5c022baf64d48cab75eb347c2d78db5 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2016-12-15Make SFC installable as a python modulejose.lausuch26-127/+75
New directory structure: <root>/sfc <root>/sfc/lib/ <root>/sfc/tests <root>/sfc/tests/functest JIRA: SFC-60 After installing sfc, the imports would be: import sfc.tests.functest.x import sfc.lib.x Change-Id: Ib15172239aefdef65056d6598210a1b28a4b2eff Signed-off-by: jose.lausuch <jose.lausuch@ericsson.com>
2016-12-14Merge "Use timethis from functest utils"Manuel Buil1-16/+1
2016-12-14Merge "tacker_client_install.sh duplicate in odl-sfc dir"Manuel Buil1-43/+0
2016-12-14Use timethis from functest utilsGeorge Paraskevopoulos1-16/+1
timethis is hosted now in the functest code. Use that and delete the local function Change-Id: Icfdfa0609cd5ac93e6852e77e09bc4b901894f18 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2016-12-13Merge "Move flavor creations params to config file"Manuel Buil3-2/+15
2016-12-13Merge "Use SSHUtils from releng"Manuel Buil3-122/+2
2016-12-13Use SSHUtils from relengGeorge Paraskevopoulos3-122/+2
JIRA: SFC-61 Releng is installed as a pip module and their SSHUtils module under "opnfv.utils.SSHUtils" is compatible with ours so we should use the one from releng repo Change-Id: I2f03eaa78b3b1b19c37b38ae77bef8553a076e7b Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>
2016-12-13Use ovs_logger module from relengGeorge Paraskevopoulos5-146/+28
JIRA: SFC-61 Releng is now installed as a python module and ovs_utils has been transfered there in "opnfv.utils.ovs_logger" module, so we should use that instead. ofctl_time_counter is still hosted here under utils.py as it's not generic enough to be moved in releng Change-Id: Ie721c95f20a7f515f7f2eb45b7f8edd0460a6ea8 Signed-off-by: George Paraskevopoulos <geopar@intracom-telecom.com>