I should be able to test it. Talk to you soon, Mike
On Sun, Aug 9, 2015 at 10:40 PM, Remi Bergsma <rberg...@schubergphilis.com> wrote: > Hi Mike, > > Thanks! It seems the testing / verification is still to do and that is the > work. Merging itself can be done with a one-liner. > > Are you able to verify the fix in your lab? > > Sent from my iPhone > > > On 10 Aug 2015, at 04:13, Mike Tutkowski <mike.tutkow...@solidfire.com> > wrote: > > > > I could take this one: > > > > CLOUDSTACK-8601. VMFS storage added as local storage can be re-added ... > > #547 > > https://github.com/apache/cloudstack/pull/547 > > ==> 1x LGTM > > > > I asked this in the PR discussion, but are we under the impression that > > this code has been tested well enough? If so, I can just merge it as the > > code itself LGTM. > > > > On Sun, Aug 9, 2015 at 12:49 PM, Remi Bergsma < > rberg...@schubergphilis.com> > > wrote: > > > >> Hi all, > >> > >> These are 9 PRs sent by Likitha. If I understand correctly Likitha is no > >> longer working with Citrix/ACS. The PRs seem mostly VMware related. Some > >> have LGTM(s), most have comments about missing unit tests. > >> > >> What do we want to do with them? There's probably not gonna be an update > >> from Likitha, so waiting for that does not make sense. Does anyone > wants to > >> step in and finalize a PR? You can get the PR to your own branch and add > >> some work on top of the existing commits and finally send it as a new > PR. > >> > >> I'll add a comment to each of them. If no one wants to take it over, I > >> think we should close the PRs without merging. It's a pity, but I would > >> rather not have long lists of orphaned PRs laying around. The less PRs > are > >> open, the better. > >> > >> Any comments? > >> > >> Regards, > >> Remi > >> > >> Cloudstack 8612 [VMware] #562 > >> https://github.com/apache/cloudstack/pull/562 > >> > >> CLOUDSTACK-8611. CS waits indefinitely for CheckS2SVpnConnectionsComm... > >> #561 > >> https://github.com/apache/cloudstack/pull/561 > >> > >> CLOUDSTACK-8609. [VMware] VM is not accessible after a migration acro... > >> #556 > >> https://github.com/apache/cloudstack/pull/556 > >> > >> CLOUDSTACK-8608. [VMware] System VM's failed to start due to permissions > >> issue. #555 > >> https://github.com/apache/cloudstack/pull/555 > >> > >> CLOUDSTACK-8610. Unable to attach 7th Disk to Windows Server 2012 R2 ... > >> #554 > >> https://github.com/apache/cloudstack/pull/554 > >> ==> This one has 2xLGTM, but also some remarks to add unit tests. > >> > >> CLOUDSTACK-8602. MigrateVirtualMachineWithVolume leaves old chain dat... > >> #548 > >> https://github.com/apache/cloudstack/pull/548 > >> ==> 1x LGTM > >> > >> CLOUDSTACK-8601. VMFS storage added as local storage can be re-added ... > >> #547 > >> https://github.com/apache/cloudstack/pull/547 > >> ==> 1x LGTM > >> > >> CLOUDSTACK-8599. CS reports failure for a successful migration. #544 > >> https://github.com/apache/cloudstack/pull/544 > >> > >> CLOUDSTACK-8415. SSVM shutdown during snapshot operation leaves behin... > >> #540 > >> https://github.com/apache/cloudstack/pull/540 > > > > > > -- > > *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>*™*