| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| | |
|
| | |
|
| |
|
|
|
|
|
|
| |
Testing libs have been upgraded.
The used java classpath now considers more .jar possible locations.
test_connection.py tests is now done first, trying to make it less flaky.
conftest.py has been added to give a way to display ZK cluster logs.
Because of ZK 3.6 and 3.7 changes, configurations related to local session has been added.
Disable GA fail-fast because of the flakiness
|
| |
|
| |
closes #644 #662 #663
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
The lock must only consider contenders with a sequence number lower than
it's own sequence number as also stated in the Zookeeper recipe
description for shared locks[0].
This wasn't working correctly as the ReadLock also considered WriteLocks
with a higher sequence number as contenders. This can lead to a deadlock
as described in #649.
[0]: https://zookeeper.apache.org/doc/r3.7.0/recipes.html#Shared+Locks
Closes #649
|
| |
|
|
|
|
|
| |
threading,gevent,eventlet (#656)
Co-authored-by: lawrentwang <lawrentwang@tencent.com>
Solve the select limitation on a maximum file handler value and dynamic choose the best poller in the system.
|
| |\
| |
| | |
fix(recipe): fix deprecation warning from threading.Event
|
| |/
|
|
|
|
|
| |
reasoning:
- `is_set` is supported since Python2.6 (https://docs.python.org/2.7/library/threading.html)
- Starting from Python3.5 docs the `isSet` spelling is not even mentioned anymore (https://docs.python.org/3.5/library/threading.html)
- Python3.10 will give a deprecation warning when the `isSet` spelling is used (https://docs.python.org/3.10/library/threading.html)
|
| | |
|
| |\
| |
| | |
chore(ci): Replace Travis with Github Actions
|
| |/ |
|
| |\
| |
| | |
docs: fix simple typo, untill -> until
|
| | |\
| |/
|/| |
|
| | |
| |
| | |
DNS resolution errors were previously not retriable, this commit allows it by changing the value returned when the DNS resolution fails.
|
| |/
|
|
|
|
| |
There is a small typo in kazoo/recipe/queue.py.
Should read `until` rather than `untill`.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Without this patch, a single select event is processed by iteration in
the 'ConnectionHandler' event loop.
In a scenario where the client issues a large number of async requests
with an important amplification factor, e.g. 'get_children_async' on a
large node, it is possible for the 'select' operation to almost always
return a "response ready" socket--as the server is often able to
process, serialize and ship a new reponse while Kazoo processes the
previous one.
That response socket often (always?) ends up at the beginning of the
list returned by 'select'.
As only 'select_result[0]' is processed in the loop, this can cause
the client to ignore the "request ready" FD for a long time, during
which no requests or pings are sent.
In effect, asynchronously "browsing" a large tree of nodes can stretch
that duration to the point where it exceeds the timeout--causing the
client to lose its session.
This patch considers both descriptors after 'select', and also
arranges for pings to be sent in case it encounters an "unending"
stream of responses to requests which were sent earlier.
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
| |
This adds a simple recovery path in case an SSL connection receives
an SSL_WANT_READ or WRITE error. Either error can occur while
reading or writing. The error indicates that the underlying
operation should be retried after the socket is once again readable
or writable (per the error code).
Closes #618
Co-authored-by: James E. Blair <jeblair@redhat.com>
|
| |\
| |
| | |
Feat/build coverage
|
| | |\
| |/
|/| |
|
| |\ \
| | |
| | | |
chore(core): Update matrix: add pypy3, drop py27
|
| | |\ \
| |/ /
|/| | |
|
| | | |
| | |
| | |
| | |
| | |
| | | |
This is a faster and more idiomatic way of using itertools.chain.
Instead of computing all the items in the iterable and storing them
in memory, they are computed one-by-one and never stored as a huge
list. This can save on both runtime and memory space.
|
| | |/ |
|
| |/
|
|
|
|
| |
Update .travis.yml definition per warnings.
Tests are now only run on py37, py38 and pypy3.
Bumped Zookeeper to 3.5.8.
|
| |
|
| |
Test Kazoo with CPython 2.7, CPython 3.7 and CPython 3.8. Deploy new Kazoo version only on CPython 3.8 job success.
|
| |
|
|
|
|
| |
As of Python 3.8, "is" with a literal is a syntax warning because of the
confusion between equality and instance identity it represents.
Issue #607
|
| |
|
|
|
| |
Let's make our life a little easier using those templates.
See https://help.github.com/en/github/building-a-strong-community/about-issue-and-pull-request-templates for reference.
|
| |\
| |
| | |
feat(core): interoperate with Go client
|
| | |
| |
| |
| | |
Per https://github.com/houndci/hound/issues/1769
|
| | | |
|
| | |
| |
| |
| |
| | |
Allows configurable multi-implementations cooperations in locks (e.g.
Zookeeper python & go clients contending for the same lock).
|
| |/ |
|
| |\
| |
| | |
fix(core): sync() return should be unchrooted
|
| |/
|
|
| |
Resolves #601
|
| |\
| |
| | |
feat(test): Move to pytest
|
| |/ |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
With this patch, requests issued while the client is in the
'CONNECTING' state get queued instead of raising a misleading
'SessionExpiredError'.
This fixes https://github.com/python-zk/kazoo/issues/374, and brings
Kazoo more in line with the Java and C clients.
See the 'kazoo.client.KazooClient.state' documentation as well as
these discussions for more details:
https://github.com/python-zk/kazoo/pull/570#issuecomment-554798550
https://github.com/python-zk/kazoo/pull/583#issuecomment-586422386
|
| |
|
|
|
| |
Satisfy new Hound style/lint checks
Upgrade to latest 3.5.6/3.4.14 Zookeeper releases.
|
| |
|
|
| |
Since krb5 1.3 due to switch to Xenial.
|
| | |
|
| | |
|
| |
|
|
| |
ensure-zookeeper-env.sh is correct
|
| |
|
|
|
|
|
|
|
|
|
|
| |
* Install debian packages for KDC as part of Travis init.
* Setup a loopback mini KDC for running tests.
* Run SASL tests as part of Travis builds.
* Improve harness cluster to support:
* Reconfiguration when environment changes.
* Different JAAS configurations (DIGEST/GSSAPI).
* Moved SASL tests into own module, with specially configured harness.
* Bumped default timeout to 15 sec to mitigate false negatives on
Travis.
|
| |\
| |
| | |
feat(core): Add create2 support
|
| |/ |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
New retry logic takes a maximum percentage off the canonical backoff,
ensure gradual predictable retries timings while still having a
controlable amount of jitter (re-introducing the `max_jitter` parameter)
to avoids swarming client retries.
Fix regression introduced in 60366d2c7910fc833991fad8e04bbe33817c0544
where retry/backoff logic produced only whole second (integer) retry
delays.
This produced inadequate retries on first retry and would generally not
work on fast network where sub miliseconds retries are desired.
Additionally, with high `max_delay` setting, as the range was always
spanning from 0 until the last delay, it would also produce extremely
random results with short delays following longer ones which is contrary
to the expected backoff logic.
|
| |
|
|
|
|
|
|
|
| |
When connection attempts fail repeatedly (e.g. all ZK servers are
unavailable), eventually the socketpair in the ConnectionHandler fills
up, and the Client gets stuck trying to write a single byte to the
socketpair.
Avoid this by ensuring we close the socketpair on a failed connection
attempt.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Switching to xenial distrib to make python 3.7 available
* Testing against 3.5.5 instead of 3.5.4-beta and deploying new version for python 3.7 and zk 3.5.5
* Testing against zk 3.4.14 instead of 3.4.13
* Change pattern for slf4j-log4j lib
* Since ZOOKEEPER-3156 it is now required to be authed to access ACLs
* Drop support for ZK 3.3.x and Python 3.{4,5}
|
