OK, I've gone ahead and updated Review Board with a new diff. Please check it out and let me know what you think.
Thanks! On Mon, Jun 17, 2013 at 2:52 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > Sounds good > > I have a little bit of clean-up work to do and then I must figure out how > to generate a patch (since I was using merge rather than rebase during > development), so it will probably be tomorrow. > > > On Mon, Jun 17, 2013 at 2:50 PM, John Burwell <jburw...@basho.com> wrote: > >> Mike, >> >> Great news. Let me know when Review Board is updated, and I will perform >> a second round review. >> >> Thanks, >> -John >> >> On Jun 17, 2013, at 4:48 PM, Mike Tutkowski <mike.tutkow...@solidfire.com> >> wrote: >> >> > FYI: I added the IOPS-capacity parameter and related code over the >> weekend. >> > >> > The final bit of work for me comes when Wei's code is merged into master >> > and I pull it down. >> > >> > At that point, I need to add the GUI and API logic to support mutual >> > exclusion of our features. >> > >> > >> > On Sat, Jun 15, 2013 at 11:12 AM, Chip Childers >> > <chip.child...@sungard.com>wrote: >> > >> >> On Fri, Jun 14, 2013 at 05:56:27PM -0400, John Burwell wrote: >> >>> Mike, >> >>> >> >>> Awesome. +1. >> >>> >> >>> Thanks for your patience with the back and forth, >> >>> -John >> >> >> >> +1 - this now looks like a great approach. >> >> >> > >> > >> > >> > -- >> > *Mike Tutkowski* >> > *Senior CloudStack Developer, SolidFire Inc.* >> > e: mike.tutkow...@solidfire.com >> > o: 303.746.7302 >> > Advancing the way the world uses the >> > cloud<http://solidfire.com/solution/overview/?video=play> >> > *™* >> >> > > > -- > *Mike Tutkowski* > *Senior CloudStack Developer, SolidFire Inc.* > e: mike.tutkow...@solidfire.com > o: 303.746.7302 > Advancing the way the world uses the > cloud<http://solidfire.com/solution/overview/?video=play> > *™* > -- *Mike Tutkowski* *Senior CloudStack Developer, SolidFire Inc.* e: mike.tutkow...@solidfire.com o: 303.746.7302 Advancing the way the world uses the cloud<http://solidfire.com/solution/overview/?video=play> *™*
