----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8688/#review14736 -----------------------------------------------------------
Ship it! commit 264a0670a85e4a8b36ddd17a38cbb8b2ec718946 Author: John Burwell <jburw...@basho.com> Date: Wed Dec 19 17:26:38 2012 -0500 - [CLOUDSTACK-509] S3-backed Secondary Storage - Adds S3 configuration support to Marvin - Moves configuration of global options to occur creation of zones since the values of global options impact the configuration and zone enablement processes - Prasanna Santhanam On Dec. 19, 2012, 10:34 p.m., John Burwell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8688/ > ----------------------------------------------------------- > > (Updated Dec. 19, 2012, 10:34 p.m.) > > > Review request for cloudstack. > > > Description > ------- > > Adds S3 configuration to Marvin. It also moves the configuration of global > parameters to occur before zone creation since the global parameters can > effect zone configuration. > > > This addresses bug CLOUDSTACK-509. > > > Diffs > ----- > > tools/marvin/marvin/deployDataCenter.py 7bdd975 > > Diff: https://reviews.apache.org/r/8688/diff/ > > > Testing > ------- > > Creating multiple zones with and without S3 configurations specified. > > > Thanks, > > John Burwell > >