summaryrefslogtreecommitdiff
path: root/tox.ini
Commit message (Collapse)AuthorAgeFilesLines
* Add a hacking rule for string interpolation at loggingForestLee2017-07-181-1/+1
| | | | | | | | | | | | | String interpolation should be delayed to be handled by the logging code, rather than being done at the point of the logging call. See the oslo i18n guideline * https://docs.openstack.org/oslo.i18n/latest/user/guidelines.html#adding-variables-to-log-messages and * https://github.com/openstack-dev/hacking/blob/master/hacking/checks/other.py#L39 Closes-Bug: #1596829 Change-Id: Iba231be2476dcbeeb0edd76d6a921e549d183758
* Remove duplicated hacking rulezlyqqq2017-06-281-0/+1
| | | | | | | | | Remove the hacking rule: CheckForAssertingNoneEquality and enable H203. http://git.openstack.org/cgit/openstack-dev/hacking/tree/HACKING.rst Change-Id: Id5c76e4befde163cfce3badbcd329e94485daed3
* Cleanup policy generationLance Bragstad2017-05-021-1/+1
| | | | | | | | | This commit only adds the sample policy file to the projects .gitignore file in the interest of keeping environments cleaner. This also adds the genpolicy tox environment to the tox environment list to be consistent with how we maintain other tox environments. Change-Id: Idf538c97f5f7383d8d6e5c18c985d9715033926b
* Move policy generator config to config-generator/Colleen Murphy2017-04-211-1/+1
| | | | | | | | | etc/ is for config files that should be installed on disk to support a running keystone. config-generator/ is for files that oslo-config-generator uses to generate sample config files and are not needed by keystone in a live deployment. Change-Id: I0614709ded739d77ec620150bcb6c2f456024b6f
* Add policy sample generationAnthony Washington2017-03-291-0/+3
| | | | | | | | | This adds the entry point and config file necessary for using the oslo.policy sample generation script. It also adds a tox target to simplify the usage of it. Change-Id: I7feb522b2ea5f56e48982169c7ebbb2ec2ef2cb3 Partially-Implements: bp policy-in-code
* Use ostestr instead of the custom pretty_tox.shLuigi Toscano2017-02-091-1/+1
| | | | | | | Switch to os-testr as wrapper for testr, instead of the custom script pretty_tox.sh. Change-Id: If10ac96ac32813714be2b83f895071820126dbd8
* Remove references to Python 3.4Eric Brown2017-01-061-3/+3
| | | | | | | Now that there exists only a gate job for Python 3.5 and not 3.4, we should remove those references to the 3.4 that is untested. Change-Id: Ia017d1199f8ca31808680b91c708823711127659
* Fix issues with keystone-dsvm-py35-functional-v3-only on py35Davanum Srinivas2017-01-051-1/+1
| | | | | | | | | | | | | | keystone/token/providers/fernet/token_formatters.py * decode payload[2] from bytes to string before comparing with a string (CONF.identity.default_domain_id) keystone_tempest_plugin/services/identity/clients.py keystone_tempest_plugin/services/identity/v3/auth_client.py keystone_tempest_plugin/services/identity/v3/identity_providers_client.py * decode the response body from bytes to string before we try to parse the json using json.loads Change-Id: I98053bc498d78c5f0076a66e725ff2d634f5b663
* Remove support for PKI and PKIz tokensSteve Martinelli2016-11-011-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the first step of several to remove PKI token support in keystone. A large issue in removing PKI support is support for the revocation list must be maintained. This patch removes support for the token format, it's surrounding tests and examples that are generated. Additionally, some wording has been changed around the CLI and config options to make the distinction between keys and certs used for PKI tokens and those used for getting the revocation list (a list of tokens that are revoked, which is signed). Future patches will: - Remove the keystone-manage commands for generating certs - Modify the revocation list (at /auth/tokens/OS-PKI/revoked) to return a 403 if pki is not configured (instead of raising a 500). We cannot remove the API as that would break an API contract. - Options to configure PKI will be marked as deprecated - If PKI is configured a normal signed list will be returned (same behavior as today) - Follow up patch to keystonemiddleware will make sure auth_token does not rely on the revocation api at all. Related-Bug: 1626778 Related-Bug: 1626779 Co-Authored-By: Boris Bobrov <bbobrov@mirantis.com> bp removed-as-of-ocata Change-Id: Icf1ebced44a675c88fb66a6c0431208ff5181574
* Merge "Add bindep environment to tox"Jenkins2016-11-011-0/+8
|\
| * Add bindep environment to toxJeremy Liu2016-10-291-0/+8
| | | | | | | | | | | | | | | | This is helpful for listing any missing system requirements [1]. [1] http://docs.openstack.org/infra/manual/drivers.html#package-requirements Change-Id: I54e3955ff0befae35eb1aada478023fa00afc5e7
* | Add structure for Devstack pluginKristi Nikolla2016-10-311-1/+1
|/ | | | | | | | | | This review creates the structure for the Devstack plugin and prints to the console to ensure its execution in the gate. Follow-up reviews will do more useful stuff like setting up the environment for our functional testing (ldap, federation). Change-Id: I820ae355ae8f3183fee2b8207e3c17e8bd10dc17
* remove legacy driver tox targetSteve Martinelli2016-10-051-9/+0
| | | | | | | Another follow up from 810e15689b200f92980e04e233523d72d885d4bd. Change-Id: I5b16b30a3891a27359b3c238f10a4a13f5281866 Depends-On: Ic4b067b472dad3b8231cb2397abbb92a52939c31
* Remove stable driver interfacesSteve Martinelli2016-09-281-8/+1
| | | | | | bp removed-as-of-ocata Change-Id: I4672cf7d9d72ef725212085972dbcd90db0e47cf
* Constraints are ready to be used for tox.iniSteve Martinelli2016-08-121-18/+0
| | | | | | | | | Per email[1] from Andreas, we don't need to hack at install_command any longer. [1] http://openstack.markmail.org/thread/a4l7tokbotwqvuoh Change-Id: Ic224c1e20693410c485e45cab5bdaa5d96192f09
* Fix warning when running tox -e api-refTin Lam2016-08-081-0/+2
| | | | | | | | | | | Running `tox -e api-ref` prints warning: WARNING:test command found but not installed in testenv cmd: /bin/rm env: /opt/stack/keystone/.tox/api-ref Maybe you forgot to specify a dependency? See also the whitelist_externals envconfig setting. Change-Id: I4c1718dac22c374f048d9e04f42051e10021bb31
* Don't include openstack/common in flake8 exclude listlilintan2016-08-021-1/+1
| | | | | | | | The directory openstack/common was used to keep codes from oslo-incubator, we have retired oslo-incubator, so don't use this directory any more. Change-Id: I0637c17f5c7aecf5bda87c8e7a6ef834be299baa
* Use constraints for coverage jobAndreas Jaeger2016-07-151-4/+0
| | | | | | | | | | 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: I3812776ab228bf28df9934273df7fe8ee0880660 Depends-On: I2c0fb1e9b5a51be409687d9cb85ee3ae70c1ce94
* Merge "Include doc directory in pep8 checks"Jenkins2016-07-021-1/+1
|\
| * Include doc directory in pep8 checksEric Brown2016-06-301-1/+1
| | | | | | | | | | | | | | | | | | Currently, flake8 runs against doc related directories such as releasenotes and api-ref. Might as well remove doc from the flake8 exclude list. Each of these directories has only one python file (conf.py). Change-Id: I0445ad083d8d9167e0309950c200c9abb766bc1a
* | Update the nosetests test regex for legacy testsDavid Stanek2016-06-291-4/+4
|/ | | | | | | | | The default regex, '(?:^|[\b_\./-])[Tt]est', will match methods that are not test methods. For example, I created a new method 'skip_test_overrides' and it was mistakenly run as test. By convention all of our tests start with 'test_' so this is a safe change. Change-Id: I4f5917de53681c950c72479f16f88e76f49f6bfc
* Merge "Let setup.py compile_catalog process all language files"Jenkins2016-06-071-3/+0
|\
| * Let setup.py compile_catalog process all language filesSven Anderson2016-05-201-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Two years ago the translation files have been split into several files, separating the log messages of different log levels from each other, like X.pot, X-log-warning.pot, X-log-info.pot, and so on. However, the setup.py command `compile_catalogs`, that comes from the babel package and compiles the corresponding .po files into .mo files, only supported one file per python package. This means that during packaging `compile_catalogs` never compiled the X-log-*.po files, so the corresponding translations were always missing. Since babel 2.3 the domain can be set to a space separated list of domains. This change adds the the additional log level files to the domain list. The obsolete check that .po and .pot files are valid is removed from tox.ini. Change-Id: I1f0bfb181e2b84ac6dd0ce61881cd2cc4400bdcb Closes-Bug: #1536226
* | PEP257: Ignore D203 because it was deprecatedDolph Mathews2016-06-031-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | PEP257's D203 check ensures that you have a blank line before class docstrings. This rule directly conflicts with D211 (no blank lines before class docstrings), which is intended to supersede D203. The original language in PEP257 which D203 was based on was actually removed from PEP257 by Guido here: https://hg.python.org/peps/rev/9b715d8246db Change-Id: Icc048b947acea8f655d00540c221123b906e7545
* | Migrate identity /v3 docs from api-ref repoSamuel de Medeiros Queiroz2016-05-241-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Docs at [1] have already been converted from WADL (SGML / XML) to RST using fairy-slipper [2]. This commit polish the results from the conversion and migrate the docs to our repository under 'api-ref/source' directory. In addition, it added missing descriptions for access_token_id and consumer_id to 'parameters.yaml'. Polishing the generated RST files include: - Removing unnecessary blank lines; - Removing empty references. Polishing the generated RST files do not include: - Modifying their content; - Modifying file names; - Wrapping lines at the maximum of 79 chars. Updating the documentation will be done after this migration step. This change also adds a tox environment to build the docs using sphinx under 'api-ref/build', which in turn is added to '.gitignore'. Lastly, 'os-api-ref' is added as a test requirement. It provides the sphinx stanzas rest_method and rest_parameter, used to define OpenStack APIs in RST docs. [1] https://github.com/openstack/api-site/tree/master/api-ref/source/identity/v3 [2] https://github.com/russell/fairy-slipper Change-Id: If1b9a3e1b2e4ea7211c337071254c26b881893a3
* | Remove helper script for py34Morgan Fainberg2016-05-211-5/+0
|/ | | | | | | | | | Remove the helper script for py34, it prevents the use of arguments being passed down e.g. tox -epy34 -- --failing due to the lack of passing {posargs} to the script. Since py34 has no special exceptions now, the specific env for py34 with a special pretty_tox script is not needed. Change-Id: I4d960d82e2844c0c82a14be5e92230f1698e11b3
* Fixing D105, D203, and D205 PEP257Navid Pustchi2016-05-031-4/+1
| | | | | | | | | | Currently tox ignores D105, D203, and D205. D105: Missing docstring in magic method. D203: 1 blank required before class docstring. D205: Blank line required between one-line summary and description. This change removes it and make keystone docstring compliant with it. Change-Id: Id916612dd246e02159907d286e448545120f3655
* Merge "Fix D400 PEP257"Jenkins2016-04-211-2/+1
|\
| * Fix D400 PEP257Navid Pustchi2016-04-211-2/+1
| | | | | | | | | | | | | | | | Currently tox ignores D401. D400: First line should end with a period. This change removes it and make keystone docstring compliantwith it. Change-Id: I9a9520e69701718ff471eebbcc52199dacdd9c68
* | Bump the required tox version to 2.3.1David Stanek2016-04-201-1/+1
|/ | | | | | | | | | Other projects are already starting to make this change and it should help with some of the strange issues we commonly get asked in IRC. When those issues come up the first step is always update to the latest tox and pip, so we might as well make that recommendation more formal. Change-Id: Ic320881797b99747357ac14c6f84636e5070523d Related-Bug: #1572202
* Merge "Remove eventlet support"Jenkins2016-04-191-1/+1
|\
| * Remove eventlet supportSteve Martinelli2016-04-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | Eventlet has been deprecated since the Kilo release and is being removed in Newton. A follow on patch will be proposed to remove the [ssl] section since it is now redundant. Co-Authored-By: Grzegorz Grasza <grzegorz.grasza@intel.com> Partially implements: bp removed-as-of-newton Change-Id: I963d94bbd188dbb6eba68623a42c5bc3f2289da4
* | Merge "Remove comment from D202 rule"Jenkins2016-04-181-1/+0
|\ \
| * | Remove comment from D202 ruleRodrigo Duarte2016-04-041-1/+0
| | | | | | | | | | | | | | | | | | The rule is no longer ignored in the list below the comment. Change-Id: Ib8e4094493e1c7841bda0d0fc7251189b90072a9
* | | Fix post jobsAndreas Jaeger2016-04-171-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The tox venv environment is run during post jobs and thus cannot use constraints. See: http://logs.openstack.org/db/db7bdf9aa0cb0ba5fbae5ae07ecdb9f024213deb/post/keystone-docs/92d1e87/ http://logs.openstack.org/db/db7bdf9aa0cb0ba5fbae5ae07ecdb9f024213deb/post/keystone-branch-tarball/eb685ad/ We run for docs jobs: "tox -e venv python setup.py build_sphinx" thus, the docs environment is not used. For branch tarball, the infra scripts use: "tox -e venv python setup.py sdist" And infra does not setup constraints for post jobs currently as this is not working with current tools Fix tox.ini for this Change-Id: I048368981e4be739c66073fdd9bc8a9663498a80
* | | Keystone jobs should honor upper-constraints.txtDavanum Srinivas2016-04-171-1/+11
| |/ |/| | | | | | | | | | | | | | | | | | | Some targets don't respect upper-constraints like cover and releasenotes, so make sure don't use the same install_command for those jobs. Temporarily add Babel to avoid problem with the keystone-coverage-db coverage job. Change-Id: I8636e7c86c6c5c608429fab88e181108ae615db9
* | Fix D401 PEP8 violation.Navid Pustchi2016-04-141-2/+1
| | | | | | | | | | | | | | | | Currently tox ignores D401 (401: First line should be in imperative mood). This change removes it and make keystoneauth docstring compliantwith it. Change-Id: I136cf810f47c4c19f29216907a63f226930b5082 Partial-Bug: 1570049
* | Merge "Add py3 debugging"Jenkins2016-04-061-0/+7
|\ \ | |/ |/|
| * Add py3 debuggingTom Cocozzello2016-03-181-0/+7
| | | | | | | | | | | | | | Add a tox testing environment that allows users to debug with python 3. Change-Id: Iaf757de29936e1361fd720710154963da76a7a10
* | Add `patch_cover` to keystoneTom Cocozzello2016-03-281-0/+4
|/ | | | | | | | | | | | | | | | | | | | | `patch_cover` checks to make sure your patch has adequate test coverage. How it works is it stashes your changes, runs `tox -e cover`, saves the results then stash pops your change and runs cover again. It then reports to show if your code in your patch has 100% test coverage or not. The intent of this job is to be a non-voting / warning like job that can inform reviewers if the review clearly needs more testing. Since cover.sh is copied from [0] the copyright should reflect the owners of the code. [0]: https://github.com/openstack/rally/blob/master/tests/ci/cover.sh Change-Id: I66572072aa4d9563b6e1f73a8f8b3437116ef3c4
* Explicitly exclude tests from bandit scanEric Brown2016-03-081-2/+2
| | | | | | | | | | | | Now that the bandit linter runs without a bandit.yaml specified, it should exclude the tests directory similar to how it did before in the bandit.yaml. The reason this has not caused a problem yet is because bandit is still finding a config bandit.yaml in .tox/pep8/etc/bandit/bandit.yaml. But in a newer version of bandit this will not be the case. Change-Id: Ie81d052cd57318a987f0ebfe718e1c01bdbffcd3
* Fix warning when running toxBrant Knudson2016-03-071-1/+3
| | | | | | | | | | | | | When running tox it would print out a warning: WARNING:test command found but not installed in testenv cmd: /usr/bin/find env: /opt/stack/keystone/.tox/py27 Maybe you forgot to specify a dependency? See also the whitelist_externals envconfig setting. This is because we forgot to specify a dependency. Change-Id: I4270c52213159cefa04a63979ee0bb2e1a2fa35f
* remove pyc files before running testsSean Dague2016-03-011-5/+16
| | | | | | | | | | | | | | pyc files for old code files and test files won't get removed from the local tree when those files are removed. They will still be discovered and used when running tox tests. This removes pyc files before running tests to ensure that test results are repeatble by end users even if they are updating older git trees. Closes-Bug: #1551312 Change-Id: I9987cc57dc9ac2bd0331af6cb59e2c5e137d9849
* Switch to configless banditBrant Knudson2016-02-221-2/+2
| | | | | | | | The 0.17.3 release of bandit supports running with no config, enabling all the tests with a default configuration for them, switch to that since it's easier to maintain. Change-Id: I507b7a11e8d1d69a6d9ec88493cbefe48be5ddcf
* Removing H405 violations from keystoneTin Lam2016-02-211-2/+1
| | | | | | | | | | Keystone's tox.ini contains an "ignore" entry for H405 violations: multi line docstring summary not separated with an empty line. All violations of H405 should be fixed so that H405 can be removed from the ignore list. Change-Id: I1b2aae0cabc20909cf3b0a405d5e31c5d91148b2 Closes-Bug: #1482773
* Add back a bandit tox jobEric Brown2016-02-171-0/+6
| | | | | | | | The bandit project uses an integration test to ensure it doesn't break other projects by introducing new changes. To run this integration, it necessary to have a common tox target of 'bandit'. Change-Id: I07c66fa2535211536d0218abd86bea0ae8644065
* Stop using nose as a Python3 test runnerDavid Stanek2016-02-111-52/+1
| | | | | | | | | | | | | There are several reasons for this change: - removes the need for the custome py3 test dependencies - makes the broken tests more visible because they are listed instead of the working ones - using a blacklist means that new test files are tested by default in py3 bp python3 Change-Id: I7472eec05d80cdd99c0ee376159578e0d7306f51
* Make pep8 *the* linting interfaceAndreas Jaeger2016-02-101-13/+3
| | | | | | | | | | | | According to the PTI (=Python Test Interface, http://governance.openstack.org/reference/cti/python_cti.html), pep8 is the interface for codestyle checks. Move all tests from linters to pep8. This basically reverts Ic9efdcb13978d2f29312a4a1f0fdd859d55c8398. Depends-On: I9346e4bf8c3a50de088c6a157f0d871c0a9333a1 Change-Id: I99df6de57d345c40e39801d66575d35d0d8a36a9
* Disable the ipv6 tests in py34David Stanek2016-02-101-1/+0
| | | | | | | Eventlet 0.18.2 breaks our py34 tests. Disabling these test makes it work again. Change-Id: Ie69342ae26dd031e3121e23404b31a133e53cf0e
* Merge "test_credential.py work with python34"Jenkins2016-02-061-0/+1
|\
View Lorry Controller Status