Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-27 Thread Animesh Chaturvedi
> On March 24, 2013, 6:08 p.m., Chip Childers wrote: > > This patch still doesn't apply cleanly. Now sure what's going on. Please > > test with a fresh copy of the 4.1 branch to ensure that it's a clean patch. > > > > git am ~/patches/10071.patch > > Applying: CS-1773: Disable baremetal funct

Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-25 Thread Chip Childers
2013 8:15 AM > >> > To: Animesh Chaturvedi > >> > Cc: dev@cloudstack.apache.org; Frank Zhang; Kelven Yang; cloudstack; > >> > Vijayendra Bhamidipati > >> > Subject: Re: Review Request: Temporarily disabling baremetal > >>functionality in

Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-25 Thread Min Chen
gt; -Original Message- >> > From: Chip Childers [mailto:chip.child...@sungard.com] >> > Sent: Monday, March 25, 2013 8:15 AM >> > To: Animesh Chaturvedi >> > Cc: dev@cloudstack.apache.org; Frank Zhang; Kelven Yang; cloudstack; >> > Vijayendra Bhamidipati

Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-25 Thread Chip Childers
apache.org; Frank Zhang; Kelven Yang; cloudstack; > > Vijayendra Bhamidipati > > Subject: Re: Review Request: Temporarily disabling baremetal functionality > > in > > CS 4.1 > > > > On Mon, Mar 25, 2013 at 08:12:52AM -0700, Animesh Chaturvedi wrote: > > > Vija

RE: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-25 Thread Animesh Chaturvedi
Request: Temporarily disabling baremetal functionality in > CS 4.1 > > On Mon, Mar 25, 2013 at 08:12:52AM -0700, Animesh Chaturvedi wrote: > > Vijay > > > > Can you check this again with a new workspace? Chip both Min and I had > verified on Friday that patch was applying

Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-25 Thread Chip Childers
On Mon, Mar 25, 2013 at 08:12:52AM -0700, Animesh Chaturvedi wrote: > Vijay > > Can you check this again with a new workspace? Chip both Min and I had > verified on Friday that patch was applying before Vijay posted it. I will > also try again with fresh 4.1. Sorry for the inconvenience. I did

Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-25 Thread Animesh Chaturvedi
Vijay Can you check this again with a new workspace? Chip both Min and I had verified on Friday that patch was applying before Vijay posted it. I will also try again with fresh 4.1. Sorry for the inconvenience. Thanks Animesh On Mar 24, 2013, at 11:08 AM, "Chip Childers" wrote: > > --

Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-24 Thread Chip Childers
hilders [mailto:chip.child...@sungard.com] > Sent: Friday, March 22, 2013 5:04 PM > To: ; Vijayendra Bhamidipati > Cc: Frank Zhang; Kelven Yang; cloudstack > Subject: Re: Review Request: Temporarily disabling baremetal functionality in > CS 4.1 > > Can *you* check it first p

Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-24 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10071/#review18329 --- This patch still doesn't apply cleanly. Now sure what's going on.

RE: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-22 Thread Vijayendra Bhamidipati
t: Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1 Can *you* check it first please? Does it apply cleanly? Does it compile? On Mar 22, 2013, at 7:55 PM, Venkata Siva Vijayendra Bhamidipati wrote: > > --- &

RE: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-22 Thread Angeline Shen
; cloudstack Subject: Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1 Can *you* check it first please? Does it apply cleanly? Does it compile? On Mar 22, 2013, at 7:55 PM, Venkata Siva Vijayendra Bhamidipati wrote

Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-22 Thread Chip Childers
Can *you* check it first please? Does it apply cleanly? Does it compile? On Mar 22, 2013, at 7:55 PM, Venkata Siva Vijayendra Bhamidipati wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apach

Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-22 Thread Venkata Siva Vijayendra Bhamidipati
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10071/ --- (Updated March 22, 2013, 11:55 p.m.) Review request for cloudstack, Kelven Yang

Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-22 Thread Animesh Chaturvedi
Vijay Please fix and submit a new patch Thanks Animesh On Mar 22, 2013, at 8:04 AM, "Chip Childers" wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10071/#review18264 >

Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-22 Thread Chip Childers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10071/#review18264 --- This patch does not apply to 4.1 cleanly. error: patch failed: serv

Review Request: Temporarily disabling baremetal functionality in CS 4.1

2013-03-21 Thread Venkata Siva Vijayendra Bhamidipati
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10071/ --- Review request for cloudstack, Kelven Yang and Frank Zhang. Description ---