cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chip Childers <chip.child...@sungard.com>
Subject Re: Review Request: Temporarily disabling baremetal functionality in CS 4.1
Date Mon, 25 Mar 2013 16:36:04 GMT
On Mon, Mar 25, 2013 at 09:33:38AM -0700, Animesh Chaturvedi wrote:
> 
> 
> > -----Original Message-----
> > From: Chip Childers [mailto:chip.childers@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: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 "git fetch origin; git checkout 4.1; git reset --hard origin/4.1" before
"git
> > am ...".  I don't quite get what the conflict is.
> > 
> > Try the process above, including the git am using the patch available within
> > the review.  Perhaps the posting process is mangling things?
> > 
> [Animesh>] Sure Chip, we will try again and try to close it soon

Animesh,

If you can get a clean "git am", and the system builds, you can go ahead
and push the commit to origin/4.1.  I'm not trying to get in the way of
getting the commits into the branch...  just to confirm that things are
going in cleanly and that the build continues to work (and that only
appropriate changes are made).

> > 
> > >
> > > Thanks
> > > Animesh
> > >
> > > On Mar 24, 2013, at 11:08 AM, "Chip Childers"
> > <chip.childers@sungard.com> wrote:
> > >
> > > >
> > > > -----------------------------------------------------------
> > > > 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.  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 functionality
> > > > error: patch failed:
> > > > server/src/com/cloud/configuration/Config.java:359
> > > > error: server/src/com/cloud/configuration/Config.java: patch does
> > > > not apply Patch failed at 0001 CS-1773: Disable baremetal
> > > > functionality
> > > >
> > > > - Chip Childers
> > > >
> > > >
> > > > On March 22, 2013, 11:55 p.m., Venkata Siva Vijayendra Bhamidipati
> > 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.
> > > >>
> > > >>
> > > >> Description
> > > >> -------
> > > >>
> > > >> Temporarily disabling baremetal functionality in CS 4.1
> > > >>
> > > >>
> > > >> This addresses bug CS-1773.
> > > >>
> > > >>
> > > >> Diffs
> > > >> -----
> > > >>
> > > >>  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
> > > >
> > >
> 

Mime
View raw message