Re: Test plan for ZK-1045 - Call for volunteers

2016-11-18 Thread Patrick Hunt
As Flavio said originally on this thread this is a big change. Based on the current status of the patch and the testing feedback it seems like we've done significant work to ensure the quality of the change. Do folks feel that there has been sufficient review/testing that we can commit this and cut

Re: [VOTE] move Apache Zookeeper to git

2016-11-18 Thread Patrick Hunt
> > > > Alternatively, the python systems I have dealt with put credentials for > > database access, etc, in configuration -- sometimes hidden -- files (say, > > .env), but yet it is clear text stored. > > > > Anyone has other suggestions? > > > > Eddie

Upcoming 3.4/3.5 releases.

2016-11-23 Thread Patrick Hunt
Hi folks. Now that ZOOKEEPER-2014 has been committed I think we should cut a 3.5.3-alpha release. We're getting close to beta with that one finalized. There are about 10 blockers, although I have not recently gone through and triaged them https://issues.apache.org/jira/secure/Dashboard.jspa?selectP

Re: Upcoming 3.4/3.5 releases.

2016-11-23 Thread Patrick Hunt
Forgot to mention. These will be our first releases using git rather than svn. Does anyone know of remaining issues we need to resolve related to this prior to cutting a release? Patrick On Wed, Nov 23, 2016 at 9:38 AM, Patrick Hunt wrote: > Hi folks. Now that ZOOKEEPER-2014 has been commit

Re: Review Request 47354: ZOOKEEPER-1045 : Quorum mutual authentication using SASL mechanism

2016-11-29 Thread Patrick Hunt
/QuorumCnxManager.java (line 449) <https://reviews.apache.org/r/47354/#comment227801> Doesn't this change change the semantics of SID? - Patrick Hunt On Nov. 25, 2016, 7:28 a.m., Rakesh R wrote: > > --- > This is an automatica

Re: Test plan for ZK-1045 - Call for volunteers

2016-11-29 Thread Patrick Hunt
lease speak up now as I intend to commit this soon. Patrick On Fri, Nov 18, 2016 at 12:15 PM, Patrick Hunt wrote: > As Flavio said originally on this thread this is a big change. Based on > the current status of the patch and the testing feedback it seems like > we've done significant

Re: Test plan for ZK-1045 - Call for volunteers

2016-11-29 Thread Patrick Hunt
3.5.3-alpha. Sorry, mixing 3.4 and 3.5. :-) Patrick On Tue, Nov 29, 2016 at 11:53 AM, Raúl Gutiérrez Segalés < r...@itevenworks.net> wrote: > On 18 November 2016 at 12:15, Patrick Hunt wrote: > > > As Flavio said originally on this thread this is a big change. Based on > t

Re: Review Request 47354: ZOOKEEPER-1045 : Quorum mutual authentication using SASL mechanism

2016-12-05 Thread Patrick Hunt
addresses my remaining issues. I'm +1. - Patrick Hunt On Nov. 29, 2016, 6:26 p.m., Rakesh R wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.a

Re: Test plan for ZK-1045 - Call for volunteers

2016-12-05 Thread Patrick Hunt
people to test. > > -Flavio > > > On 29 Nov 2016, at 17:23, Patrick Hunt wrote: > > > > I did a bunch of manual testing last week. I tried running multiple > cluster > > sizes, tried running new server against old server, also went through the > > rolling upgrade t

Re: How to use Kerberose auth with client API

2016-12-05 Thread Patrick Hunt
There's a wiki page here: https://cwiki.apache.org/confluence/display/ZOOKEEPER/Zookeeper+and+SASL Also see my docker environment for a concrete example: https://github.com/phunt/zk-docker-devenv/tree/master/kerberos.centos.6.7 Patrick On Tue, Nov 29, 2016 at 9:52 PM, Xie Gang wrote: > Hi, > >

Re: [VOTE] move Apache Zookeeper to git

2016-12-05 Thread Patrick Hunt
; you can close it in the GH mirror. In any case, I manually closed it. > > I will ping folks from other Apache projects using similar tool to inquire > why it didn't close the PR and why it gave this 'write access' error > message. > > Edward > > > >

Re: committing doc changes

2016-12-05 Thread Patrick Hunt
As Flavio mentioned we (committers) commit the docs so that users interested in d/l the source and using it don't need to generate the docs, which requires forrest and could often be a pita. In the early days this was seen as a benefit. That's the history at least. Patrick On Thu, Dec 1, 2016 at

Re: Test plan for ZK-1045 - Call for volunteers

2016-12-05 Thread Patrick Hunt
Shoot. Make that 3.4.10. Urg. Patrick On Mon, Dec 5, 2016 at 3:03 PM, Patrick Hunt wrote: > I like your second suggestion - let's cut an RC and give everyone the time > you are suggesting (2 weeks). That way folks can test and/or vote over the > period, rather than waiting the

Re: reconfig APIs

