summaryrefslogtreecommitdiff
path: root/tox.ini
AgeCommit message (Collapse)Author
2019-01-24Fix spurious pylint import errors for ddt and mockTom Barron
Set dependencies correctly for pylint job in tox.ini so it does not report spurious E041 import errors on mock and ddt modules. Also fix a reference to Cinder instead of Manila in the pylint script. Closes-Bug: #1813156 Change-Id: Ib160caae57e41a4d45483ff4faa599143e694abc Notes (review): Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Workflow+1: Goutham Pacha Ravi <gouthampravi@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 24 Jan 2019 20:35:32 +0000 Reviewed-on: https://review.openstack.org/632992 Project: openstack/manila Branch: refs/heads/master
2018-10-17Merge "change tox envlist from 3.5 to 3"Zuul
2018-10-11Simplify running pylintChuck Short
This commit does several things: - Setup and run pylint directly rather than running through a script. This allows the user to see what is happening while the user is running through pylint. - Allow the user to either run pylint on a particular changeset, or the entire manila tree. - Allow the user to run on a particular changeset. Using like HEAD~1, etc. - I disabled the tests that were reported by pylint. The thought here would be go through the failures and correct them. - Update pylint to 2.1.1. Change-Id: I398d1ba01a3fd7f1c86ad4065a7984d276f33383 Signed-off-by: Chuck Short <chucks@redhat.com> Notes (review): Code-Review+2: Tom Barron <tpb@dyncloud.net> Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Workflow+1: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 16 Oct 2018 18:03:43 +0000 Reviewed-on: https://review.openstack.org/609791 Project: openstack/manila Branch: refs/heads/master
2018-10-09Don't quote {posargs} in tox.iniNguyen Van Trung
Quotes around {posargs} cause the entire string to be combined into one arg that gets passed to stestr. This prevents passing multiple args (e.g. '--concurrency=16 some-regex') Change-Id: I1088cfbab31befbd24e56add806d6e86fe46e5eb Notes (review): Code-Review+2: Tom Barron <tpb@dyncloud.net> Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Workflow+1: Goutham Pacha Ravi <gouthampravi@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 10 Oct 2018 02:13:34 +0000 Reviewed-on: https://review.openstack.org/608825 Project: openstack/manila Branch: refs/heads/master
2018-10-04change tox envlist from 3.5 to 3Tom Barron
base python version is 3, not 3.5 and python 3.5 is not installed in some current distros so running 'tox' locally fails. Change-Id: Ie2449a433e07f4cafb55fe1db247ba9f854cef97 Notes (review): Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Code-Review+2: xing-yang <xingyang105@gmail.com> Workflow+1: xing-yang <xingyang105@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 17 Oct 2018 02:05:40 +0000 Reviewed-on: https://review.openstack.org/607959 Project: openstack/manila Branch: refs/heads/master
2018-10-04Remove run_tests.shTom Barron
It is deprecated in favor of tox. Closes-Bug: #1790470 Change-Id: If3b6a274dab0d035b9666b3b9876604cefbc2877 Notes (review): Code-Review+1: Maurice Schreiber <maurice.schreiber@sap.com> Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Code-Review+2: xing-yang <xingyang105@gmail.com> Workflow+1: xing-yang <xingyang105@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 04 Oct 2018 21:44:25 +0000 Reviewed-on: https://review.openstack.org/607724 Project: openstack/manila Branch: refs/heads/master
2018-06-08Merge "fix tox python3 overrides"Zuul
2018-06-06Default pylint to run using python37.0.0.0b2Sean McGinnis
Updates the pylint and lint jobs to run under python3 by changing the target python and updating the version of pylint used. Change-Id: Ie0ac158674fb22054b80864d33d9d7ecc82e3b36 Closes-bug: #1673824 Notes (review): Code-Review+2: Tom Barron <tpb@dyncloud.net> Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Code-Review+2: zhongjun <jun.zhongjun2@gmail.com> Workflow+1: zhongjun <jun.zhongjun2@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 07 Jun 2018 01:41:54 +0000 Reviewed-on: https://review.openstack.org/572991 Project: openstack/manila Branch: refs/heads/master
2018-06-06fix tox python3 overridesDoug Hellmann
We want to default to running all tox environments under python 3, so set the basepython value in each environment. We do not want to specify a minor version number, because we do not want to have to update the file every time we upgrade python. We do not want to set the override once in testenv, because that breaks the more specific versions used in default environments like py35 and py36. Change-Id: I59ac21cb0d34f0b6767c736d0419eb6b874efca1 Signed-off-by: Doug Hellmann <doug@doughellmann.com> Notes (review): Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Code-Review+1: Nguyen Hai <nguyentrihai93@gmail.com> Code-Review+1: zhongshengping <chdzsp@163.com> Code-Review+2: zhongjun <jun.zhongjun2@gmail.com> Workflow+1: zhongjun <jun.zhongjun2@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 08 Jun 2018 18:59:51 +0000 Reviewed-on: https://review.openstack.org/572965 Project: openstack/manila Branch: refs/heads/master
2018-05-10Switch from ostestr to stestrliuyamin
This change just updates the tox.ini file to call stestr directly instead of going through ostestr. Change-Id: Ibe957cda167b59ef44f5c7e9dfc415c227213565 Notes (review): Code-Review+2: Tom Barron <tpb@dyncloud.net> Code-Review+1: junbo.li <junbo85.li@gmail.com> Code-Review+1: huangshan <huangshan@fiberhome.com> Code-Review+2: zhongjun <jun.zhongjun2@gmail.com> Workflow+1: zhongjun <jun.zhongjun2@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 31 May 2018 02:59:06 +0000 Reviewed-on: https://review.openstack.org/567097 Project: openstack/manila Branch: refs/heads/master
2018-05-04Fix title overline too short when generate docsjunboli
when running tox -e docs locally, it failed because the title overline too short. Change-Id: I9b1d85f12ff559e032c465d7737e9c27efb459c3 Closes-bug: #1769073 Notes (review): Code-Review+2: Tom Barron <tpb@dyncloud.net> Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Workflow+1: Goutham Pacha Ravi <gouthampravi@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 04 May 2018 17:59:26 +0000 Reviewed-on: https://review.openstack.org/566232 Project: openstack/manila Branch: refs/heads/master
2018-04-07Run pep8/fast8 with python3Tom Barron
Change-Id: I056d7228834f5becf36ccbd4f7473e532cc5a431 Notes (review): Code-Review+2: zhongjun <jun.zhongjun2@gmail.com> Code-Review+2: xing-yang <xingyang105@gmail.com> Workflow+1: xing-yang <xingyang105@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 09 Apr 2018 09:31:14 +0000 Reviewed-on: https://review.openstack.org/559489 Project: openstack/manila Branch: refs/heads/master
2018-03-22add lower-constraints jobDoug Hellmann
Create a tox environment for running the unit tests against the lower bounds of the dependencies. Create a lower-constraints.txt to be used to enforce the lower bounds in those tests. Add openstack-tox-lower-constraints job to the zuul configuration. See http://lists.openstack.org/pipermail/openstack-dev/2018-March/128352.html for more details. Change-Id: I5f9a218429c1efc1fddae1945aef7110c4e031ee Depends-On: https://review.openstack.org/555034 Signed-off-by: Doug Hellmann <doug@doughellmann.com> Notes (review): Code-Review+2: Tom Barron <tpb@dyncloud.net> Code-Review+1: Huifeng Qin <huifeng_qin@163.com> Code-Review+1: Victoria Martinez de la Cruz <victoria@redhat.com> Code-Review+2: xing-yang <xingyang105@gmail.com> Workflow+1: xing-yang <xingyang105@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Sun, 25 Mar 2018 04:30:40 +0000 Reviewed-on: https://review.openstack.org/555760 Project: openstack/manila Branch: refs/heads/master
2018-01-17Follow the new PTI for document buildjunboli
For compliance with the Project Testing Interface as described in [1]: [1] https://governance.openstack.org/tc/reference/project-testing-interface.html For more detials information, please refer to [2]: [2] http://lists.openstack.org/pipermail/openstack-dev/2017-December/125710.html Change-Id: I1fe5352e8e4a7703bd9908e7221f04110163d6fb Notes (review): Code-Review+2: Ben Swartzlander <ben@swartzlander.org> Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Code-Review+2: zhongjun <jun.zhongjun2@gmail.com> Workflow+1: zhongjun <jun.zhongjun2@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 18 Jan 2018 08:28:40 +0000 Reviewed-on: https://review.openstack.org/530866 Project: openstack/manila Branch: refs/heads/master
2018-01-14Use stestr for coveragelijunbo
Change [1] restored .testr.conf since it was needed for the manila coverage job. Modify script used by the coverage job to invoke stestr rather than setup.py with testr so that this job no longer requires .testr.conf so remove the dotfile and update .gitignore. TrivialFix [1] I451e383ed654d00a5a017cb28521c47566911f33 Change-Id: Ibf549220303395744380f79993fbfcf04b2ed5b7 Co-Authored-By: Tom Barron <tpb@dyncloud.net> Notes (review): Code-Review+1: Tom Barron <tpb@dyncloud.net> Code-Review+2: xing-yang <xyang@alum.mit.edu> Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Workflow+1: Goutham Pacha Ravi <gouthampravi@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 16 Jan 2018 09:26:15 +0000 Reviewed-on: https://review.openstack.org/533381 Project: openstack/manila Branch: refs/heads/master
2017-12-22Remove in-tree tempest pluginRaissa Sarmento
Remove in-tree tempest plugin in favor of newly created manila-tempest-plugin repository. Change-Id: I1fb124598f38067fee469df124ee684f748c6f57 Notes (review): Code-Review+2: Tom Barron <tpb@dyncloud.net> Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Workflow+1: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 22 Dec 2017 13:40:44 +0000 Reviewed-on: https://review.openstack.org/512300 Project: openstack/manila Branch: refs/heads/master
2017-11-16[policy in code] Add support for share instance export location resourcezhongjun
This is the basic patch which consits of the framework code for default policy in code feature as well as share instance export location resource. Partial-Implements: blueprint policy-in-code Change-Id: Iedde7a4a674a60e760b47d5eb2973f42d79226d8 Notes (review): Code-Review+2: Ben Swartzlander <ben@swartzlander.org> Code-Review+2: Tom Barron <tpb@dyncloud.net> Workflow+1: Tom Barron <tpb@dyncloud.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 22 Nov 2017 20:29:07 +0000 Reviewed-on: https://review.openstack.org/507054 Project: openstack/manila Branch: refs/heads/master
2017-09-13[install-guide] remove install-guide docjunboli
1. manila/install-guide is completely the same doc as the one in the directory of manila/doc/source/install, this change is to remove manila/install-guide because I think this is unnecessary and also let us maintenance a copy of install-guide doc. 2. remove install-guide in tox.ini file, cause build doc will generate install-guide doc. Change-Id: Ife2d2b366f3f8b2bff8f9dda0f972858e1505a21 Notes (review): Code-Review+2: Tom Barron <tpb@dyncloud.net> Code-Review+1: zhongjun <jun.zhongjun2@gmail.com> Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Workflow+1: Goutham Pacha Ravi <gouthampravi@gmail.com> Code-Review+1: ShangXiao <shangxiaobj@inspur.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 14 Sep 2017 05:07:10 +0000 Reviewed-on: https://review.openstack.org/503541 Project: openstack/manila Branch: refs/heads/master
2017-08-03Enable some off-by-default checksjunboli
Some of the available checks are disabled by default, like: [H106] Don't put vim configuration in source files [H203] Use assertIs(Not)None to check for None [H904] Use ',' instead of '%', String interpolation should be delayed to be handled by the logging code, rather than being done at the point of the logging call. Change-Id: Ie985fcf78997a86d41e40eacbb4a5ace8592a348 Notes (review): Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Workflow+1: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 03 Aug 2017 13:21:42 +0000 Reviewed-on: https://review.openstack.org/484707 Project: openstack/manila Branch: refs/heads/master
2017-06-01Clean releasenotes and install-guide build dirTom Barron
to ensure artefacts from previous runs are not left over when running these document builds locally. TrivialFix Change-Id: I0062c1bf94490ed4e5d0eb3675226b144f7124c1 Notes (review): Code-Review+1: zhongjun <jun.zhongjun2@gmail.com> Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Workflow+1: Goutham Pacha Ravi <gouthampravi@gmail.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Sun, 18 Jun 2017 09:25:14 +0000 Reviewed-on: https://review.openstack.org/470019 Project: openstack/manila Branch: refs/heads/master
2017-04-13Add comment explaining ignore D001 for doc8Tom Barron
because we allow lines in excess of 79 characters in our doc source. Change-Id: I44fc982386f624fe44a45e97ef791a5e41ee15fd Notes (review): Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Workflow+1: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 13 Apr 2017 12:31:08 +0000 Reviewed-on: https://review.openstack.org/456553 Project: openstack/manila Branch: refs/heads/master
2017-03-20set basepython for pylint tox envTom Barron
to python2.7 so that running locally will succeed and the results will match those for the gate pylint job. Partial-Bug: #1673824 Change-Id: I6c45a32ba8e77c35403357b0100379bb536a36b3 Notes (review): Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Workflow+1: Goutham Pacha Ravi <gouthampravi@gmail.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Mon, 20 Mar 2017 20:26:22 +0000 Reviewed-on: https://review.openstack.org/447049 Project: openstack/manila Branch: refs/heads/master
2017-03-03Unblock gate failure on docs buildTom Barron
Our form of todo extension does not work with sphinx 1.5.2. Remove it to unblock gate: we can add todos back in later with more current methods. Also clean up some dangling cross-references in the doc. Co-Authored-By: Valeriy Ponomaryov <vponomaryov@mirantis.com> Change-Id: If74b9e32ad2b5d8d70da60895d85588ff993797e Notes (review): Code-Review+2: Ben Swartzlander <ben@swartzlander.org> Code-Review+2: xing-yang <xing.yang@emc.com> Workflow+1: xing-yang <xing.yang@emc.com> Code-Review+2: Thomas Bechtold <tbechtold@suse.com> Code-Review+2: Mark Sturdevant <mark.sturdevant@ibm.com> Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Fri, 03 Mar 2017 18:09:42 +0000 Reviewed-on: https://review.openstack.org/441156 Project: openstack/manila Branch: refs/heads/master
2017-02-15doc: verify all rst filesDanny Al-Gaaf
Make use of doc8 to verify all rst files which are not autogenerated for errors and fail if there are any issues found. The doc8 checks are now part of the tox 'docs' environment and ran automatically. Checks can also be called direcly via 'tox -e docs'. Fix all issues found by doc8. Closes-Bug: #1664841 Change-Id: I9215524d35646de7485504e4c5ff86fd91a1d09f Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de> Notes (review): Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Workflow+1: Goutham Pacha Ravi <gouthampravi@gmail.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 16 Feb 2017 21:34:19 +0000 Reviewed-on: https://review.openstack.org/434086 Project: openstack/manila Branch: refs/heads/master
2017-02-07Remove a py34 environment from toxCao Xuan Hoang
The gating on python 3.4 is restricted to <= Mitaka. This is due to the change from Ubuntu Trusty to Xenial, where only python3.5 is available. There is no need to continue to keep these settings. Change-Id: Id8a9dda9ea25b47afc35e0605f4df2f3f22c5f38 Notes (review): Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Code-Review+2: Mark Sturdevant <mark.sturdevant@ibm.com> Workflow+1: Mark Sturdevant <mark.sturdevant@ibm.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Wed, 08 Feb 2017 21:05:11 +0000 Reviewed-on: https://review.openstack.org/430082 Project: openstack/manila Branch: refs/heads/master
2017-01-05Support python 3.5 in toxTom Barron
Our current tox configuration provides no support for python 3.5 even though it is the version that ships with current ubuntu (xenial) and Fedora (25) systems, and even though in gate the python3 jobs run on xenial with python 3.5. Add py35 env to tox.ini alongside py34, py27, and pep8. TrivialFix Change-Id: Ib32069f9185c5233c8f96164781ec66c56db33bd Notes (review): Code-Review+2: Goutham Pacha Ravi <gouthampravi@gmail.com> Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Workflow+1: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Fri, 06 Jan 2017 11:02:27 +0000 Reviewed-on: https://review.openstack.org/417245 Project: openstack/manila Branch: refs/heads/master
2016-09-28[TrivialFix] Correct file mode settingsxiaozhuangqing
Drop the undesired executable bit for the following files: 1. manila_tempest_tests/services/share/v2/json/shares_client.py 2. manila_tempest_tests/tests/api/test_share_networks.py Also add pep8 check for new executable files under manila_tempest_tests. Change-Id: Iaa509fdf4e0ebeb60d71b053f62459992c938468 Notes (review): Code-Review+1: Yingzhe Zeng <zengyingzhe@huawei.com> Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+2: Tom Barron <tpb@dyncloud.net> Workflow+1: Tom Barron <tpb@dyncloud.net> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Wed, 28 Sep 2016 12:13:09 +0000 Reviewed-on: https://review.openstack.org/375958 Project: openstack/manila Branch: refs/heads/master
2016-09-16delete python bytecode including pyo before every test runLucky samadhiya
Change-Id: Ib1cee0ee3d482b23bbaa19956a10bb2e8fc7d559 Notes (review): Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+1: zhongjun <jun.zhongjun2@gmail.com> Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Workflow+1: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 20 Sep 2016 15:12:23 +0000 Reviewed-on: https://review.openstack.org/371357 Project: openstack/manila Branch: refs/heads/master
2016-08-16Merge "In-tree Install Guide"Jenkins
2016-08-14Tox Upper Constraints - strip out reinstalls for remaining jobsGoutham Pacha Ravi
releasenotes and venv tox jobs had redefined the install_command to reinstall any packages from requirements/test-requirements files. Since Infra started supporting constrained requirements for these jobs as well, we can drop the reinstall command. Ref: http://lists.openstack.org/pipermail/openstack-dev/2016-August/101474.html Change-Id: Ib4b8c6cef22a9911030c2e3e3f5281e7e22575fc Notes (review): Code-Review+2: Tom Barron <tpb@dyncloud.net> Code-Review+2: xing-yang <xing.yang@emc.com> Workflow+1: xing-yang <xing.yang@emc.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Mon, 15 Aug 2016 22:14:48 +0000 Reviewed-on: https://review.openstack.org/355255 Project: openstack/manila Branch: refs/heads/master
2016-08-12In-tree Install GuideGoutham Pacha Ravi
The OpenStack install guide team has recommended that projects that are not part of the starter-kit:compute [1] (and a few other projects like Cinder and Horizon) maintain their install guide in-tree. [2] [1] http://governance.openstack.org/reference/tags/starter-kit_compute.html [2] http://git.openstack.org/cgit/openstack/docs-specs/tree/specs/newton/project-specific-installguides.rst Change-Id: I2b49fcfd99b3be40bb1ae7d7b8348abdb5b58b90 Co-Authored-By: Denis Cavalcante <dencaval@gmail.com> Implements: blueprint manila-in-tree-install-guide Partially-implements: blueprint projectspecificinstallguides Notes (review): Code-Review+2: Tom Barron <tpb@dyncloud.net> Code-Review+2: Thomas Bechtold <tbechtold@suse.com> Workflow+1: Thomas Bechtold <tbechtold@suse.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 16 Aug 2016 08:32:34 +0000 Reviewed-on: https://review.openstack.org/317152 Project: openstack/manila Branch: refs/heads/master
2016-07-25Migrate API reference into treeDaniel Gonzalez
The API Documentation team has requires [1] that projects maintain their api-reference in-tree and build it to the developer.openstack.org website. This version of the API reference uses os-api-ref and compiles API documentation close to that of the OpenStack compute service (nova). Co-Authored-By: Goutham Pacha Ravi <gouthamr@netapp.com> [0] http://lists.openstack.org/pipermail/openstack-dev/2016-May/093765.html Implements: bp move-manila-api-reference-in-tree Change-Id: I67bb3354162d6e0e00fa2788edde864cc14920ee Notes (review): Code-Review+2: xing-yang <xing.yang@emc.com> Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+1: zhongjun <jun.zhongjun@huawei.com> Workflow+1: Valeriy Ponomaryov <vponomaryov@mirantis.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 26 Jul 2016 14:46:46 +0000 Reviewed-on: https://review.openstack.org/313874 Project: openstack/manila Branch: refs/heads/master
2016-07-17Use constraints for coverage jobAndreas Jaeger
OpenStack CI supports now constraints in the coverage job - as a first job to check that the recent changes for zuul-cloner work correctly. Use constraints for cover environment. Change-Id: I81a985a2049221a8a7ac1f20642137f22847973f Notes (review): Code-Review+1: Goutham Pacha Ravi <gouthamr@netapp.com> Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+2: Clinton Knight <cknight@netapp.com> Workflow+1: Clinton Knight <cknight@netapp.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Mon, 18 Jul 2016 14:08:24 +0000 Reviewed-on: https://review.openstack.org/343301 Project: openstack/manila Branch: refs/heads/master
2016-07-15Add tox job for db revision creationMarc Koderer
Trivial-Fix Change-Id: Ie87b84ddd546149ca5d9eaf2179c514071c75695 Notes (review): Code-Review+1: Goutham Pacha Ravi <gouthamr@netapp.com> Code-Review+1: Daniel Gonzalez Nothnagel <daniel@gonzalez-nothnagel.de> Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+2: xing-yang <xing.yang@emc.com> Workflow+1: xing-yang <xing.yang@emc.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Fri, 15 Jul 2016 18:41:29 +0000 Reviewed-on: https://review.openstack.org/332760 Project: openstack/manila Branch: refs/heads/master
2016-07-10update min tox version to 2.0Matt Riedemann
The commands used by constraints need at least tox 2.0. Update to reflect reality, which should help with local running of constraints targets. Change-Id: I27348462a21daab479d76b24b48e3a4e017504cc Notes (review): Code-Review+1: Tom Barron <tpb@dyncloud.net> Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Code-Review+1: zhongjun <jun.zhongjun@huawei.com> Code-Review+2: xing-yang <xing.yang@emc.com> Code-Review+2: Clinton Knight <cknight@netapp.com> Workflow+1: Clinton Knight <cknight@netapp.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 12 Jul 2016 13:46:24 +0000 Reviewed-on: https://review.openstack.org/340163 Project: openstack/manila Branch: refs/heads/master
2016-06-25Use upper-constraints in tox installsSean McGinnis
The tox targets do not currently use upper constaints, resulting in stable branch test runs pulling down newer library versions than what should be supported for those releases. This adds upper-constaints for test environment setup. Change-Id: I1ff164242901a891fb45da03b1595b2e9246a765 Closes-bug: #1596246 Notes (review): Code-Review+1: Goutham Pacha Ravi <gouthamr@netapp.com> Code-Review+1: zhongjun <jun.zhongjun@huawei.com> Code-Review+1: Tom Barron <tpb@dyncloud.net> Code-Review+2: xing-yang <xing.yang@emc.com> Code-Review+2: Ben Swartzlander <ben@swartzlander.org> Workflow+1: Ben Swartzlander <ben@swartzlander.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Mon, 27 Jun 2016 14:38:36 +0000 Reviewed-on: https://review.openstack.org/334207 Project: openstack/manila Branch: refs/heads/master
2016-06-08Add more dir exceptions to pep8 tox jobValeriy Ponomaryov
Manila has more dynamically created dirs than we exclude in pep8 job. Such as 'build' and 'cover'. Therefore, exclude them too to speed up passing of pep8 job on dev machine where lots of dynamically created things exist. TrivialFix Change-Id: I256720846acdd584cadaf59e9dedaad0ff741ae3 Notes (review): Code-Review+1: Telles Mota Vidal Nóbrega <tenobreg@redhat.com> Code-Review+1: Alexey Ovchinnikov <aovchinnikov@mirantis.com> Code-Review+1: zhongjun <jun.zhongjun@huawei.com> Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Code-Review+2: xing-yang <xing.yang@emc.com> Workflow+1: xing-yang <xing.yang@emc.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 14 Jun 2016 14:47:33 +0000 Reviewed-on: https://review.openstack.org/327074 Project: openstack/manila Branch: refs/heads/master
2016-05-24Squash E042 and E043 bashate warningsTom Barron
The bashate tool has been upgraded and now issues new warnings. Squash these E042 and E043 warnings since our shell scripts are doing the right thing currently and changing them to avoid the warnings makes them less readable and more fragile. Change-Id: I25ff337160ab176bb150a6186f6c3198cc068732 Notes (review): Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+2: Thomas Bechtold <tbechtold@suse.com> Workflow+1: Thomas Bechtold <tbechtold@suse.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Wed, 25 May 2016 14:10:49 +0000 Reviewed-on: https://review.openstack.org/320103 Project: openstack/manila Branch: refs/heads/master
2016-04-22Add Grenade support to Manilavponomaryov
Grenade is an OpenStack test harness to exercise the upgrade process between releases. It uses DevStack to perform an initial OpenStack install and as a reference for the final configuration. Add support of it to Manila project, doing following changes: - Add all required scripts for 'grenade' job, such as following: -- settings -- shutdown.sh -- upgrade.sh - Add optional 'resources.sh' script that performs functional tests using bash and manilaclient. List of scenarios: -- 1. create and delete share setting metadata and access rule -- 2. attach security service to share network -- 3. update quotas -- 4. create private share type and add access for project Note following: - Tempest is not used by design. - Manilaclient is used of stable version - version of 'base' devstack, which is 'stable/mitaka'. It also means that used API microversion is latest for that branch. Change-Id: I7d7e0c50933776ca1a710bcf069abb0282b821ac Depends-On: I4232382d3c944666bbf0916a3f285b84f6ff9d64 Implements blueprint grenade Notes (review): Code-Review+2: Ben Swartzlander <ben@swartzlander.org> Code-Review+1: Goutham Pacha Ravi <gouthamr@netapp.com> Code-Review+2: xing-yang <xing.yang@emc.com> Workflow+1: xing-yang <xing.yang@emc.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 05 May 2016 19:06:33 +0000 Reviewed-on: https://review.openstack.org/302801 Project: openstack/manila Branch: refs/heads/master
2016-04-08Detect addition of executable filesTom Barron
When running pep8 checks, fail if executable files have accidentally been added. Co-Authored-By: Eric Harney <eharney@redhat.com> Closes-Bug: 1566328 Change-Id: I7280c0403cd473cd4b88cfc021d4f605a7bb35f2 Notes (review): Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+1: Tiago Pasqualini da Silva <tiago.pasqualini@gmail.com> Code-Review+2: Rodrigo Barbieri <rodrigo.barbieri2010@gmail.com> Code-Review+1: Goutham Pacha Ravi <gouthamr@netapp.com> Code-Review+2: Clinton Knight <cknight@netapp.com> Workflow+1: Clinton Knight <cknight@netapp.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 12 Apr 2016 11:45:07 +0000 Reviewed-on: https://review.openstack.org/301576 Project: openstack/manila Branch: refs/heads/master
2016-04-05Remove unused logging import and LOG globalTom Barron
This patch removes unused global LOG variable and logging imports from various manila modules, and adds a script to be run as part of pep8 that will ensure that these do not creep back into the codebase. Change-Id: I162c4b2478df45aaf6ea8009b102d6de1a4e309e Notes (review): Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+2: xing-yang <xing.yang@emc.com> Workflow+1: xing-yang <xing.yang@emc.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 05 Apr 2016 12:58:28 +0000 Reviewed-on: https://review.openstack.org/301054 Project: openstack/manila Branch: refs/heads/master
2016-02-22Merge "Add tox fast8 option"Jenkins
2016-02-22Merge "Use ostestr to run unit test"Jenkins
2016-02-21Squash E006 bashate warningsTom Barron
bashate, which is invoked when running 'tox -epep8', currently emits many 'E006: Line too long' warnings. We apparently don't really care about line length in shell scripts and ignore these warnings. This commit instructs bashate to suppress these warnings so that we can see any warnings that we really care about. Change-Id: I565aeaef1f0590e5a6ee6a866974765c748f2965 Notes (review): Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+2: Clinton Knight <cknight@netapp.com> Workflow+1: Clinton Knight <cknight@netapp.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Mon, 22 Feb 2016 16:39:52 +0000 Reviewed-on: https://review.openstack.org/282875 Project: openstack/manila Branch: refs/heads/master
2016-02-21Add tox fast8 optionTom Barron
This commit ports over the fast8 option from cinder, which enables one to run pep8 only on one's changeset rather than against the whole body of code via: tox -efast8 Change-Id: Icb0286c3c66983f2f106c163cb11c690b2c57849 Notes (review): Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+2: Clinton Knight <cknight@netapp.com> Workflow+1: Clinton Knight <cknight@netapp.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Mon, 22 Feb 2016 23:23:44 +0000 Reviewed-on: https://review.openstack.org/282869 Project: openstack/manila Branch: refs/heads/master
2016-02-18Use ostestr to run unit testting.wang
The ostestr command provides a wrapper around the testr command. By default it also has output that is much more useful for OpenStack’s test including lists of all and failed tests and lists of the 10 slowest tests. ref: http://docs.openstack.org/developer/os-testr/ostestr.html Change-Id: If005d1987b2a4e586cf35f0cc1d07739f84235d2 Notes (review): Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+2: Clinton Knight <cknight@netapp.com> Workflow+1: Clinton Knight <cknight@netapp.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Mon, 22 Feb 2016 23:23:09 +0000 Reviewed-on: https://review.openstack.org/281910 Project: openstack/manila Branch: refs/heads/master
2016-02-14Removed ignored checks from tox.ini and fixed pep8 issuesDavid Sariel
Due to the hacking version update several pep8 checks where ignored in the flakes8 section of the tox.ini. Removed ignored checks and fixed appropriate pep8 issues. The list of fixed issues: - H105 Don't use author tags - H238 Old style class declaration, use new style (inherit from `object`) - H501 Do not use self.__dict__ for string formatting Closes-Bug: 1530719 Change-Id: I890ba4c54b0da10060767620ff5ff7e480d79a9c Notes (review): Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+2: Clinton Knight <cknight@netapp.com> Workflow+1: Clinton Knight <cknight@netapp.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Mon, 15 Feb 2016 01:21:16 +0000 Reviewed-on: https://review.openstack.org/262781 Project: openstack/manila Branch: refs/heads/master
2016-01-15Merge "Clean up removed hacking rule from [flake8] ignore lists"Jenkins
2016-01-14Clean up removed hacking rule from [flake8] ignore listsLiuNanke
We bump hacking>=0.10.0, and hacking removed some rules, for the full list of rules please see [1]. So don't need them any more. Hacking related commits: Remove H904 in commit b1fe19ebebe47a36b905d709467f5e82521bbd96 Add H105 using author tags Add H238 hacking no old style class Add H501 hacking no locals [1]https://github.com/openstack-dev/hacking/blob/master/setup.cfg#L30 Change-Id: I6a1f3c3ef22be0962ffb81a5da5660837552e5d7 Notes (review): Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+2: Clinton Knight <cknight@netapp.com> Code-Review+1: Marc Koderer <marc@koderer.com> Workflow+1: Valeriy Ponomaryov <vponomaryov@mirantis.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Fri, 15 Jan 2016 14:56:59 +0000 Reviewed-on: https://review.openstack.org/267394 Project: openstack/manila Branch: refs/heads/master
2016-01-11Add debug testenv in toxting.wang
Once we add debug testenv, we can use "tox -e debug -- --debug" to debug test cases when tox is running. Change-Id: If12d6c7bd8b1ae04096b526ad37978f60fdb70db Notes (review): Code-Review+2: Valeriy Ponomaryov <vponomaryov@mirantis.com> Code-Review+1: Goutham Pacha Ravi <gouthamr@netapp.com> Code-Review+1: Marc Koderer <marc@koderer.com> Code-Review+2: Clinton Knight <cknight@netapp.com> Workflow+1: Clinton Knight <cknight@netapp.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Fri, 15 Jan 2016 03:34:26 +0000 Reviewed-on: https://review.openstack.org/265574 Project: openstack/manila Branch: refs/heads/master