+1 (non-binding) Thank you Arshad, nice job!
My checks: - I built the java source code and run all the unit tests on MacOS (using OpenJDK 8u212 and maven 3.6.3) - checkstyle and spotbugs passed - apache-rat passed - owasp (CVE check) passed - fatjar built - I built the the C and zkPython code (also run the zkPython and C-client tests successfully) in docker using Ubuntu 18.04.3 with OpenJDK 11.0.10 and maven 3.6.0. - I executed quick rolling-upgrade tests (using https://github.com/symat/zk-rolling-upgrade-test): - from 3.5.9 to the current RC (3.6.3) - from the current RC (3.6.3) to 3.7.0 Kind regards, Mate On Mon, Apr 12, 2021 at 1:00 PM Norbert Kalmar <[email protected]> wrote: > +1 (non-binding) > > Checked on macOS: licenses, build and test passed, ran ZK and few standard > commands, signature. > Also built and run unit tests on ubuntu. > > Thanks Arshad! > > - Norbert > > On Sat, Apr 10, 2021 at 10:34 AM Damien Diederen <[email protected]> > wrote: > > > > > Hi Arshad, all, > > > > LGTM! +1 (advisory): > > > > * Tarball contents match repository tag; > > > > * Verified checksums and signatures; > > > > * Built and smoke-tested on NixOS with a slightly adapted version of > > the Nix recipe and test case; > > > > * Smoke-tested a standalone server with the (corresponding) Java, C > > and Perl clients, as well as the zkfuse contrib; > > > > * Smoke-tested a 3-ensemble with the (corresponding) Java client and > > SASL/GSSAPI. > > > > Sorry if I made you feel you had to go around with RC 0! Oh well, at > > least we got some additional CVE fixes bundled in. > > > > Best, -D > > > > P.-S. — As with my previous review, I have *not* tested under Ubuntu nor > > used the staging repo. These are important points, but I figured most > > other testers would focus on them. > > > > > > > > Mohammad Arshad <[email protected]> writes: > > > This is a bug fix release candidate for 3.6.3. It fixes 52 issues, > > > including multiple CVE fixes. > > > > > > The full release notes is available at: > > > > > > https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12310801&version=12348703 > > > > > > **** Please download, test and vote by Sunday, April 11th 2021, 23:59 > > > UTC+0. **** > > > > > > Source and binary files: > > > https://people.apache.org/~arshad/zookeeper-3.6.3-candidate-2/ > > > > > > Maven staging repo: > > > > > > https://repository.apache.org/content/repositories/orgapachezookeeper-1071 > > > > > > The release candidate tag in git to be voted upon: release-3.6.3-2 > > > https://github.com/apache/zookeeper/tree/release-3.6.3-2 > > > > > > ZooKeeper's KEYS file containing PGP keys we use to sign the release: > > > https://www.apache.org/dist/zookeeper/KEYS > > > > > > The staging version of the website is: > > > https://people.apache.org/~arshad/zookeeper-3.6.3-candidate-2/website/ > > > > > > Should we release this candidate? > > > > > > Thanks & Regards > > > Arshad > > >