2016-12-07 Thread Patrick Hunt
This problem has been a long standing blocker issue for 3.5 and identified early on as something that would need to change. This has been one of the reasons why 3.5 has stayed in alpha - because we allow non-backward compatible changes to new APIs in alpha and we knew we would have to fix this. The

Re: [VOTE] move Apache Zookeeper to git

2016-12-07 Thread Patrick Hunt
pache git repo? > > Answer: "We now always use GitHub, we don't use patches any more" > Edward > > On Mon, Dec 5, 2016 at 9:07 PM, Patrick Hunt wrote: > > > No problem Edward. Did you get any insight on what to do with the PR? > > > > Pa

Re: reconfig APIs

2016-12-07 Thread Patrick Hunt
o reason to move it to a different class that I can see. Am I missing > something? > > > Jordan Zimmerman > > > On Dec 7, 2016, at 4:37 PM, Patrick Hunt wrote: > > > > This problem has been a long standing blocker issue for 3.5 and > identified

Re: reconfig APIs

2016-12-07 Thread Patrick Hunt
e.org/jira/browse/ZOOKEEPER-2642> - I hope we can > move these APIs back into ZooKeeper.java. > > -Jordan > > > On Dec 7, 2016, at 5:54 PM, Patrick Hunt wrote: > > > > It's discussed in more depth in the JIRA iirc, but basically; > > ZooKeeper.java provid

Re: Upcoming 3.4/3.5 releases.

2016-12-08 Thread Patrick Hunt
fer that > option too. If folks feel it is necessary to vote to capture more precisely > the decision, I can start a vote thread on the dev list. > > -Flavio > > > > On 23 Nov 2016, at 17:39, Patrick Hunt wrote: > > > > Forgot to mention. These will be ou

Re: reconfig APIs

2016-12-08 Thread Patrick Hunt
ache Curator, we’re > now put in a position where the reconfig() APIs have disappeared. In order > to maintain our own internal semantic versioning we will have to consider a > third branch of Curator (we currently have a ZooKeeper 3.4.x compatible > version and a ZooKeeper 3.5.x compatib

Re: ZooKeeper cwiki - Updated ZooKeeper and SASL auth 1045 work

2016-12-13 Thread Patrick Hunt
Nice job Rakesh, some comments: 1) the appendix is a great idea, should be useful for many people. One thing I noticed "There is no additional dependencies needed to use SASL with Java since it is part of the the Java Standard Edition." - you might want to mention/link the JCE? The JVM doesn't com

Re: ZooKeeper cwiki - Updated ZooKeeper and SASL auth 1045 work

2016-12-19 Thread Patrick Hunt
LGTM. Those changes are very helpful, thanks Rakesh! Patrick On Mon, Dec 19, 2016 at 12:04 PM, Rakesh Radhakrishnan wrote: > Thanks a lot Patrick Hunt for the review comments. Please take another look > at the wiki page when you get a chance. > > I've updated the wiki page

Re: ZooKeeper-trunk-WinVS2008 - Build # 2357 - Still Failing

2016-12-19 Thread Patrick Hunt
I made that change, but now it's failing not able to find msbuild - I updated the INFRA JIRA. Patrick On Thu, Dec 15, 2016 at 9:27 PM, Michael Han wrote: > Logged https://issues.apache.org/jira/browse/INFRA-13112 to track this. > > On Mon, Dec 12, 2016 at 9:45 AM, Michael Han wrote: > > > I as

[ANNOUNCE] New ZooKeeper committer: Michael Han

2017-01-03 Thread Patrick Hunt
The Apache ZooKeeper PMC recently extended committer karma to Michael and he has accepted. Michael has made some great contributions and we are looking forward to even more :) Congratulations and welcome aboard, Michael! Patrick

Re: Unable to contribute on JIRA

2016-06-08 Thread Patrick Hunt
I've added Martin as a contributor, give it another try. Patrick On Wed, Jun 8, 2016 at 3:21 PM, Marshall McMullen < marshall.mcmul...@gmail.com> wrote: > That makes sense. I would appreciate if a committer can change Martin's > role to be contributer. Otherwise we'll reach out to the Infra team

Re: Official Docker Image

2016-06-14 Thread Patrick Hunt
I'm no expert but Tim Crowder has written a blog post about zk/docker in the past and commented here in the list: http://markmail.org/message/7lczngv643jk6fcs have you been able to address? Or perhaps docker/zk has moved fwd in some way that helps? Patrick On Sun, Jun 5, 2016 at 8:32 AM, Елисей З

Re: Zookeeper 3.4.8 is bundled with old version of Netty:jar

2016-06-14 Thread Patrick Hunt
Pallavi do you have any insight into this? Michael? Are we ok with 3.x netty or is there some security related fix we are missing that would require 3.4 to upgrade to 4.x? Patrick On Wed, Jun 8, 2016 at 8:31 AM, Raúl Gutiérrez Segalés wrote: > On 7 June 2016 at 18:48, Patrick Hunt wr

