Thanks a lot Enrico, great work so far. If you’re feeling having hard a time as a RM, you’re definitely on the right track. :)
Referring 3.6.0 as a “major” release made me thinking a bit. This release is definitely a big leap in ZK’s history, but strictly based on the versioning it is “only” a minor release: version number consists of major.minor.maintenance parts. https://cwiki.apache.org/confluence/display/ZOOKEEPER/ReleaseManagement Andor > On 2020. Jan 16., at 14:31, Enrico Olivelli <eolive...@gmail.com> wrote: > > Thank you Patrick and Andor > > > > Il giorno gio 16 gen 2020 alle ore 14:18 Andor Molnar <an...@apache.org> ha > scritto: > >> Hi, >> >> 1) I think TLD “parent-3.6” is a consequence of the following rename: >> https://github.com/apache/zookeeper/pull/892 >> >> Which was done due to enabling us to rename ‘zookeeper-server’ to >> ‘zookeeper’ and keep the legacy Maven structure. I suspect 2 Maven projects >> cannot have the same name, so we probably need something other than >> “zookeeper”. In previous releases the TLD was ‘apache-zookeeper-X.X.X’, so >> maybe we could rename the top level project to “apache-zookeeper”. >> > > I will try to fix this personally. > I will create a JIRA and deliver the fix for the next RC. > If I can't fix it with the zip file I will resume the source tarball > > >> >> 2) “lib” tarball >> I think we’ve already talked about releasing C binaries and I had always >> been against it. These libraries are not portable and unless we release >> separate artifacts for all major distributions (including Windows?), I >> don’t see the point of introducing it. Plus the things that Patrick >> mentioned, I strongly believe that we should remove it from the release. >> > > From this comment and from Patrick's email it is better to remove this part > for 3.6, we can continue the discussion for the next Major release. > My plan is to simply NOT include the apache-zookeeper-3.6.0-lib.tar.gz in > the next VOTE rc > > For me having the binaries is a +0, I am not sure how users will benefit > from them, > maybe it would be better to have some deployment of ZooKeeper C lib in > public repositories, but I really don't know how it works. > > > >> >> 3) “test23” >> I haven’t checked that yet, but that’s related to the multi-address patch >> which was added recently. Mate and I can take a look. >> > > Regarding the 'test23' failure I am able to reproduce exactly Patricks's > problem. > Thank you Mate and Andor for your time, I am not able to dig into the cause > this week. > > > Please consider this VOTE thread closed. > > I will leave the staged artifact available on my people.apache.org web > space and on repositories.apache.org, in case people want to test the > client and compatibility with applications > built for 3.5.x > > This is a major release and I think it is expected to have more iterations > over the release > > Best regards > Enrico > > > > > >> >> Regards, >> Andor >> >> >> >> >>> On 2020. Jan 16., at 4:45, Patrick Hunt <ph...@apache.org> wrote: >>> >>> wrt the addition of lib - we've had to move to explicitly telling users >>> _not_ to use the client binaries except as learning aids >>> >>> "NOTE: the ZooKeeper C client shell (cli_st and cli_mt) is meant as a >>> example/sample of ZooKeeper C client API usage. It is not a full >>> fledged client and not meant for production usage - see the Java >>> client shell for a fully featured shell." >>> >>> due to repeated security issues being raised and attempting to reduce the >>> burden security reports impose - the libs are fine, but the C clients >>> themselves were not created with security in mind and users should use >> the >>> Java CLIs instead. We could fix that, I assume, but afaik so far that >>> hasn't been done (harden the c cli/load binaries). >>> >>> We shouldn't include the bin/* IMO, although having an artifact for the >>> libs themselves is fine. >>> >>> >>> I'm sure we all know this, but I also want to clarify that the _source_ >> is >>> what we are releasing - the bin/lib/etc... are all "conveniences" >>> https://www.apache.org/dev/release-publishing.html#valid >>> language such as "we are releasing a new binary package with the C client >>> (with OpenSSL support)" concerns me a bit as the binaries are not the >>> release. Sorry for the nit picking but it is important for the community >> to >>> follow and understand established Apache guidance on this. >>> >>> Patrick >>> >>> >>> On Wed, Jan 15, 2020 at 7:31 PM Patrick Hunt <ph...@apache.org> wrote: >>> >>>> Unzipping the source on mac "unzip apache-zookeeper-3.6.0.zip" results >> in >>>> a tld of "parent-3.6.0" - can we fix this? The other artifacts >> de-archive >>>> correctly, expect this one. It seems a shame. Given this issue and the >>>> tar.gz->zip - perhaps we should reconsider and just generate the source >>>> ourselves? Or is there a way to ensure the tld name is correct? >>>> (setting/config?) >>>> >>>> Patrick >>>> >>>> On Wed, Jan 15, 2020 at 8:05 AM Enrico Olivelli <eolive...@gmail.com> >>>> wrote: >>>> >>>>> Alexander, >>>>> I have pasted a wrong link in the VOTE email, I am sorry >>>>> >>>>> The good link is >>>>> >>>>> >> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12310801&version=12326518 >>>>> >>>>> You can also see the staged released notes in the website >>>>> >>>>> >> https://people.apache.org/~eolivelli/zookeeper-3.6.0-candidate-0/website/releasenotes.html >>>>> >>>>> Thank you so much for reporting this issue >>>>> >>>>> Happy testing >>>>> >>>>> Enrico >>>>> >>>>> Il giorno mer 15 gen 2020 alle ore 16:55 Alexander Shraer < >>>>> shra...@gmail.com> >>>>> ha scritto: >>>>> >>>>>> Hi Enrico, >>>>>> >>>>>> Thank you for driving this release! >>>>>> >>>>>> I have a question - i believe that Zookeeper-2024 (an order of >> magnitude >>>>>> throughput improvement for mixed workloads) is part of the 3.6.0 >>>>> release, >>>>>> but it isn't mentioned in the release notes or the summary. >>>>>> Could you please clarify ? >>>>>> >>>>>> Thanks, >>>>>> Alex >>>>>> >>>>>> >>>>>> On Wed, Jan 15, 2020 at 7:29 AM Flavio Junqueira <f...@apache.org> >>>>> wrote: >>>>>> >>>>>>> I can't parse Rudy's message, is it an issue with my mail >> application? >>>>>>> >>>>>>> -Flavio >>>>>>> >>>>>>>> On 15 Jan 2020, at 15:00, rudy_steiner <rudy_stei...@163.com> >>>>> wrote: >>>>>>>> >>>>>>>> environment: <br/> * MacOS High Sierra 10.13.1 <br/> * JDK >>>>>>> 1.8.0_172<br/><br/>I try to run junit test on branch-3.6, and unit >>>>> test >>>>>>> thread get stuck, log as follows:<br/><br/>.....<br/>INFO] Running >>>>>>> org.apache.zookeeper.common.X509UtilTest<br/>[INFO] Tests run: 3, >>>>>> Failures: >>>>>>> 0, Errors: 0, Skipped: 0, Time elapsed: 27.797 s - in >>>>>>> org.apache.zookeeper.server.SnapshotDigestTest<br/>[INFO] Running >>>>>>> org.apache.zookeeper.common.TimeTest<br/>[INFO] Tests run: 1, >>>>> Failures: >>>>>> 0, >>>>>>> Errors: 0, Skipped: 0, Time elapsed: 0.718 s - in >>>>>>> org.apache.zookeeper.common.TimeTest<br/>[INFO] Tests run: 352, >>>>> Failures: >>>>>>> 0, Errors: 0, Skipped: 0, Time elapsed: 7.425 s - in >>>>>>> org.apache.zookeeper.common.X509UtilTest<br/>[INFO] Running >>>>>>> org.apache.zookeeper.common.PEMFileLoaderTest<br/>[INFO] Running >>>>>>> org.apache.zookeeper.common.KeyStoreFileTypeTest<br/>[INFO] Tests >>>>> run: 9, >>>>>>> Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.144 s - in >>>>>>> org.apache.zookeeper.common.KeyStoreFileTypeTest<br/>[INFO] Running >>>>>>> org.apache.zookeeper.audit.AuditEventTest<br/>[INFO] Tests run: 2, >>>>>>> Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.084 s - in >>>>>>> org.apache.zookeeper.audit.AuditEventTest<br/>[INFO] Running >>>>>>> org.apache.zookeeper.audit.StandaloneServerAuditTest<br/>[INFO] Tests >>>>>> run: >>>>>>> 72, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.027 s - in >>>>>>> org.apache.zookeeper.common.PEMFileLoaderTest<br/>[INFO] Tests run: >> 5, >>>>>>> Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 16.197 s - in >>>>>>> org.apache.zookeeper.common.FileChangeWatcherTest<br/>[INFO] Tests >>>>> run: >>>>>> 1, >>>>>>> Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.755 s - in >>>>>>> org.apache.zookeeper.audit.StandaloneServerAuditTest<br/>[INFO] >>>>> Running >>>>>>> org.apache.zookeeper.audit.Log4jAuditLoggerTest<br/>[INFO] Running >>>>>>> org.apache.zookeeper.ZKUtilTest<br/>[ERROR] Tests run: 4, Failures: >> 1, >>>>>>> Errors: 0, Skipped: 0, Time elapsed: 0.194 s <<< FAILURE! - >>>>> in >>>>>>> org.apache.zookeeper.ZKUtilTest<br/>[ERROR] >>>>>>> testUnreadableFileInput(org.apache.zookeeper.ZKUtilTest) Time >>>>> elapsed: >>>>>>> 0.014 s <<< FAILURE!<br/>java.lang.AssertionError<br/> >>>>> at >>>>>>> >>>>>> >>>>> >> org.apache.zookeeper.ZKUtilTest.testUnreadableFileInput(ZKUtilTest.java:83)<br/><br/>[INFO] >>>>>>> Running org.apache.zookeeper.PortAssignmentTest<br/>[INFO] Tests run: >>>>> 13, >>>>>>> Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.157 s - in >>>>>>> org.apache.zookeeper.PortAssignmentTest<br/>[INFO] Running >>>>>>> org.apache.zookeeper.VerGenTest<br/>[INFO] Tests run: 6, Failures: 0, >>>>>>> Errors: 0, Skipped: 0, Time elapsed: 1.747 s - in >>>>>>> org.apache.zookeeper.audit.Log4jAuditLoggerTest<br/>[INFO] Tests run: >>>>> 14, >>>>>>> Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.327 s - in >>>>>>> org.apache.zookeeper.VerGenTest<br/>[INFO] Running >>>>>>> org.apache.zookeeper.ZooKeeperTest<br/>[INFO] Running >>>>>>> org.apache.zookeeper.GetAllChildrenNumberTest<br/>[INFO] Running >>>>>>> org.apache.zookeeper.RemoveWatchesCmdTest<br/>[INFO] Tests run: 2, >>>>>>> Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.511 s - in >>>>>>> org.apache.zookeeper.GetAllChildrenNumberTest<br/>[INFO] Running >>>>>>> org.apache.zookeeper.ClientRequestTimeoutTest<br/>[INFO] Tests run: >> 7, >>>>>>> Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 4.004 s - in >>>>>>> org.apache.zookeeper.RemoveWatchesCmdTest<br/>[INFO] Running >>>>>>> org.apache.zookeeper.ClientCanonicalizeTest<br/>[INFO] Tests run: 4, >>>>>>> Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.861 s - in >>>>>>> org.apache.zookeeper.ClientCanonicalizeTest<br/>[INFO] Running >>>>>>> org.apache.zookeeper.client.ZKClientConfigTest<br/>[INFO] Tests run: >>>>> 5, >>>>>>> Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.155 s - in >>>>>>> org.apache.zookeeper.client.ZKClientConfigTest<br/>[INFO] Tests run: >>>>> 35, >>>>>>> Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 10.74 s - in >>>>>>> org.apache.zookeeper.ZooKeeperTest<br/>[INFO] Tests run: 1, Failures: >>>>> 0, >>>>>>> Errors: 0, Skipped: 0, Time elapsed: 16.372 s - in >>>>>>> org.apache.zookeeper.ClientRequestTimeoutTest<br/>[INFO] Tests run: >>>>> 46, >>>>>>> Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 61.592 s - in >>>>>>> org.apache.zookeeper.RemoveWatchesTest<br/>[INFO] Tests run: 24, >>>>>> Failures: >>>>>>> 0, Errors: 0, Skipped: 0, Time elapsed: 166.152 s - in >>>>>>> org.apache.zookeeper.server.quorum.QuorumPeerMainTest<br/>[INFO] >> Tests >>>>>> run: >>>>>>> 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 158.386 s - in >>>>>>> org.apache.zookeeper.server.quorum.ReconfigRecoveryTest<br/>[INFO] >>>>> Tests >>>>>>> run: 13, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 416.635 s >>>>> - in >>>>>>> org.apache.zookeeper.server.quorum.QuorumSSLTest<br/>..... >>>>>>> <br/><br/><br/>and I found serveral processes by ps -ef|grep java >>>>>>> :<br/><br/> 0 6809 87919 0 9:28下午 ?? 2:13.75 >>>>>>> >>>>> >> /Library/Java/JavaVirtualMachines/jdk1.8.0_172.jdk/Contents/Home/bin/java >>>>>>> -agentlib:jdwp=transport=dt_socket,address=127.0.0.1:62202 >>>>>> ,suspend=y,server=n >>>>>>> -Dvisualvm.id=962684532457553 >>>>>>> -Dmaven.multiModuleProjectDirectory=/export/workspace/zookeeper >>>>>>> -Dmaven.home=/Applications/IntelliJ >>>>>>> IDEA.app/Contents/plugins/maven/lib/maven3 >>>>>>> -Dclassworlds.conf=/Applications/IntelliJ >>>>>>> IDEA.app/Contents/plugins/maven/lib/maven3/bin/m2.conf >>>>>>> >>>>>> >>>>> >> -javaagent:/var/root/Library/Caches/IntelliJIdea2018.2/captureAgent/debugger-agent.jar=file:/private/var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/capture.props >>>>>>> -Dfile.encoding=UTF-8 -classpath /Applications/IntelliJ >>>>>>> >>>>>> >>>>> >> IDEA.app/Contents/plugins/maven/lib/maven3/boot/plexus-classworlds-2.5.2.jar:/Applications/IntelliJ >>>>>>> IDEA.app/Contents/lib/idea_rt.jar org.codehaus.classworlds.Launcher >>>>>>> -Didea.version=2018.2.4 test<br/> 0 11582 6809 0 9:35下午 ?? >>>>>>> 0:00.01 /bin/sh -c cd /export/workspace/zookeeper/zookeeper-server >>>>>>> && >>>>>>> >>>>>> >>>>> >> /Library/Java/JavaVirtualMachines/jdk1.8.0_172.jdk/Contents/Home/jre/bin/java >>>>>>> -Xmx512m -Dtest.junit.threads=8 -Dzookeeper.junit.threadid=5 >>>>>>> >>>>>> >>>>> >> -javaagent:/var/root/.m2/repository/org/jmockit/jmockit/1.48/jmockit-1.48.jar >>>>>>> -jar >>>>>>> >>>>>> >>>>> >> /export/workspace/zookeeper/zookeeper-server/target/surefire/surefirebooter3509879420266754639.jar >>>>>>> /export/workspace/zookeeper/zookeeper-server/target/surefire >>>>>>> 2020-01-15T21-28-59_364-jvmRun5 surefire3587429514371015586tmp >>>>>>> surefire_2145165207885893729779tmp<br/> 0 11583 11582 0 9:35下午 >>>>> ?? >>>>>>> 0:05.16 >>>>>>> >>>>>> >>>>> >> /Library/Java/JavaVirtualMachines/jdk1.8.0_172.jdk/Contents/Home/jre/bin/java >>>>>>> -Xmx512m -Dtest.junit.threads=8 -Dzookeeper.junit.threadid=5 >>>>>>> >>>>>> >>>>> >> -javaagent:/var/root/.m2/repository/org/jmockit/jmockit/1.48/jmockit-1.48.jar >>>>>>> -jar >>>>>>> >>>>>> >>>>> >> /export/workspace/zookeeper/zookeeper-server/target/surefire/surefirebooter3509879420266754639.jar >>>>>>> /export/workspace/zookeeper/zookeeper-server/target/surefire >>>>>>> 2020-01-15T21-28-59_364-jvmRun5 surefire3587429514371015586tmp >>>>>>> surefire_2145165207885893729779tmp<br/><br/><br/>I suspect the >>>>> processes >>>>>>> for unit test not close correctly, lead successor test stuck >>>>>>>> At 2020-01-15 07:18:25, "Enrico Olivelli" <eolive...@gmail.com> >>>>> wrote: >>>>>>>>> This is a release candidate for 3.6.0. >>>>>>>>> >>>>>>>>> It is a major release and it introduces a lot of new features, most >>>>>>> notably: >>>>>>>>> - Built-in data consistency check inside ZooKeeper >>>>>>>>> - Allow Followers to host Observers >>>>>>>>> - A new feature proposal to ZooKeeper: authentication enforcement >>>>>>>>> - Pluggable metrics system for ZooKeeper (and Prometheus.io >>>>>> integration) >>>>>>>>> - TLS Port unification >>>>>>>>> - Audit logging in ZooKeeper servers >>>>>>>>> - Improve resilience to network (advertise multiple addresses for >>>>>>> members >>>>>>>>> of a Zookeeper cluster) >>>>>>>>> - Persistent Recursive Watches >>>>>>>>> - add an API and the corresponding CLI to get total count of >>>>> recursive >>>>>>> sub >>>>>>>>> nodes under a specific path >>>>>>>>> >>>>>>>>> The full release notes is available at: >>>>>>>>> >>>>>>>>> >>>>>>> >>>>>> >>>>> >> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12310801&version=12346617 >>>>>>>>> >>>>>>>>> *** Please download, test and vote by January 19th 2020, 23:59 >>>>> UTC+0. >>>>>>> *** >>>>>>>>> >>>>>>>>> Source files: >>>>>>>>> https://people.apache.org/~eolivelli/zookeeper-3.6.0-candidate-0/ >>>>>>>>> >>>>>>>>> Maven staging repo: >>>>>>>>> >>>>>>> >>>>>> >>>>> >> https://repository.apache.org/content/repositories/orgapachezookeeper-1045/ >>>>>>>>> >>>>>>>>> The release candidate tag in git to be voted upon: release-3.6.0-0 >>>>>>>>> https://github.com/apache/zookeeper/tree/release-3.6.0-0 >>>>>>>>> >>>>>>>>> ZooKeeper's KEYS file containing PGP keys we use to sign the >>>>> release: >>>>>>>>> https://www.apache.org/dist/zookeeper/KEYS >>>>>>>>> >>>>>>>>> Please note that this new major release introduces these change in >>>>> the >>>>>>>>> staging area: >>>>>>>>> - the source artifacts package is in zip format >>>>>>>>> - we are releasing a new binary package with the C client (with >>>>>> OpenSSL >>>>>>>>> support) >>>>>>>>> - website preview >>>>>>>>> >>>>>>>>> We are also adding a new JAR: zookeeper-metrics-providers >>>>>>>>> >>>>>>>>> The staging version of the website is: >>>>>>>>> >>>>>>> >>>>>> >>>>> >> https://people.apache.org/~eolivelli/zookeeper-3.6.0-candidate-0/website/ >>>>>>>>> >>>>>>>>> Should we release this candidate? >>>>>>>>> >>>>>>>>> Enrico Olivelli >>>>>>> >>>>>>> >>>>>> >>>>> >>>> >> >>