Re: PR permission to kick Jenkins?

2017-05-05 Thread shane knapp
alright. this kinda sucks. after looking at that PR, i found a bunch of comments by tom and marcelo on the morning of may 2nd. i grepped for that PR# in the jenkins logs and found things that made me REALLY SAD (as well as making me ask myself how this even works to begin with): INFO: Pull

Re: PR permission to kick Jenkins?

2017-05-05 Thread shane knapp
ok cool, thanks. i'll check the jenkins logs from then and see if i can find anything. no promises. :\ On Fri, May 5, 2017 at 1:10 PM, Marcelo Vanzin wrote: > I noticed it on this one (before I said "ok to test"): > https://github.com/apache/spark/pull/17658 > > My

Re: PR permission to kick Jenkins?

2017-05-05 Thread Marcelo Vanzin
I noticed it on this one (before I said "ok to test"): https://github.com/apache/spark/pull/17658 My requests went through, Tom's seemed to be ignored. On Fri, May 5, 2017 at 1:06 PM, shane knapp wrote: > also, do we have any recent PRs where i can see this happening? it

Re: PR permission to kick Jenkins?

2017-05-05 Thread shane knapp
also, do we have any recent PRs where i can see this happening? it will make my log-diving a bit easier. On Fri, May 5, 2017 at 12:48 PM, shane knapp wrote: > tom: i checked and your username is still on the list. > > anyways, it's a pretty big list... i pulled a couple

Re: PR permission to kick Jenkins?

2017-05-05 Thread shane knapp
tom: i checked and your username is still on the list. anyways, it's a pretty big list... i pulled a couple of names out of it (including mine -- i can add myself back if needed), but i'd be down to have someone (rxin, maybe?) audit the list and pare it down. also, after checking out the code

Re: PR permission to kick Jenkins?

2017-05-05 Thread Reynold Xin
I suspect the list is getting too big for Jenkins to function well. It stopped working for me a while ago. On Fri, May 5, 2017 at 12:06 PM, Tom Graves wrote: > Does anyone know how to configure Jenkins to allow committers to tell it > to test prs? I used to have

PR permission to kick Jenkins?

2017-05-05 Thread Tom Graves
Does  anyone know how to configure Jenkins to allow committers to tell it to test prs?  I used to have this access but lately it is either not working or only intermittently working. The commands like "ok to test", "test this please", etc.. Thanks,Tom