diff options
| author | James E. Blair <jim@acmegating.com> | 2021-07-02 11:38:58 -0700 |
|---|---|---|
| committer | James E. Blair <jim@acmegating.com> | 2021-07-02 15:22:29 -0700 |
| commit | 10966948d723ea75ca845f77d22b8623cb44eba4 (patch) | |
| tree | 2f0fcd1e721ee482a2f67bb00dada2e34f25825f /tools | |
| parent | 4f0ecff3a3d6f0cbe389e2ff78a682d73b4ef52a (diff) | |
| download | zuul-10966948d723ea75ca845f77d22b8623cb44eba4.tar.gz | |
Add ExistingDataWatch class
This adds a modified kazoo.DataWatch class which does not set
watches on paths that don't exist. This is mostly so that when
a DataWatched path is deleted, we don't leave a watch in place.
Without that, the executor api will leak watches on the ZK cluster.
We vendor the entire watches.py file from kazoo (ASL2 licensed) to
avoid any issues with API changes. Separately we will see if they
are interested in this class upstream.
This updates the tests to use the wchp 4lw to test that we don't
leak watches.
Depends-On: https://review.opendev.org/c/zuul/zuul-jobs/+/799334
Change-Id: Ie4491eef03b58d858d95c78ba9454839d169cff1
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/zoo.cfg | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tools/zoo.cfg b/tools/zoo.cfg index bf1ac6cf7..7d9f3cee4 100644 --- a/tools/zoo.cfg +++ b/tools/zoo.cfg @@ -14,3 +14,4 @@ serverCnxnFactory=org.apache.zookeeper.server.NettyServerCnxnFactory secureClientPort=2281 ssl.keyStore.location=/var/certs/keystores/zuul-test-zookeeper.pem ssl.trustStore.location=/var/certs/certs/cacert.pem +4lw.commands.whitelist=* |