Re: svn commit: r1747408 - in /zookeeper/trunk: CHANGES.txt src/java/main/org/apache/zookeeper/server/quorum/QuorumCnxManager.java

2016-06-14 Thread Patrick Hunt
Raul, what's the status on this? Seems like trunk was committed but not 3.5? We don't want to lose track. Patrick On Wed, Jun 8, 2016 at 8:43 AM, wrote: > Author: rgs > Date: Wed Jun 8 15:43:15 2016 > New Revision: 1747408 > > URL: http://svn.apache.org/viewvc?rev=1747408&view=rev > Log: > ZOO

Re: Review Request 47354: ZOOKEEPER-1045 : Quorum mutual authentication using SASL mechanism

2016-06-15 Thread Patrick Hunt
QuorumPeer.java (line 402) <https://reviews.apache.org/r/47354/#comment203068> the comment here is not super helpful - perhaps we could indicate why 20 vs some other number? - Patrick Hunt On June 5, 2016, 7:56 a.m., Rakesh R wrote: > > --

Re: Review Request 47354: ZOOKEEPER-1045 : Quorum mutual authentication using SASL mechanism

2016-06-15 Thread Patrick Hunt
and eliminate as many as possible (hopefully all). - Patrick Hunt On June 5, 2016, 7:56 a.m., Rakesh R wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 47354: ZOOKEEPER-1045 : Quorum mutual authentication using SASL mechanism

2016-06-15 Thread Patrick Hunt
ay not be wise to expose all of this. a different class but see ZOOKEEPER-2405 for some background. pull out something useful but not sensitive - e.g. a pricipal name or somesuch. src/java/main/org/apache/zookeeper/util/SecurityUtils.java (line 218) <https://reviews.apache.org/r/47354/#comment2

Re: ZooKeeper 3.5.2-alpha Release Coordination

2016-06-21 Thread Patrick Hunt
s still under active code > review that we want for 3.5.2. I'll keep those in. > > --Chris Nauroth > > > > > On 5/12/16, 2:44 PM, "Patrick Hunt" wrote: > > >Hi Chris. +1 on that. I believe it's imperative that we get 3.5.2-alpha > >out &

Re: Review Request 47354: ZOOKEEPER-1045 : Quorum mutual authentication using SASL mechanism

2016-06-21 Thread Patrick Hunt
> On June 16, 2016, 12:01 a.m., Patrick Hunt wrote: > > src/java/main/org/apache/zookeeper/server/quorum/LearnerHandler.java, line > > 172 > > <https://reviews.apache.org/r/47354/diff/6/?file=1406658#file1406658line172> > > > > should we say "

Re: ZooKeeper 3.5.2-alpha Release Coordination

2016-06-21 Thread Patrick Hunt
uding ZK-1045 in 3.5.2, right? > > -Flavio > > > On 21 Jun 2016, at 17:43, Patrick Hunt wrote: > > > > Can we set a deadline/date for the release? It's fine to have blockers > > while we're still in alpha. We should cut a release for folks that want > t

Re: Review Request 49112: ZOOKEEPER-1898

2016-06-22 Thread Patrick Hunt
could also have a subclass that understands keeper exceptions? - Patrick Hunt On June 22, 2016, 10:27 p.m., Abraham Fine wrote: > > --- > This is an automatically generated e-mail. To reply, vi

Re: Review Request 49112: ZOOKEEPER-1898

2016-06-22 Thread Patrick Hunt
> On June 22, 2016, 10:52 p.m., Patrick Hunt wrote: > > src/java/main/org/apache/zookeeper/ZooKeeperMain.java, line 626 > > <https://reviews.apache.org/r/49112/diff/1/?file=1427805#file1427805line626> > > > > See my comment on the exception class. Should we

Re: Official Docker Image

2016-06-24 Thread Patrick Hunt
ks, > > Elisey > > > > On Wed, Jun 15, 2016 at 11:11 PM Елисей Занько > > wrote: > > > >> Patrick, > >> > >>Thank you for pointing to this! > >>Yes, I've addressed this. There is a section called "... via > >>

Re: Official Docker Image

2016-06-24 Thread Patrick Hunt
at 10:33 AM, Flavio Junqueira < fpjunque...@yahoo.com.invalid> wrote: > My interpretation is that it is official from a Docker perspective, not > from a Apache ZooKeeper perspective. I wonder if we can make the official > docker image also official apache zk image. > > -Flavio

Re: Article about configuration management with Etcd, Consul and ZooKeeper

2016-06-24 Thread Patrick Hunt
Hey, we're only 10 years old, but I guess in code years that might be grandpa. ;-) I thought this was a good article, however it lacked focus on reliability/availability/correctness/etc... vs ease of use/features. When I talk to people about ZK vs another option it almost always comes back to "we

Re: Official Docker Image

