| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Introduced changes:
- pre-commit config and rules
- Add pre-commit to pep8 gate, Flake8 is covered in the pre-commithooks.
- Applying fixes for pre-commit compliance in all code.
Also commit hash will be used instead of version tags in pre-commit to
prevend arbitrary code from running in developer's machines.
pre-commit will be used to:
- trailing whitespace;
- Replaces or checks mixed line ending (mixed-line-ending);
- Forbid files which have a UTF-8 byte-order marker
(check-byte-order-marker);
- Checks that non-binary executables have a proper
shebang (check-executables-have-shebangs);
- Check for files that contain merge conflict strings
(check-merge-conflict);
- Check for debugger imports and py37+ breakpoint()
calls in python source (debug-statements);
- Attempts to load all yaml files to verify syntax (check-yaml);
- Run flake8 checks (flake8) (local)
For further details about tests please refer to:
https://github.com/pre-commit/pre-commit-hooks
Change-Id: Iba1e2ce707051ea393ac559fc51b1f07da1f6ee8
Signed-off-by: Moisés Guimarães de Medeiros <moguimar@redhat.com>
|
|
|
|
|
|
|
|
| |
This patch bumps bandit allowed version to >=1.6.0,<1.7.0 in order to
avoid the errors detailed here https://github.com/PyCQA/bandit/pull/393
Change-Id: I7b3144bd417f29cd57c16575ba47b45a4132aae7
Signed-off-by: Moisés Guimarães de Medeiros <moguimar@redhat.com>
|
|
|
|
| |
Change-Id: I95fd7fa65018dd3a887889ef11a88c63eaee74cf
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| | |
Now that we no longer support py27, we can use the standard library
unittest.mock module instead of the third party mock lib.
Change-Id: I4c5451afece8dfff30aa1ec4c7e0d5eb277043fd
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
|
|/
|
|
|
|
|
|
|
|
|
|
| |
The repo is Python 3 now, so update hacking to version 3.0 which
supports Python 3.
Fix problems found.
Remove hacking and friends from lower-constraints, they are not needed
for installation.
Change-Id: I3f23bf09ef24fe34e128102c34382da98e10f5c1
|
|
|
|
|
|
|
|
|
|
|
|
| |
Bandit 1.6.0 accidentally changed how the exclusion list option is
handled and breaks our use of it. Cap to the previous version until
Bandit has fixed the problem.
Sphinx 2.0 no longer works on python 2.7, so we need to start capping
it there as well.
Change-Id: I719a8cff50dcc0dea62db14edf7b9ab35b72facd
Reference: https://github.com/PyCQA/bandit/pull/489
|
|
|
|
|
|
|
| |
Use latest release 1.1.0 and compatible changes w.r.t pep8
Change-Id: Ide3a556918f341de6eafecb36ca431da40a0aed0
Closes-Bug: #1815715
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The threading-compatible eventlet Event class has a race condition on
the wait method. If greenthread A is blocked on the wait, but another
greenthread B calls clear() and then set(), B calls self._event.send(),
but A is waiting on a different eventlet Event which is no longer used
by the oslo.service Event...
To resolve this, when clearing an Event trigger the underlying eventlet
Event immediately, then have the wait() method resume waiting on the new
eventlet Event.
Change-Id: I81579e2977bb965a5398a2cb4e3e24f5671e856a
Co-Authored-By: Victor Stinner <vstinner@redhat.com>
Co-Authored-By: Hervé Beraud <hberaud@redhat.com>
Closes-Bug: #1805706
|
|
|
|
|
|
|
|
|
|
| |
According to Openstack summit session [1],
stestr is maintained project to which all Openstack projects should migrate.
Let's switch to stestr as other projects have already moved to it.
[1] https://etherpad.openstack.org/p/YVR-python-pti
Change-Id: Ica0262653e8c18681d049acca6d5920ac38f97e6
|
|
|
|
| |
Change-Id: I68a6de52dc412e026f8611052208afdc293f4069
|
|
|
|
| |
Change-Id: I73293123611d79ed3dad7ff6475c07b97f742cc9
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
For compliance with the Project Testing Interface as described in:
https://governance.openstack.org/tc/reference/project-testing-interface.html
For more detials information, please refer to:
http://lists.openstack.org/pipermail/openstack-dev/2017-December/125710.html
Change-Id: I3d86f1897dc900ba815c0c3975e95e27bb29b532
|
|/
|
|
|
|
|
|
|
| |
python-subunit is not used directly anywhere
and it is dependency of both testrepository
and os-testr
(probably was used by some tox wrapper script before)
Change-Id: If17dfc040d87981ed0a46abbe25e3d09f6e38e46
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Method validate_integer() is used in Cinder[1], Nova[2] and
Masakari[3].
So it's a good candidate to add this method in Oslo.utils.
[1] https://github.com/openstack/cinder/blob/master/cinder/utils.py#L1032
[2] https://github.com/openstack/nova/blob/master/nova/utils.py#L846
[3] https://github.com/openstack/masakari/blob/master/masakari/utils.py#L244
Change-Id: I9086b9057e24aa4747a81507ab14aad60e39ab40
|
| |
| |
| |
| | |
Change-Id: Ic6136f13d0eda06c3cf98f3aa0c148a13d3cbe75
|
|/
|
|
| |
Change-Id: Iec6b77a93b4651695affbe570b2c50f19798c225
|
|
|
|
| |
Change-Id: I6ec98117a03224efe0a7a9fbed6293cea0cd9982
|
|
|
|
| |
Change-Id: I6a497f6c3038717eea89f2c9e997f142f7a9657b
|
|
|
|
| |
Change-Id: I02e5e76f8596e21cb229038276b14dd10b2ef0c4
|
|
|
|
| |
Change-Id: Ic1b1d2706b92ac7f20fc76576762b4da553dd435
|
|
|
|
| |
Change-Id: Ief1b96ec6d08552f285540a68994c70cc9af3098
|
|
|
|
| |
Change-Id: Ia3a3874dbd3c76716a97899967ef15c04d7fae2e
|
|
|
|
| |
Change-Id: If29cd52629b0c2d1df84fcb52fa3dfe4a4680606
|
|
|
|
| |
Change-Id: Ib6a72c4dd0803ac0578e5b136cf84267de2fdf5e
|
|
|
|
| |
Change-Id: Idcf31dff3714eaf5134ac58d17a1d5885c7d7ecc
|
|
|
|
| |
Change-Id: I5e4241f321a43ffb69e7da83d030cb09c6252ad8
|
|
|
|
| |
Change-Id: I3a8470b683b6e856b906d525fc6a085d0e6ee442
|
|
|
|
| |
Change-Id: Ib03ae9bfbc080421a4eb7bbba4edbae04cb00eca
|
|
|
|
| |
Change-Id: I16ce7840503273a0a1c545123930e2c65ab04276
|
|
|
|
|
|
|
|
|
| |
Since pbr already landed and the old version of hacking seems not
work very well with pbr>=2, we should update it to match global
requirement.
Partial-Bug: #1668848
Change-Id: Iae0d1394c2596a9c188531015c950de6108c7acb
|
|
|
|
| |
Change-Id: I7ba38fff9ff21765b7fac1618005dd003f797594
|
|
|
|
| |
Change-Id: Iac0d856b42f6289fd7d4aa9d332a54e865270e3b
|
|\ |
|
| |
| |
| |
| |
| |
| | |
Closes-Bug: #1634356
Change-Id: I6462e42df2a28cd63d6059ef5320cec3ca1473d9
|
| |
| |
| |
| | |
Change-Id: I456f3e45481415d95261b18cd5a20a1aaa7de389
|
|/
|
|
| |
Change-Id: Ia5261e6fcd08649f0b3e4a85082d59847f0d828b
|
|
|
|
| |
Change-Id: I88aa3788933e496031c4a772daa5ad8b6e9b8093
|
|
|
|
| |
Change-Id: I154ae5bf00ae6f7b965e67d98bd0dd23ca4352bc
|
|
|
|
| |
Change-Id: Icfaff6feda793bfd4b495e3898a214ba79bf2981
|
|
|
|
| |
Change-Id: I02a0ce99c358cc92e33bb5a419f33fb9f9470443
|
|
|
|
|
|
| |
It's only needed for python < 2.7 which is not supported
Change-Id: Iea5e3c9c64672eeef59e9cb6cefdf161b55721d8
|
|
|
|
| |
Change-Id: I455892333110709485d568ea30049a3c6d29941b
|
|
|
|
| |
Change-Id: I4863ef81ab4ed9b6c57a3fe965f06e342451c9ac
|
|
|
|
| |
Change-Id: I8526da9d6aa4b6b40a688f8c06c541eda2d4b203
|
|
|
|
| |
Change-Id: I100563774b3c1cf1720d67d365e6bed7ee75b59c
|
|
|
|
| |
Change-Id: I29559392ad010346acb5cf3cde6b9638eabf99d4
|
|
|
|
| |
Change-Id: Ib32c3a895d4c23b4603c516f82d092623948c6a6
|