Re: ReviewBoard

2014-02-02 Thread David Nalley
OK - this is now done - we shrank from 93 to 44 reviews due to this. Lets see if we can get these reviewed or rejected and keep up with this. --David On Mon, Jan 27, 2014 at 2:59 PM, David Nalley wrote: > Hi folks: > > ReviewBoard is pretty bloated. There are currently 93 reviews that are open.

Re: ReviewBoard

2014-02-02 Thread David Nalley
I have not. Yesterday RB went down for some period of time, though AFAIK, infra restored it. --David On Sat, Feb 1, 2014 at 4:11 PM, Daan Hoogland wrote: > H David, > > have you started on this yet? > I can't reach the review board at the moment. > > On Mon, Jan 27, 2014 at 8:59 PM, David Nalley

Re: ReviewBoard

2014-02-01 Thread Daan Hoogland
H David, have you started on this yet? I can't reach the review board at the moment. On Mon, Jan 27, 2014 at 8:59 PM, David Nalley wrote: > Hi folks: > > ReviewBoard is pretty bloated. There are currently 93 reviews that are open. > > Some of those haven't been updated in 7 months old. > > Unles

Re: Reviewboard needs your attention

2013-11-08 Thread Mike Tutkowski
7 AM > > To: dev@cloudstack.apache.org > > Subject: Re: Reviewboard needs your attention > > > > This is what I see when logged in: > > > > http://i.imgur.com/RZvXuy1.png > > > > Am I missing something? > > > > Thanks! > > > > &

RE: Reviewboard needs your attention

2013-11-08 Thread Animesh Chaturvedi
7 AM > To: dev@cloudstack.apache.org > Subject: Re: Reviewboard needs your attention > > This is what I see when logged in: > > http://i.imgur.com/RZvXuy1.png > > Am I missing something? > > Thanks! > > > On Thu, Nov 7, 2013 at 10:01 PM, Animesh Chaturvedi < > ani

Re: Reviewboard needs your attention

2013-11-08 Thread Mike Tutkowski
--- > > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > > Sent: Thursday, November 07, 2013 6:03 PM > > To: dev@cloudstack.apache.org > > Subject: Re: Reviewboard needs your attention > > > > Hi Animesh, > > > > For whatever reason, I can n

RE: Reviewboard needs your attention

2013-11-07 Thread Animesh Chaturvedi
I can see it fine, were you logged in at that time > -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: Thursday, November 07, 2013 6:03 PM > To: dev@cloudstack.apache.org > Subject: Re: Reviewboard needs your attention > >

Re: Reviewboard needs your attention

2013-11-07 Thread Mike Tutkowski
Hi Animesh, For whatever reason, I can no longer edit https://reviews.apache.org/r/13865/. Any thoughts on this? Thanks! On Thu, Nov 7, 2013 at 6:41 PM, Nguyen Anh Tu wrote: > 2013/11/8 Animesh Chaturvedi > > > Given that we are approaching the 4.3 feature freeze tomorrow if you have > > su

Re: Reviewboard needs your attention

2013-11-07 Thread Nguyen Anh Tu
2013/11/8 Animesh Chaturvedi > Given that we are approaching the 4.3 feature freeze tomorrow if you have > submitted patches and are still pending reviews please reach out to your > reviewers Dear Sebastien & Hugo, I have two pending patches on review board need to review: Rebase OVS (sdnexte

Re: Reviewboard needs your attention

2013-11-07 Thread Sebastien Goasguen
I hate to move deadline, but it might be good to have couple days to try to clean review board. -Sebastien On 7 Nov 2013, at 19:26, Animesh Chaturvedi wrote: > There are large number of patches in review board. I went through some of > them and followed up and closed out few that are already

Re: Reviewboard and jenkins

2013-07-26 Thread Hugo
Hey Chip, Working on it! :-) Cheers, Hugo On Friday, July 26, 2013, Chip Childers wrote: > Hugo, > > Any chance you could add some logic to pick the right branch for the build > test? > > > On Mon, Jul 22, 2013 at 6:06 AM, Hugo Trippaers > > > wrote: > > > The job is running on jenkins.clouds

Re: Reviewboard and jenkins

2013-07-25 Thread Chip Childers
Hugo, Any chance you could add some logic to pick the right branch for the build test? On Mon, Jul 22, 2013 at 6:06 AM, Hugo Trippaers wrote: > The job is running on jenkins.cloudstack.org > > This is de job doing the work: > > http://jenkins.cloudstack.org/view/management/job/mgmt-build-revie