2016-06-24 Thread Patrick Hunt
f there was confusion. Patrick > Regards, > Elisey > > On Fri, Jun 24, 2016 at 10:46 PM Patrick Hunt wrote: > > > So I have it backward then? In that case what would we need to > contribute? > > I do see tomcat and httpd there, but from what I can tell the "offi

Re: [VOTE] Apache ZooKeeper release 3.5.2-alpha candidate 0

2016-06-24 Thread Patrick Hunt
+1. sig/xsum are valid, RAT ran clean and no other discrepancies noticed. I ran it through a number of manual tests (including starting a 23 server ensemble) - no issues found. Patrick On Thu, Jun 23, 2016 at 10:33 PM, Chris Nauroth wrote: > This is a release candidate for 3.5.2-alpha. The full

Re: ZooKeeper 3.5.2-alpha Release Coordination

2016-06-24 Thread Patrick Hunt
5 PM, "Chris Nauroth" wrote: > > >I'd like to propose a deadline of 12 noon Thursday, 6/23 PST. If anyone > >needs a later deadline, please let me know. > > > >--Chris Nauroth > > > > > > > > > >On 6/21/16, 10:08 AM, &qu

Re: unexpected server response ZRUNTIMEINCONSISTENCY

2016-06-28 Thread Patrick Hunt
What version(s) are you seeing this against? (I put similar request in to the jira https://issues.apache.org/jira/browse/ZOOKEEPER-2455) Is there a small test case that reproduces this issue that you can provide? Logs? (if so please add them to the jira) Patrick On Tue, Jun 28, 2016 at 7:33 AM,

Re: Review Request 49112: ZOOKEEPER-1898

2016-06-28 Thread Patrick Hunt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49112/#review139890 --- Ship it! lgtm. - Patrick Hunt On June 23, 2016, 10:17 p.m

Re: ZooKeeper 3.5.2-alpha Release Coordination

2016-06-29 Thread Patrick Hunt
for getting the release candidate out. > > > >Thank you for volunteering to be the release manager for the 3.5.3 > >release, Pat. > > > >-Flavio > > > > > >> On 25 Jun 2016, at 00:16, Patrick Hunt wrote: > >> > >> Kudos on getting t

Re: [VOTE] Apache ZooKeeper release 3.5.2-alpha candidate 0

2016-06-29 Thread Patrick Hunt
erver. > Maybe it would be useful to hide the dependency in future release but > it is not a showstopper for my projects. > > > > 2016-06-28 19:52 GMT+02:00 Abraham Fine : > > +1 tested SSL support, everything appears to be working as expected. > > > > On Fri, Ju

Re: ZK-2458 and ZK-2459

2016-06-29 Thread Patrick Hunt
Thanks Flavio. I reviewed/committed both of them. Patrick On Wed, Jun 29, 2016 at 4:03 PM, Flavio Junqueira wrote: > I have submitted two patches to fix the license issues I found. Id' > appreciate if someone could have a look at them and possibly check them in > if LG. > > Thanks, > -Flavio

Re: [VOTE] Apache ZooKeeper release 3.5.2-alpha candidate 0

2016-06-29 Thread Patrick Hunt
Following Operations are verified > > create node (ALL API, All node types) > > delete node > > set data > > get data > > setAcl > > getAcl > > multi operation > > Few Four letter word commands > > 3) Veri

Re: ZRUNTIMEINCONSISTENCY

2016-06-29 Thread Patrick Hunt
Folks are trying to get more details on the jira. I posted a comment there with more details: https://issues.apache.org/jira/browse/ZOOKEEPER-2455?focusedCommentId=15356509&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15356509 Patrick On Wed, Jun 29, 2016 at 6:46

Re: [VOTE] Apache ZooKeeper release 3.5.2-alpha candidate 0

2016-06-29 Thread Patrick Hunt
possible. I don't know > if I have time today in order to submit a patch > > Enrico > > Il Gio 30 Giu 2016 06:19 Patrick Hunt ha scritto: > > > I've committed 2458 and 2459 to 3.5 and trunk branches if you want to cut > > another RC. These two patches addre

Re: Backporting ZOOKEEPER-2120 (SSL support) to 3.4.8

2016-07-11 Thread Patrick Hunt
I'd like to recommend that we consider a special 3.4 branch for this. (sorry I'm late to the party but I was on vacation till today, just back). We could create a 3.4+ssl branch, or something like that, and give special permissions for non-committers to commit to the branch. People that are intere

Re: Post 3.4 GA (was Re: Backporting ZOOKEEPER-2120 (SSL support) to 3.4.8)

2016-07-11 Thread Patrick Hunt
Hi Andrew. We've hashed through this a few times in the last couple months, not sure if you're following on the list. Summary: Primary gates are captured on the jira, the dashboard gives a good overview https://issues.apache.org/jira/secure/Dashboard.jspa?selectPageId=12327688 Blocker issues in p

