| 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-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: Ibd0c3d64fdc5c293d9d676d33eab828d9fde971f
Co-authored-by: Moisés Guimarães de Medeiros <moguimar@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
As per victoria cycle testing runtime and community goal[1]
we need to migrate upstream CI/CD to Ubuntu Focal(20.04).
Fixing:
- bug#1886298
Bump the lower constraints for required deps which added python3.8 support
in their later version.
Story: #2007865
Task: #40207
Closes-Bug: #1886298
[1] https://governance.openstack.org/tc/goals/selected/victoria/migrate-ci-cd-jobs-to-ubuntu-focal.h>
Change-Id: Id6b499239fa8b6480c5f67d460215ab458677c6d
|
|
|
|
|
|
|
|
| |
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: I9235560667f664643007b8ca0be1707eab4126ad
Signed-off-by: Moisés Guimarães de Medeiros <moguimar@redhat.com>
|
|
|
|
| |
Change-Id: I3282e4962e92bdd5a401c5b1c61fced68e8d0c8a
|
|
|
|
|
|
|
|
|
| |
Remove six, the python 2/3 compatibility library. It's not needed
anymore since the repo is python3 only.
Remove a now unneeded hacking test.
Change-Id: I40522c4accb4aaf8115d11fee8b081e2d991cb4d
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The repo is Python 3 now, so update hacking to version 3.0 which
supports Python 3.
Update local hacking check for new flake8 version.
Blacklist:
W504 line break after binary operator
Fix:
E741 ambiguous variable name
E117 over-indented
E305 expected 2 blank lines after class or function definition, found 1
F841 local variable 'e' is assigned to but never used
W605 invalid escape sequence '\.'
Change-Id: I99d574ca6569f1f177d2c5ce1011f269f4343619
|
|
|
|
|
|
|
|
|
|
|
| |
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: Ie6b379f2c99862c37891ac03c52464e07bc2b2cc
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch messaging intermediaries used for the amqp1 driver
test. The combination of the apache qpid-dispatch-router (qdrouterd)
with an attached apache artemis broker is used in place of the qpidd
broker. The qdrouterd will directly message oslo.messaging rpc
communications and will link route oslo.messaging notify
communications to the artemis broker (e.g. for persistence).
This patch:
* updates the dependencies
* updates the test environment setup
* uses the pifpaf drivers to create servers
Change-Id: I9fb56450346fca84bc1573fa5e9b6fe81c14e33a
|
|
|
|
|
|
|
|
| |
update the hacking to latest, duo to[1]
[1]: https://github.com/openstack/oslo.messaging/blob/master/HACKING.rst
Change-Id: I1ee7387272690f0de0d7b1937d7c2b0ee404063c
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch switches the kafka python client from kafka-python to
confluent-kafka due to documented threading issues with the
kafka-python consumer and the recommendation to use multiplrocessing.
The confluent-kafka client leverages the high performance librdkafka
C client and is safe for multiple thread use.
This patch:
* switches to confluent-kafka library
* revises consumer and producer message operations
* utilizes event.tpool method for confluent-kafka blocking calls
* updates unit tests
* adds kafka specific timeouts for functional tests
* adds release note
Depends-On: Ice374dca539b8ed1b1965b75379bad5140121483
Change-Id: Idfb9fe3700d882c8285c6dc56b0620951178eba2
|
|
|
|
|
|
|
|
| |
Users of the oslo.messaging RPC communications service must use
the rabbit ("rabbit://...") or AMQP 1.0 ("amqp://...") drivers.
Change-Id: If3474142f1fe99d41d7b4466061ed0e23ca38549
Closes-Bug: 1789259
|
|
|
|
|
|
|
|
|
|
| |
As the comment from Stephen [1],
after moving to stestr, the package should be in exact position
where testrepository is removed.
[1] https://review.openstack.org/#/c/579743/2/test-requirements.txt@16
Change-Id: Ib748e4f8ba1263f2c6475c8641c800b7df872b88
|
|
|
|
|
|
|
|
|
|
| |
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: I89b785c91738ab62e3f42da42c431b9df8ccfd70
|
|
|
|
|
|
|
|
|
|
|
| |
Move the requirements for the optional drivers to
test-requirements.txt and setup.cfg. The default rabbitmq driver's
dependencies should be the only hard requirements for the base
package.
Leaving ZeroMQ deps unchanged for now as it will be removed in Stein
Change-Id: I19dd699ccf87e43202ccefb99258fbaa9ea17b7e
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix the lower constraints to match the expected values.
We will manage the eventlet version using constraints now. See the
thread starting at
http://lists.openstack.org/pipermail/openstack-dev/2018-April/129096.html
for more details.
Change-Id: I61d5680182c53774713ee83808b1f6e7a4fba9ff
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
|
|
|
|
| |
Change-Id: I9500a0d27953881cfad24d5542356c539353d6a7
|
|
|
|
| |
Change-Id: I289f97bc931e865feea2beb7b78456f8020da8c6
|
|
|
|
| |
Change-Id: Ieee5fa811e69e7ae4f597ce361319758d8030bde
|
|
|
|
| |
Change-Id: Id5276f0b18005af895cad852bedfdc7d8eb9b8e8
|
|
|
|
| |
Change-Id: I1315cfd256afa17aaa3be3ae4a3a3f6ee5cbb564
|
|
|
|
| |
Change-Id: I674e086307ef8785244a14fd4e83ffaedc927eb9
|
|
|
|
| |
Change-Id: Ida4133ebeb4cc0cbfbd0e8ee62a52236dc4a0256
|
|
|
|
| |
Change-Id: I8e7c47dc199798a3490ebfffa0581cf74566f45b
|
|
|
|
| |
Change-Id: Ic3d9b9cbef0040512e37c8e99df7bb028e344bc9
|
|
|
|
| |
Change-Id: I3c492ddd176882989a8a083fb560ab5c5df2452f
|
|
|
|
| |
Change-Id: I9313f1d68f67c3839a261ac9661e85bbdb03b94e
|
|
|
|
| |
Change-Id: I8f6d464a259ce1afc83728ddced8e8803e04f657
|
|
|
|
| |
Change-Id: Ic860a75be324005d072159e30d84b4c55b198242
|
|
|
|
| |
Change-Id: Ic34a619440024e0f59f4e11fe6e248370995d5c7
|
|
|
|
| |
Change-Id: Ia0139a39a0328725f57c1364fcd9c8bd4a1a5d0c
|
|\ |
|
| |
| |
| |
| |
| |
| |
| | |
oslo.messaging does not import mox3 directly, it uses mox3 through
oslotest.
Change-Id: Iaae0b03d2c0511d12f1ab2c8e93622dfaa6c27ce
|
|/
|
|
|
|
|
|
|
| |
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: I29b64d2f8d6e3305d052562f160050fdd05c2a55
|
|
|
|
| |
Change-Id: I1a5882d3fe0189ab3e651155681a1410c37d8a82
|
|
|
|
| |
Change-Id: Ie304f1ed34156bfb60b9bc9acd4ad0aa5bad4657
|
|\ |
|
| |
| |
| |
| | |
Change-Id: Ie386dea5cc4b9ff02dc2b638d2d6908c3cdad883
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently Kafka driver for an oslo.messaging uses kafka-python==0.9.5
and mostly broken. This package version supports only low level Kafka
producer and consumer API which are marked as deprecated now [1]. Using
of these interfaces bring a big concern to the message processing,
because current KafkaConsumer has not any consuming coordination. This
fact causes a message duplication for the several consumers of one
topic. This behavior is specific to Ceilometer and services which read
and process notifications from other services.
New version of kafka-python allows to use async thread safe message
producers and coordinated consumers [1].
[1] http://kafka-python.readthedocs.io/en/master/changelog.html#feb-15-2016
The driver is currently experimental, python-kafka<1.0.0 API have major
issue described above that can't make the oslo.messaging driver works,
so we prefer having a working driver with a non-synced dependencies, that the
reverse.
Co-Authored-By: Mehdi Abaakouk <sileht@redhat.com>
Change-Id: I29862ed7bf56b9d8878fa8e9fb1cbd9d643908a4
|
|
|
|
| |
Change-Id: I1358feb2f87821e8a18ac8b77461df57fa2a6168
|
|
|
|
| |
Change-Id: I8dced43299478078afb75c71b187b7f0607276ad
|
|
|
|
| |
Change-Id: Ib121090566ec7765b52f22227e1ccfe2fe413aff
|
|
|
|
| |
Change-Id: I4cb1977760b9286bc7b66c0a30e602596a02637c
|
|\ |
|
| |
| |
| |
| |
| |
| | |
It's only needed for python < 2.7 which is not supported
Change-Id: Ic899f5f03e847c53c22dbe0b7ee02313648e4be8
|
| |
| |
| |
| | |
Change-Id: Iae11896324f16164fd62a53c089ebd2948437098
|
| |
| |
| |
| | |
Change-Id: I3d565fabb11b45c6420ed80c81504886b53b99c7
|
|\ \
| |/
|/| |
|
| |
| |
| |
| |
| |
| |
| | |
This leverages the pifpaf library to exercice
rabbitmq failover.
Change-Id: Ic6acc5d006344e08c219724e488fc9222786d849
|
| |
| |
| |
| | |
Change-Id: Ie47c04a5c439bd6303e50794909c2a81bc64e4d1
|
| |
| |
| |
| | |
Change-Id: I0cc8324da21fa246c936685048782ad2c2a94368
|