On Fri, Mar 22, 2013 at 05:35:56PM -0700, Angeline Shen wrote: > Chip, Vijay: > > Let me know When the build is ready, I can help test if baremetal feature > has been deactivated from UI.
The patch doesn't apply cleanly. > > -----Original Message----- > From: Chip Childers [mailto:chip.child...@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 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, visit: > > https://reviews.apache.org/r/10071/ > > ----------------------------------------------------------- > > > > (Updated March 22, 2013, 11:55 p.m.) > > > > > > Review request for cloudstack, Kelven Yang and Frank Zhang. > > > > > > Changes > > ------- > > > > Hi Chip! > > > > Reuploading a consolidated patch with the sql changes as well, plus another > > change in the plugins/pom.xml file - this is on top of the 4.1 branch - > > generated using git format-patch.. can you please check and let me if it > > applies? Thanks! > > > > Regards, > > Vijay > > > > > > Description > > ------- > > > > Temporarily disabling baremetal functionality in CS 4.1 > > > > > > This addresses bug CS-1773. > > > > > > Diffs (updated) > > ----- > > > > client/pom.xml 38ba405 > > client/tomcatconf/componentContext.xml.in ff7376e > > client/tomcatconf/nonossComponentContext.xml.in 35e1b28 > > plugins/pom.xml 02459b4 > > server/src/com/cloud/configuration/Config.java 17fe62b > > setup/db/db/schema-40to410.sql 754bfb6 > > > > Diff: https://reviews.apache.org/r/10071/diff/ > > > > > > Testing > > ------- > > > > Baremetal and Cisco UCS related APIs confirmed to not be exposed to clients. > > > > > > Thanks, > > > > Venkata Siva Vijayendra Bhamidipati > > >
