| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
hacking 3.1.0 depended on 'flake8<3.8.0,>=3.6.0', while we were
specifying flake 3.8.3. This resulted in an error when using the
dependency resolver introduced in pip 20.3. Resolve this by bumping to
hacking 3.2.0.
We also remove bandit and pre-commit from test-requirements, since these
are linters which are not managed by upper-constraints and are not
necessary to run tests. oslo.context is also specified in both
requirements.txt and test-requirements.txt, so we remove it from the
latter.
Change-Id: I829870c327b73b583877b9b969ee38f0bcaa1495
Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
flake8 new release 3.8.0 added new checks and gate pep8
job start failing. hacking 3.0.1 fix the pinning of flake8 to
avoid bringing in a new version with new checks.
Though it is fixed in latest hacking but 2.0 and 3.0 has cap for
flake8 as <4.0.0 which mean flake8 new version 3.9.0 can also
break the pep8 job if new check are added.
To avoid similar gate break in future, we need to bump the hacking min
version.
- http://lists.openstack.org/pipermail/openstack-discuss/2020-May/014828.html
Change-Id: I20e3db3b092462c85d784aad51d5e98f3499f3f1
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Introduced changes:
- pre-commit config and rules
- Add pre-commit to pep8 gate, Flake8 is covered in the pre-commit hooks.
- 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: Ia9f7040f9966f1492c590a005f55ef7f3b67f0c9
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: I0570c916cffc08bcbaebb385a9cc4a4c7038b215
Signed-off-by: Moisés Guimarães de Medeiros <moguimar@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The repo is Python 3 now, so update hacking to version 3.0 which
supports Python 3.
Blacklist:
W503 line break before binary operator
W504 line break after binary operator
Fix:
E123 closing bracket does not match indentation of opening bracket's line
E126 continuation line over-indented for hanging indent
Change-Id: I39003496a3f4be5a4cb05cdbae53a9c097e34e14
|
|
|
|
| |
Change-Id: I2f12fb5058984c1fc6beda4d13f934dd9b218218
|
|
|
|
| |
Change-Id: Iafa01298a49342ce8b0a1405d2d8ea2b61b3e198
|
|
|
|
|
|
|
|
|
|
|
|
| |
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: Idead9b4198c6b05d72bae60dee06e5aebc223822
Reference: https://github.com/PyCQA/bandit/pull/489
|
|
|
|
| |
Change-Id: I7009b639743c22f885cebd532a3d6c1b0f9f1226
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This commit makes it easier for services to protect APIs meant for
domain-only operations. It does this by making "domain-scope" an
official scope type to check for during policy enforcement.
A good example of where this would be useful is protecting the user
API in keystone, since user's are technically owned by domains.
This commit bumps the version of oslo.context to 2.22.0, which also
has domain support.
Depends-On: https://review.openstack.org/#/c/613635/
Change-Id: Ifc83a5f261bc823060eca5c4d0a4bf07966794c4
|
|
|
|
|
|
|
|
|
|
|
|
| |
The ``creds`` dictionary passed into oslo.policy's enforce() method
assumes a lot of the same values already specified by oslo.context
RequestContext objects.
This commit teaches enforce() to handle being passed an instance of
a RequestContext object, and populate credential values accordingly.
Change-Id: Ia74bf6c40b1e05a1c958f4325e00f68be28d91b9
Closes-Bug: 1779172
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This commit fixes two issues that are currently blocking the gate.
The first is that it bumps the Sphinx requirement to be within
acceptable constraints. The second is that it converts oslo.policy to
use stestr instead of testr. This is all being done in one patch
because proposing them individually causes deadlock (the patch to
bump the sphinx requirement fails because we're still using testr and
the patch to convert to stestr fails the requirements-check job).
The following explains the reasoning behind the stestr change.
With the upgrade to oslotest 3.6.0 [0], testr no longer works [1].
This is because oslotest no longer requires testr and we don't depend
on it directly in oslo.policy.
[0] https://github.com/openstack/requirements/commit/d5a3c58f7195517a6083032e41b702c2a0aca431
[1] https://github.com/openstack/oslotest/commit/897823fbd61ae9ba015af38e9fed5a5728de64c2
Change-Id: I6dac4c8e7b39c9b80cc8f3728763e8d783c9e940
|
|
|
|
| |
Change-Id: Iff836145c26a9e27410d78c9163086753c271cd6
|
|
|
|
| |
Change-Id: I54badf14549f0438bbe537be9aa96094f8e8c6d0
|
|
|
|
| |
Change-Id: Ib71072486918f8b9f3027b9e49ab010c0b228fa8
|
|
|
|
| |
Change-Id: I522a9788eb706cb8bea02fced8e949e1b1839579
|
|
|
|
|
|
| |
Add the bandit security scanner to the pep8 job.
Change-Id: I3cf1630c8d69515700e98a9274f42aa3f3b05303
|
|
|
|
| |
Change-Id: I7ee98c62f34c1758fadabfed3e5884602249b204
|
|
|
|
| |
Change-Id: Ia721dd1e8016f1c9d51f3299575a570a3b65ba95
|
|
|
|
| |
Change-Id: I5492440da5fad4a77ee3c6659a2897845208439b
|
|
|
|
|
| |
Change-Id: I3a35af82ff83946e30f00171bd50c2eac7b9d0a5
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
|
|
|
|
| |
Change-Id: I9badce163f0f2b7b01ea8d9436ef6db67fe561d5
|
|
|
|
| |
Change-Id: I649936d45e36b7e58d3e38f8936aa297d0d0524d
|
|
|
|
| |
Change-Id: Icd75dfdbb6a67322eba158be4a495bdfafca0fb2
|
|
|
|
| |
Change-Id: I4de956d302b387697ea79c14784e2a5df28efa9e
|
|
|
|
| |
Change-Id: I5ce59cfc43b51e5cc27f16844c5375a22825ad62
|
|
|
|
|
|
|
|
|
| |
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: Ia07952992bd90213ed9ad9b0adaa7a47553708f7
|
|
|
|
| |
Change-Id: I0b7cf1597fc844d698d2711058ead291ac692769
|
|
|
|
| |
Change-Id: I8da9f6fa8ddb2b2cd4ddcc84351f1e4ef63f2aa4
|
|
|
|
| |
Change-Id: I7d8dd22af3a938f0b8a4823ffde01cab94578585
|
|
|
|
| |
Change-Id: I437a4b5edd2353fa0b1888c1730a3eef67b6fbef
|
|
|
|
| |
Change-Id: Ie990f8b2e9b562a2341bfe741e1f4fdedafc9ebb
|
|
|
|
| |
Change-Id: Ib05af2ba08af4783942a1f4599ed572bc11aa9a7
|
|
|
|
| |
Change-Id: I5daf4add079c41a6bb98b53b687fb8ed53c508b0
|
|
|
|
| |
Change-Id: I9c467fc429143a81948988201940fcab69965008
|
|
|
|
|
|
|
| |
An initial patch to add reno and create a base directory for
release notes
Change-Id: If7aeb47ca244cfe03a34b8b8a80f0075de5f4b02
|
|
|
|
| |
Change-Id: Ia8fce74c2caa158ac80a01ab68a15b85fbe4c085
|
|
|
|
| |
Change-Id: Ib877d6f5cdaa5aa0548e4f71701870354670b8fa
|
|
|
|
| |
Change-Id: I05f878645a8624470490eeefa311e6f7e4b796ff
|
|
|
|
|
|
|
|
|
| |
We're trying to remove httpretty from openstack requirements in favour
of requests-mock.
Replace httpretty in testing.
Change-Id: I3d02d2ccffc6535d519e49e4fbd9f0ab7490c3b5
|
|
|
|
| |
Change-Id: I83945e4206d63cecb2d0df8961c943735df8c864
|
|
|
|
|
|
|
| |
Running tox -e cover would fail since the `cover` tool wasn't
being installed in the venv.
Change-Id: Ie2f68b03751faedf4e973e1fdcb5546338bab41d
|
|
|
|
| |
Change-Id: I01f6d068caaf030d7e76a3dc9078463a4bf7a697
|
|
|
|
|
|
|
|
| |
The requests interface is much nicer and easier to
use so we might as well use it instead of direct urllib
usage.
Change-Id: I364ddb5f86900a3e166f4480d9f4889a68de247f
|
|
|
|
| |
Change-Id: I00e47063a5eaff9352a03cd3985981538ac9b0f5
|
|
|
|
| |
Change-Id: I2a9b613a0cafeee3d972eb017537f37fdd4c6e23
|
|
|
|
| |
Change-Id: Ie699eaf8afc2d64a559b1b07e342436db8555713
|
|
|
|
| |
Change-Id: Ic58778ba5c28c3b5e139ef12bbfa37cb4c10da03
|
|
|
|
|
| |
Change-Id: Ic40b70b488873bf3679b9187d315b3088737e5a3
Depends-On: Ib948b756b8e6ca47a4c9c44c48031e54b7386a06
|
|
|
|
| |
Change-Id: Ia416f8945ecc9073fbf303be37c411b6f20bbb92
|