Re: Script to spin up ZK cluster

2016-07-15 Thread Patrick Hunt
Consider posting links to stuff like this from https://cwiki.apache.org/confluence/display/ZOOKEEPER/UsefulTools Patrick On Fri, Jul 15, 2016 at 3:23 PM, Jeremy Duvall wrote: > I have a similar script that works for AWS if anyone's interested? I need > to clean it up a bit before posting on GH.

Re: [VOTE] Apache ZooKeeper release 3.5.2-alpha candidate 1

2016-07-17 Thread Patrick Hunt
+1 - sig/xsum are valid, rat ran clean, tested with a couple different ensemble sizes and cli and everything looked fine to me. Patrick On Fri, Jul 1, 2016 at 12:45 AM, Chris Nauroth wrote: > This is a release candidate for 3.5.2-alpha. The full release notes are > available at: > > https://iss

Re: Backporting ZOOKEEPER-2120 (SSL support) to 3.4.8

2016-07-18 Thread Patrick Hunt
On Mon, Jul 18, 2016 at 9:51 PM, Flavio Junqueira wrote: > > > That's a great idea, I'm +1. > > > > -Flavio > > > > > On 11 Jul 2016, at 21:13, Patrick Hunt wrote: > > > > > > I'd like to recommend that we consider a special 3.4 branch for

Re: ZooKeeper 3.4.9 release discussion

2016-07-18 Thread Patrick Hunt
On Sat, Jul 9, 2016 at 10:26 AM, Rakesh Radhakrishnan wrote: > > >ZK-1045 is a major change and I suggest we give time for folks to try > it out before cutting a release candidate. Ideally, we check it in and give > a few weeks until we cut a release candidate. > > Agreed, will try to push ZK-

Re: Status: Apache ZooKeeper release 3.5.2-alpha candidate 1

2016-07-20 Thread Patrick Hunt
Thanks for pointing this out Chris, I'll update the site. FYI committers can view the official list here: https://whimsy3.apache.org/roster/committee/zookeeper Patrick On Tue, Jul 19, 2016 at 10:09 PM, Chris Nauroth wrote: > I had miscounted the vote from Raúl as non-binding, based on this page

Re: Status: Apache ZooKeeper release 3.5.2-alpha candidate 1

2016-07-20 Thread Patrick Hunt
Site is updated. Patrick On Wed, Jul 20, 2016 at 2:51 PM, Patrick Hunt wrote: > Thanks for pointing this out Chris, I'll update the site. > > FYI committers can view the official list here: > https://whimsy3.apache.org/roster/committee/zookeeper > > Patrick > > On

Re: DSN 2011 paper link broken on cwiki

2016-07-21 Thread Patrick Hunt
Thanks Michael, I updated the link. Patrick On Thu, Jul 21, 2016 at 9:12 AM, Michael Han wrote: > https://cwiki.apache.org/confluence/display/ZOOKEEPER/Zab+vs.+Paxos > > The DSN 2011 paper at end of the page > is now 404. This should be the same paper: > htt

Re: Official Docker Image

2016-07-22 Thread Patrick Hunt
out? >> If it looks good could someone please leave a short comment on behalf >> of the committers team in the PR >> <https://github.com/docker-library/official-images/pull/1765>? >> >> Thanks, >> Elisey >> >> On Fri, Ju

Re: [ANNOUNCE] Apache ZooKeeper 3.5.2-alpha

2016-07-22 Thread Patrick Hunt
Kudos all, thanks Chris! Patrick On Thu, Jul 21, 2016 at 7:46 PM, Flavio Junqueira wrote: > Thanks everyone for the push to get 3.5.2-alpha out, and specially to Chris > for being the release manager. > > -Flavio > >> On 21 Jul 2016, at 13:08, Chris Nauroth wrote: >> >> The Apache ZooKeeper te

Re: Official Docker Image

2016-07-22 Thread Patrick Hunt
rary will be merged > until having any proof of upstream awareness and acceptance. > > Do I understand it right that you guys are OK with the image as long as > it uses the official distribution and you don't have any suggestions about > it right now, neither want to collab

Re: CHANGES.txt inconsistency

2016-07-22 Thread Patrick Hunt
On Thu, Jul 21, 2016 at 3:59 PM, Chris Nauroth wrote: > While working on the 3.5.2-alpha release, I noticed that CHANGES.txt in > trunk is not completely synchronized with the 3.5 release line. In > branch-3.5, we have sections for releases 3.5.0, 3.5.1 and 3.5.2 (added by > me), each indicating

Re: Flaky unit tests

2016-07-22 Thread Patrick Hunt
Thanks for taking this up Michael, much appreciated! If folks are looking for ways to participate in the community helping to address these issues is a great way to do so, it's also a great way to learn the project! Patrick On Fri, Jul 22, 2016 at 2:39 PM, Michael Han wrote: > Hi, > > I've noti

Re: PreCommitBuild - Adding comment to Jira

