summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* feat: pep8 all the thingsfeat/issue-445Ben Bangert2017-06-0119-87/+160
| | | | | | Brings the code-base back up to validating flake8 checks. Closes #445
* Merge pull request #443 from python-zk/feat/issue-441Ben Bangert2017-06-013-29/+12
|\ | | | | feat: drop Python 2.6/3.3 official support, add 3.5/3.6 testing
| * feat: drop Python 2.6/3.3 official support, add 3.5/3.6 testingBen Bangert2017-06-013-29/+12
|/ | | | Closes #441
* Merge pull request #440 from python-zk/release/2.3Ben Bangert2017-06-013-3/+22
|\ | | | | Release/2.3
| * Merge branch 'release/2.3' of github.com:python-zk/kazoo into release/2.32.3.1release/2.3Ben Bangert2017-06-011-2/+0
| |\
| | * chore: tag 2.3.0 release2.3.0Ben Bangert2017-05-318-782/+811
| | |
| * | chore: tag 2.3.1 releaseBen Bangert2017-06-013-3/+24
|/ /
* | Merge pull request #438 from mrterry/fix_manifestBen Bangert2017-06-011-3/+3
|\ \ | | | | | | chore: update MANIFEST.in to reflect changes to filenames
| * | chore: update MANIFEST.in to reflect changes to filenamesMatt Terry2017-06-011-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | Looks like the .rst files got converted to .md, but MANIFEST.in wasn't updated. This also fixes the sdist, since setup.py expects README.md & CONTRIBUTING.md to be in the source tarball. closes #437
* | | Merge pull request #439 from python-zk/feat/issue-436Ben Bangert2017-06-013-54/+61
|\ \ \ | |/ / |/| | chore: add travis deployment and update ZK versions
| * | chore: add travis deployment and update ZK versionsBen Bangert2017-06-013-54/+61
|/ / | | | | | | | | | | | | Add's automatic PyPI release based on tags. The versions of Zookeeper are now the latest stable releases for 3.4. Closes #436
* | chore: tag 2.3.0 releaseBen Bangert2017-05-318-782/+811
|/
* Revert "New recipe proposal: TreeCache"Ben Bangert2017-05-314-694/+0
|
* Merge pull request #430 from anskornyakov/hotfix/semaphore-deadlockBen Bangert2017-05-311-3/+3
|\ | | | | fix deadlock in nonblocking semaphore acquiring
| * fix deadlock in nonblocking semaphore acquiring, do inner lock release for ↵Anton Skornyakov2017-04-201-3/+3
| | | | | | | | every lock acquiring attempt
* | Merge pull request #418 from josephaltmaier/masterBen Bangert2017-05-311-2/+4
|\ \ | | | | | | Always call _safe_close() on KazooClient.stop() so we don't leak work…
| * | Always call _safe_close() on KazooClient.stop() so we don't leak worker threadsjoseph2017-03-011-2/+4
| | |
* | | Merge pull request #400 from tonyseek/hotfix/xid-mismatchBen Bangert2017-05-312-2/+75
|\ \ \ | | | | | | | | Fix the client.add_auth hangs by xids mismatch.
| * | | Fix the client.add_auth hangs by xids mismatch.Jiangge Zhang2016-07-292-2/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit fixes https://github.com/python-zk/kazoo/issues/229 by throwing the runtime exception into the dequeued async_result. It will end the waiting of user threading. But this commit doesn't fix the xids mismatch itself. The unordered xids may caused by a bug from the ZooKeeper server side, such as the official issues https://issues.apache.org/jira/browse/ZOOKEEPER-1863 described.
* | | | Merge pull request #398 from tonyseek/feature/treecache-recipeBen Bangert2017-05-314-0/+694
|\ \ \ \ | | | | | | | | | | New recipe proposal: TreeCache
| * | | | Fix the infinite refresh after root node deletedqifei.wan2017-04-242-3/+7
| | | | |
| * | | | Closed tree cache could not be started againJiangge Zhang2017-04-172-1/+13
| | | | |
| * | | | Rename "was_xxx" to "on_xxx" in TreeCacheJiangge Zhang2017-02-252-16/+16
| | | | | | | | | | | | | | | | | | | | https://github.com/python-zk/kazoo/pull/398#discussion_r90782660
| * | | | Use KazooException instead of RuntimeError in TreeCacheJiangge Zhang2017-02-252-3/+4
| | | | | | | | | | | | | | | | | | | | https://github.com/python-zk/kazoo/pull/398#discussion_r90782555
| * | | | Add default logger if no listener is present.Jiangge Zhang2016-12-051-7/+9
| | | | |
| * | | | Implement and test the TreeCache recipe.Jiangge Zhang2016-06-234-0/+675
| |/ / /
* | | | Merge pull request #397 from paperlessreceipts/counter-pre-post-valuesBen Bangert2017-05-312-2/+23
|\ \ \ \ | | | | | | | | | | Counter: expose the previous and changed values.
| * | | | Counter: expose the previous and changed values.Lukasz Kawczynski2016-06-102-2/+23
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit extends the `Counter` API with the `pre_value` and `post_value` attributes. These attributes are set when a counter is changed. Previously, if the counter was modified concurrently, it was not possible to access the exact value to which the counter was set after a successful increment or decrement.
* | | | Merge pull request #396 from adamchainz/readthedocs.ioBen Bangert2017-05-312-2/+2
|\ \ \ \ | | | | | | | | | | Convert readthedocs link for their .org -> .io migration for hosted projects
| * | | | Convert readthedocs link for their .org -> .io migration for hosted projectsAdam Chainz2016-05-292-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As per [their blog post of the 27th April](https://blog.readthedocs.com/securing-subdomains/) ‘Securing subdomains’: > Starting today, Read the Docs will start hosting projects from subdomains on the domain readthedocs.io, instead of on readthedocs.org. This change addresses some security concerns around site cookies while hosting user generated data on the same domain as our dashboard. Test Plan: Manually visited all the links I’ve modified.
* | | | | Merge pull request #391 from michielbaird/watcher_changesBen Bangert2017-05-314-2/+40
|\ \ \ \ \ | | | | | | | | | | | | Ensure pending watches are not dropped when connection is lost
| * | | | | Added test add fixed a regressionMichiel Johan Baird2016-04-203-2/+20
| | | | | |
| * | | | | Ensure watches are notified when connection state is lostMichiel Johan Baird2016-04-202-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - This behaviour matches the reference implementation in Zookeeper - New EventType added: NONE - Path defaults to None as in reference implementation
* | | | | | Merge pull request #369 from FedericoCeratto/patch-1Ben Bangert2017-05-311-2/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | Set default max_delay to one minute
| * | | | | | Set default max_delay to one minuteFederico Ceratto2015-11-301-2/+2
| | | | | | |
* | | | | | | Merge pull request #365 from radix/construct-exceptions-with-saner-argumentsBen Bangert2017-05-312-4/+7
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | construct empty exceptions with no arguments, instead of () and {}
| * | | | | | | add test for new exception construction behaviorChristopher Armstrong2015-11-131-0/+6
| | | | | | | |
| * | | | | | | construct empty exceptions with no arguments, instead of () and {}Christopher Armstrong2015-11-131-4/+1
| | | | | | | |
* | | | | | | | Merge pull request #359 from rockerbox/queue-patchBen Bangert2017-05-311-7/+3
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Fixed Queue to clear children before retry
| * | | | | | | | made recipe.queue clear children on retrypatrickotoole2015-09-181-7/+3
| | | | | | | | |
* | | | | | | | | Merge pull request #354 from d3matt/FIX/potential_leakBen Bangert2017-05-311-0/+2
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Fix potential leak in DataWatch and ChildrenWatch
| * | | | | | | | | Fix potential leak in DataWatch and ChildrenWatchMatthew Stoltenberg2015-08-191-0/+2
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the func passed to DataWatcher or ChildrenWatch is bound to an object it will keep a reference to the object around which could prevent it from getting garbage collected.
* | | | | | | | | Merge pull request #349 from harlowja/use-more-sixBen Bangert2017-05-311-6/+3
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Remove code not needed when six is useable
| * | | | | | | | | Remove code not needed when six is useableJoshua Harlow2015-07-231-6/+3
| |/ / / / / / / /
* | | | | | | | | Merge pull request #348 from rgs1/allow-override-cluster-size-plus-observersBen Bangert2017-05-313-6/+17
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Allow override cluster size plus observers
| * | | | | | | | | Allow having observers in the cluster via ZK_OBSERVER_START_IDRaul Gutierrez S2015-07-113-7/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For many testing scenarios, it's convenient to have Observers (specially with all the upcoming reconfig features in 3.5.0). This is now possible by mixing two environment variables: ZOOKEEPER_CLUSTER_SIZE: overrides the cluster's size ZOOKEEPER_OBSERVER_START_ID: servers with ids >=, will be observers For example, a cluster of size 5 with 3 participants and 2 observers could be build like this: ``` import os from kazoo.testing.harness import get_global_cluster os.environ["ZOOKEEPER_CLUSTER_SIZE"] = "5" os.environ["ZOOKEEPER_OBSERVER_START_ID"] = "4" get_global_cluster().start() ``` This works both with ZK 3.4 and >= 3.5.0. Signed-off-by: Raul Gutierrez S <rgs@itevenworks.net>
| * | | | | | | | | Allow setting ZooKeeperCluster's size via ZK_CLUSTER_SIZERaul Gutierrez S2015-07-111-0/+2
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Raul Gutierrez S <rgs@itevenworks.net>
* | | | | | | | | Merge pull request #419 from jeblair/shared-locksBen Bangert2017-05-314-8/+153
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Implement read-write (shared) lock from ZooKeeper recipe
| * | | | | | | | | Set watch on immediate predecessor in locksJames E. Blair2017-03-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The original lock code would set the watch on the last node which preceded our node. This is efficient because if there are N predecessors, then we would only set the watch on the latest and would therefore only wake up and go through the loop after all the predecessors had released their locks. The change to support shared locks, however, set the watch on the first node which preceded ours, meaning that we will wake up and iterate through the loop for all N predecessor nodes. This patch restores the original behavior where the watch was only set on the immediate predecessor.
| * | | | | | | | | Fix typo in ReadLock exampleJames E. Blair2017-03-111-1/+1
| | | | | | | | | |