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

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 a

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

2013-03-25 Thread Animesh Chaturvedi
-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 Subject: Re: Review Request: Temporarily disabling

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

2013-03-25 Thread Chip Childers
; 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: Vijay Can you check this again with a new workspace? Chip both Min and I had verified on Friday

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

2013-03-25 Thread Min Chen
...@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 Subject: Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1 On Mon, Mar 25, 2013 at 08:12

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

2013-03-25 Thread Chip Childers
: 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 before Vijay posted it. I

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-24 Thread Chip Childers
...@sungard.com] Sent: Friday, March 22, 2013 5:04 PM To: dev@cloudstack.apache.org; 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 please? Does it apply cleanly? Does

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 chip.child...@sungard.com wrote: --- This is an automatically generated e-mail. To reply, visit:

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

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 vijayendra.bhamidip...@citrix.com wrote: --- This is an automatically generated e-mail. To reply,

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

2013-03-22 Thread Angeline Shen
: Frank Zhang; Kelven Yang; 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 vijayendra.bhamidip...@citrix.com

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

2013-03-22 Thread Vijayendra Bhamidipati
; 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 vijayendra.bhamidip...@citrix.com wrote