2016-07-25 Thread Patrick Hunt
I'd recommend you start by opening an INFRA jira against Jenkins. The INFRA folks are usually pretty quick to address. Patrick On Mon, Jul 25, 2016 at 10:26 AM, Flavio Junqueira wrote: > I've only logged on to the jenkins console, I've never tried to access the > machine itself. > > -Flavio > >

Re: PreCommitBuild - Adding comment to Jira

2016-07-26 Thread Patrick Hunt
: > Thanks Michael for reporting the infra issue. Lets us watch the progress. > > Rakesh > > On Tue, Jul 26, 2016 at 12:25 AM, Michael Han wrote: > > > Filed https://issues.apache.org/jira/browse/INFRA-12327. > > > > On Mon, Jul 25, 2016 at 11:15 AM, Patrick Hu

Re: PreCommitBuild - Adding comment to Jira

2016-07-26 Thread Patrick Hunt
Note that there is a similar, but different, issue now affecting trunk builds. Seems to have started a few days ago, on July23. https://issues.apache.org/jira/browse/INFRA-12335 Patrick On Tue, Jul 26, 2016 at 3:28 PM, Patrick Hunt wrote: > FYI - I'm working through some of the sug

ZooKeeper Jenkins builds are broken (environmental)

2016-07-27 Thread Patrick Hunt
Hi folks - just a heads up that the Jenkins environment has changed and as a result our builds are failing. We're working with INFRA team to resolve this https://issues.apache.org/jira/browse/INFRA-12327 Patrick

Re: ZooKeeper Jenkins builds are broken (environmental)

2016-07-28 Thread Patrick Hunt
bot to JIRA, so I closed the > INFRA-12327 <https://issues.apache.org/jira/browse/INFRA-12327> as fixed. > > On Wed, Jul 27, 2016 at 10:39 AM, Patrick Hunt wrote: > > > Hi folks - just a heads up that the Jenkins environment has changed and > as > > a result our

Re: Flaky unit tests

2016-07-28 Thread Patrick Hunt
/#showFailuresLink are those two tests captured already, or do we need to create new jiras for these? (would you mind, thanks!) Patrick On Fri, Jul 22, 2016 at 8:51 PM, Patrick Hunt wrote: > Thanks for taking this up Michael, much appreciated! If folks are looking > for ways to participate in the com

Re: ZooKeeper Jenkins builds are broken (environmental)

2016-07-29 Thread Patrick Hunt
t; > > > https://builds.apache.org/view/All/job/ZooKeeper_branch35_openjdk7/lastCompletedBuild/testReport/org.apache.zookeeper.server.quorum/ReconfigRecoveryTest/testCurrentServersAreObserversInNextConfig/ > > > > > > > I've just seen it in the 35 openjdk build. >

Re: myid documentation

2016-08-05 Thread Patrick Hunt
I believe that the myid is used as part of the server id, in order to get a unique value for quorum formation. iirc it's the top 8 bits of the server id. Hence the reason why we mention that in the docs. afaik we never "enforced" this at the code level. On the plus side it would be a good idea, on

Re: Proposal: Drop all project specific labels.

2016-08-06 Thread Patrick Hunt
Hi Gavin. afaik we had those labels in order to run our "hadoop qa bot" process. Looks like four jobs currently have that label: https://builds.apache.org/label/Zookeeper/ as long as we can continue to run the "hadoop qa bot" on any other (non H* nodes) I believe we don't have any other reason to

Re: Proposal: Drop all project specific labels.

2016-08-06 Thread Patrick Hunt
Sure thing, sounds reasonable to me. Thanks Gavin. Patrick On Sat, Aug 6, 2016 at 7:28 PM, Gav wrote: > Hi, > > On Sun, Aug 7, 2016 at 11:08 AM, Patrick Hunt wrote: > >> Hi Gavin. afaik we had those labels in order to run our "hadoop qa bot" >> process. L

Re: [ANNOUNCE] Chris Nauroth joins the Apache ZooKeeper PMC

2016-08-07 Thread Patrick Hunt
Kudos Chris! Patrick On Sun, Aug 7, 2016 at 11:05 AM, Flavio Junqueira wrote: > In recognition of all his contributions to the project, the Apache > ZooKeeper PMC has invited Chris Nauroth to join the PMC and he has > accepted. I'd like to take the opportunity to thank Chris for his > contribut

Fwd: Jenkins Ant Matrix - and consolidating of versions.

2016-08-07 Thread Patrick Hunt
FYI in case you notice any weirdness: -- Forwarded message -- From: Patrick Hunt Date: Sun, Aug 7, 2016 at 5:25 PM Subject: Re: Jenkins Ant Matrix - and consolidating of versions. To: gmcdon...@apache.org Cc: builds , "infrastruct...@apache.org Infrastructure" <

Re: http://zookeeper.apache.org/doc/r3.5.2-alpha/ showing old document