Re: Reviewboard and jenkins

2013-07-22 Thread Prasanna Santhanam
On Mon, Jul 22, 2013 at 12:06:31PM +0200, Hugo Trippaers wrote: > The job is running on jenkins.cloudstack.org > > This is de job doing the work: > > http://jenkins.cloudstack.org/view/management/job/mgmt-build-reviewboard-requests/ > > This is the job being executed with the patch: >

Re: Reviewboard and jenkins

2013-07-22 Thread Hugo Trippaers
The job is running on jenkins.cloudstack.org This is de job doing the work: http://jenkins.cloudstack.org/view/management/job/mgmt-build-reviewboard-requests/ This is the job being executed with the patch: http://jenkins.cloudstack.org/view/master/job/cloudstack-master-with-patc

Re: Reviewboard and jenkins

2013-07-20 Thread Prasanna Santhanam
On Sat, Jul 20, 2013 at 09:46:59AM -0400, David Nalley wrote: > On Sat, Jul 20, 2013 at 5:53 AM, Hugo Trippaers wrote: > > > > > > Sent from my iPhone > > > > On 20 jul. 2013, at 10:09, Prasanna Santhanam wrote: > > > >> On Sat, Jul 20, 2013 at 12:20:11AM +0200, Hugo Trippaers wrote: > >>> > >>>

Re: Reviewboard and jenkins

2013-07-20 Thread David Nalley
On Sat, Jul 20, 2013 at 5:53 AM, Hugo Trippaers wrote: > > > Sent from my iPhone > > On 20 jul. 2013, at 10:09, Prasanna Santhanam wrote: > >> On Sat, Jul 20, 2013 at 12:20:11AM +0200, Hugo Trippaers wrote: >>> >>> On Jul 19, 2013, at 7:15 PM, Daan Hoogland wrote: >>> good stuff, does it ru

Re: Reviewboard and jenkins

2013-07-20 Thread Hugo Trippaers
Sent from my iPhone On 20 jul. 2013, at 10:09, Prasanna Santhanam wrote: > On Sat, Jul 20, 2013 at 12:20:11AM +0200, Hugo Trippaers wrote: >> >> On Jul 19, 2013, at 7:15 PM, Daan Hoogland wrote: >> >>> good stuff, does it run the risk of running while another instance is busy? >> >> Probab

Re: Reviewboard and jenkins

2013-07-20 Thread Prasanna Santhanam
On Sat, Jul 20, 2013 at 12:20:11AM +0200, Hugo Trippaers wrote: > > On Jul 19, 2013, at 7:15 PM, Daan Hoogland wrote: > > > good stuff, does it run the risk of running while another instance is busy? > > Probably not, it's a single threaded script. I intend to use jenkins > to schedule it and i

Re: Reviewboard and jenkins

2013-07-19 Thread Prasanna Santhanam
On Fri, Jul 19, 2013 at 06:51:29PM +0200, Hugo Trippaers wrote: > Hey all, > > I've been working on some scripting to get patches submitted on review board > automagically tested with Jenkins. > > The way it works is that a script will check if the jenkins user on > review board (catofong) has c

Re: Reviewboard and jenkins

2013-07-19 Thread Hugo Trippaers
On Jul 19, 2013, at 7:15 PM, Daan Hoogland wrote: > good stuff, does it run the risk of running while another instance is busy? Probably not, it's a single threaded script. I intend to use jenkins to schedule it and i can tell jenkins not to start another instance of a job once one is running

Re: Reviewboard and jenkins

2013-07-19 Thread Daan Hoogland
good stuff, does it run the risk of running while another instance is busy? On Fri, Jul 19, 2013 at 6:57 PM, Prasanna Santhanam wrote: > On Fri, Jul 19, 2013 at 06:51:29PM +0200, Hugo Trippaers wrote: > > Hey all, > > > > I've been working on some scripting to get patches submitted on review >

Re: Reviewboard and jenkins

2013-07-19 Thread Chip Childers
Freaking awesome. I saw that hit the ML, and was just about to ask about it. On Fri, Jul 19, 2013 at 12:51 PM, Hugo Trippaers wrote: > Hey all, > > I've been working on some scripting to get patches submitted on review > board automagically tested with Jenkins. > > The way it works is that a s