Re: [Review Request] Re-enabling baremetal on master

2013-06-21 Thread Sheng Yang
Thank you all! I would commit the change to MASTER soon. --Sheng On Thu, Jun 20, 2013 at 7:22 PM, David Nalley wrote: > Yes > Happy to +1. Sheng, thanks for stepping up and getting this done. > > --David > On Jun 20, 2013 7:19 PM, "Chip Childers" > wrote: > > > Nice! I'm glad the feature ha

Re: [Review Request] Re-enabling baremetal on master

2013-06-20 Thread David Nalley
Yes Happy to +1. Sheng, thanks for stepping up and getting this done. --David On Jun 20, 2013 7:19 PM, "Chip Childers" wrote: > Nice! I'm glad the feature has the benefit of tests now. Thanks for > doing this Sheng! > > David - are you comfortable with this, and will you now +1 the feature? >

Re: [Review Request] Re-enabling baremetal on master

2013-06-20 Thread Chip Childers
Nice! I'm glad the feature has the benefit of tests now. Thanks for doing this Sheng! David - are you comfortable with this, and will you now +1 the feature? On Thu, Jun 20, 2013 at 9:55 PM, Sheng Yang wrote: > Hi, > > I've updated baremetal-4.2 branch, added integration test for some of > bar

Re: [Review Request] Re-enabling baremetal on master

2013-06-20 Thread Sheng Yang
Hi, I've updated baremetal-4.2 branch, added integration test for some of baremetal related APIs, also fixed a bunch of baremetal API issues exposed by the testing. Thanks! --Sheng On Wed, Jun 19, 2013 at 11:41 AM, Chip Childers wrote: > On Wed, Jun 19, 2013 at 11:36:11AM -0700, Sheng Yang

Re: [Review Request] Re-enabling baremetal on master

2013-06-19 Thread Chip Childers
On Wed, Jun 19, 2013 at 11:36:11AM -0700, Sheng Yang wrote: > On Wed, Jun 19, 2013 at 11:24 AM, Chip Childers > wrote: > > > On Wed, Jun 19, 2013 at 11:00:33AM -0700, Sheng Yang wrote: > > > Hi, > > > > > > I've created the https://reviews.apache.org/r/11977/ for review. The > > > branch re-enabl

Re: [Review Request] Re-enabling baremetal on master

2013-06-19 Thread Sheng Yang
On Wed, Jun 19, 2013 at 11:24 AM, Chip Childers wrote: > On Wed, Jun 19, 2013 at 11:00:33AM -0700, Sheng Yang wrote: > > Hi, > > > > I've created the https://reviews.apache.org/r/11977/ for review. The > > branch re-enabled the baremetal for master. And all major bugs are > cleaned. > > > > https

Re: [Review Request] Re-enabling baremetal on master

2013-06-19 Thread Chip Childers
On Wed, Jun 19, 2013 at 11:00:33AM -0700, Sheng Yang wrote: > Hi, > > I've created the https://reviews.apache.org/r/11977/ for review. The > branch re-enabled the baremetal for master. And all major bugs are cleaned. > > https://issues.apache.org/jira/browse/CLOUDSTACK-1610 > https://issues.apac

[Review Request] Re-enabling baremetal on master

2013-06-19 Thread Sheng Yang
Hi, I've created the https://reviews.apache.org/r/11977/ for review. The branch re-enabled the baremetal for master. And all major bugs are cleaned. https://issues.apache.org/jira/browse/CLOUDSTACK-1610 https://issues.apache.org/jira/browse/CLOUDSTACK-1618 https://issues.apache.org/jira/browse/C