2016-08-09 Thread Patrick Hunt
Thanks for catching this Mohammad, it looks like this commit changed the source on 3.5 branch but the generated docs were not re-generated: + ZOOKEEPER-2402: Document client side properties (Arshad Mohammad via fpj) svn commit: r1745280 - in /zookeeper/branches/branch-3.5: CHANGES.txt src/docs/s

Fwd: [PLANNED OUTAGE] - Jenkins upgrade at builds.a.o

2016-08-09 Thread Patrick Hunt
FYI, jenkins upgrade is coming: -- Forwarded message -- From: Gav Date: Tue, Aug 9, 2016 at 7:39 PM Subject: [PLANNED OUTAGE] - Jenkins upgrade at builds.a.o Hi All, Jenkins - at builds.apache.org is going to be upgraded. Currently on version 1.6.42.2 and upgrading to 2.7.2 LTS

Re: http://zookeeper.apache.org/doc/r3.5.2-alpha/ showing old document

2016-08-16 Thread Patrick Hunt
> in the > > next release > > >> 2. Can we generate and update the documentation site > > http://zookeeper.apache.org/doc/r3.5.2-alpha/, because mostly people > > refer this website for document reference. > > >> > > >> -Arshad

Seems Reddit is using ZK.

2016-08-16 Thread Patrick Hunt
Unfortunate incident, some ops issues while migrating ZK/infra led to site outage. Sheds some light on how Reddit uses ZK though: "We use a system called Zookeeper to keep track of most of our servers and their health." https://www.reddit.com/r/announcements/comments/4y0m56/why_reddit_was_down_on

Re: ZooKeeper 3.4.9 release discussion

2016-08-16 Thread Patrick Hunt
t; > (3) I'd love to include ZK-2355 also. It looks like the fix is good, > I've > > added few comments. Perhaps, will wait to see feedback from others. > > > > Kindly requesting everyone to look at your issues marked for 3.4.9 > a

Re: ZooKeeper 3.4.9 release discussion

2016-08-16 Thread Patrick Hunt
--Chris Nauroth > > On 8/16/16, 2:52 PM, "Patrick Hunt" wrote: > > Rakesh pointed out that the current "how to release" docs no longer are > correct for branch-3.4 as in branch-3.5 and later we updated the > release > notes creation proc

Re: ZooKeeper 3.4.9 release discussion

2016-08-17 Thread Patrick Hunt
then add >> our current 3.4.9 release notes, isn't it? >> >> >> Thanks, >> Rakesh >> >> On Wed, Aug 17, 2016 at 8:13 AM, Rakesh Radhakrishnan > > wrote: >> >>> Thanks Patrick and Chris for quickly resolving this. I will continue

Re: [VOTE] Apache ZooKeeper release 3.4.9 candidate 0

2016-08-20 Thread Patrick Hunt
+1 sig/xsum are correct. RAT ran clean. I was able to start/use a number of different size ensembles with the code. Looks good to me. Patrick On Thu, Aug 18, 2016 at 9:24 AM, Rakesh Radhakrishnan wrote: > This is a bugfix release candidate for 3.4.9. It fixes 21 issues, including > issues that

Re: [VOTE] Apache ZooKeeper release 3.4.9 candidate 0

2016-08-22 Thread Patrick Hunt
Rakesh - part of running step 1 of the "building" (ant test package tar) is that you should end up with (notice this is in the build directory): phunt@phunt-MBP13:~/dev/zookeeper-br34[branch-3.4]$ ls build/zookeeper-3.4.9/src/c/ ChangeLog NOTICE.txt c-doc.Doxyfile configure.ac missing

Re: [VOTE] Apache ZooKeeper release 3.4.9 candidate 0

2016-08-22 Thread Patrick Hunt
it has > created the tarball perfectly with all these missing files. > > Rakesh > > On Mon, Aug 22, 2016 at 10:31 PM, Patrick Hunt wrote: > > > Rakesh - part of running step 1 of the "building" (ant test package tar) > is > > that you should end up with (

Re: [VOTE] Apache ZooKeeper release 3.4.9 candidate 2

2016-08-23 Thread Patrick Hunt
+1 xsum/sig are valid. RAT ran clean. I was able to compile the code and successfully put a few ensemble sizes through their paces. fwiw I also did a "diff" btw 3.4.8 release artifact and this rc. There were a number of changed files, obviously. However I did not notice any missing files, as we've

Re: Procedure for adding a new feature

2016-08-24 Thread Patrick Hunt
The "how to contribute" page is also a good resource for new folks interested in participating: https://cwiki.apache.org/confluence/display/ZOOKEEPER/HowToContribute Patrick On Tue, Aug 23, 2016 at 5:27 PM, Camille Fournier wrote: > We are generally open to new features if they seem useful to

Re: ZOOKEEPER-2169

2016-08-26 Thread Patrick Hunt
Hi Jordan. This looks like a great new feature, but I'm afraid I'm focused on other things atm. I don't have much time after work/home currently, as such I've been focused on other priorities; 1) supporting existing users/issues in 3.4, and 2) trying to get 3.5 branch to production ready. There are

