Woops...I meant to provide the URL to the diff: https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commitdiff;h=2c8d179b7abf6da1c99390788c3329f243e172db;hp=abfe33a2441679b0105e2023472614a8648e6fcb
On Wed, Aug 5, 2015 at 10:48 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > Hi everyone, > > Hopefully this isn't an issue, but I occasionally have super trivial, but > useful changes to SolidFire-only code that I'd like to push without > bothering with a PR. > > For example: > > > https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob;f=plugins/storage/volume/solidfire/src/org/apache/cloudstack/storage/datastore/lifecycle/SolidFireSharedPrimaryDataStoreLifeCycle.java;h=7cb690014bbc4834c430febe050ccb133528e1fb;hb=2c8d179b7abf6da1c99390788c3329f243e172db > > In this commit, I renamed two variables to be more descriptive. It only > impacts the SolidFire plug-in and is purely for readability. > > Does this sound OK to do or do we literally want ever single change (no > matter how trivial) to go in via PR? > > Thanks! > > -- > *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>*™*