----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8399/#review14162 -----------------------------------------------------------
Hi James, good to see your patch. Pl. add Prasanna (tsp) and myself as the reviewers. tsp, help me review the patch. May be in future you can split them, so you can get it reviewed fast and get more git commit karma. - Rohit Yadav On Dec. 7, 2012, 4:53 p.m., James Martin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8399/ > ----------------------------------------------------------- > > (Updated Dec. 7, 2012, 4:53 p.m.) > > > Review request for cloudstack. > > > Description > ------- > > This diff cleans up the devcloud build process. > > Please see tools/devcloud/README.md for more information. > > I also have a fork of cloudstack available with this change: > > https://github.com/jsmartin/incubator-cloudstack/tree/devcloud-cleanup > > > Diffs > ----- > > > Diff: https://reviews.apache.org/r/8399/diff/ > > > Testing > ------- > > Built the veewee basebox and the vagrant xenbox. > Built a cloudstack vagrant box. > Was able to reach cloudstack at http://192.168.56.10:8080/client > > > Thanks, > > James Martin > >