Re: switching to git?

2016-08-26 Thread Patrick Hunt
I really love it when the new guy shows up and immediately requests tool changes. ;-) afaik given we have a git mirror already, moving to git from an infra perspective is pretty straightforward. Definitely some impact in some other areas (like cutting a release) but we should be able to chase thos

Re: ZOOKEEPER-2169

2016-08-26 Thread Patrick Hunt
I'm the slacker that proposed us doing this in the first place and I'm > THRILLED that you have done it Jordan, thank you so much. > Pat, I can review and merge, unless you are concerned with interference on > other issues. LMK. > > C > > On Fri, Aug 26, 2016 at 10:42 A

ZK and java9 beta

2016-08-28 Thread Patrick Hunt
For the hell of it I cloned one of our trunk jobs and turned on the latest java9 beta available on apache jenkins. Passed 2 out of 5 runs: https://builds.apache.org/view/S-Z/view/ZooKeeper/job/ZooKeeper-trunk-java9/ In particular I noticed org.apache.zookeeper.test.NettyNettySuiteTest.testRemoveA

Re: switching to git?

2016-08-28 Thread Patrick Hunt
original proposal from many years ago! > > On Fri, Aug 26, 2016 at 6:15 PM, Patrick Hunt wrote: > > > I really love it when the new guy shows up and immediately requests tool > > changes. ;-) > > > > afaik given we have a git mirror already, moving to git from

Re: Issues with release notes in 3.4.9 RC2 (was: Re: [VOTE] Apache ZooKeeper release 3.4.9 candidate 2)

2016-08-28 Thread Patrick Hunt
gt; > a/browse/ZOOKEEPER-1676>] > > > - C client zookeeper_interest returning ZOK on Connection Loss > > > > > > There is no bits related to this JIRA went to 3.4.9 either, as > > > this is closed as 'not a bug'. I'd expect such

Re: Issues with release notes in 3.4.9 RC2 (was: Re: [VOTE] Apache ZooKeeper release 3.4.9 candidate 2)

2016-08-28 Thread Patrick Hunt
x27; and included > in 3.4.6 release note. Should we follow the same pattern and > include ZOOKEEPER-1676 also in 3.4.9 release note, please correct me if I > missed anything. > > https://issues.apache.org/jira/secure/ReleaseNote.jspa? > projectId=12310801&version=12323310 >

Re: switching to git?

2016-08-29 Thread Patrick Hunt
gt; I don't have experience with gerritt and I have see only one project > > referring to gerritt in Apache: Kudo. > > > > -Flavio > > > > > On 29 Aug 2016, at 14:25, Camille Fournier wrote: > > > > > > I'm confused what is the a

Re: Question on 3.4.9 RC2

2016-08-31 Thread Patrick Hunt
Well, the "affects" (what version(s) this issue affects) is blank and the "fix version" (what version(s) we want to fix it in) is explicitly set to 3.5+. Do we need it for 3.4? My recommendation would be to create a new jira for 3.4.10. Patrick On Wed, Aug 31, 2016 at 9:15 AM, Flavio Junqueira

Re: switching to git?

2016-08-31 Thread Patrick Hunt
; > > > > > i'm curious, can we get the qa > > > bot to trigger off a pull request? > > > > Yes, we can. See this Kafka build queue: > > > > https://builds.apache.org/job/kafka-trunk-git-pr-jdk7/ < > > https://builds.apache.org

Re: Issue with NettyServerCnxn.java

2016-08-31 Thread Patrick Hunt
Hi Yuliya - my read is that sendResponse in NIOServerCnxn is logging, then dropping, any Exceptions encountered during sendResponse. In other words it's doing best effort response. Not sure if that is "correct", but that's what it's currently doing in NIO. Surprisingly it's also hiding any IOExcept

Re: [VOTE] move Apache Zookeeper to git

2016-08-31 Thread Patrick Hunt
+1. Patrick On Wed, Aug 31, 2016 at 9:48 PM, Chris Nauroth wrote: > +1 > > --Chris Nauroth > > On 8/31/16, 6:29 PM, "Benjamin Reed" wrote: > > flip the switch to git and update the relevant scripts and docs. > > i couldn't figure out which timeframe this falls under in the voting >

Re: FollowerResyncConcurrencyTest failed in ZooKeeper 3.3.3

2016-09-03 Thread Patrick Hunt
Also there were a number of patches that went into 3.4+ in order to support newer versions of Java - given you are on 3.3 you will not be seeing the benefit of this. Patrick On Thu, Sep 1, 2016 at 9:35 AM, Michael Han wrote: > Hi Kang, > > Have you checked https://issues.apache.org/jira/browse/

<    1   2   3   4   5   6   7   8   9   10   >