Re: Problem with CLOUDSTACK-10240 (Cannot migrate local volume to shared storage)
Mike, are you able to pin-point in the old/replaced code the bit that was
handling your use case? I took the most care not to break anything.
Also, your test case, isn't it in the ACS' integration test suite? In
theory, all test passed when we merged the PR.
I sure can take a look at it. Can you detail your use case? I mean, the
high level execution flow. What API methods you do, what you expected to
happen, and what is happening today.
On Sun, Jul 15, 2018 at 3:25 AM, Tutkowski, Mike <Mike.Tutkowski@xxxxxxxxxx>
> It looks like this is the problematic PR:
> On 7/15/18, 12:20 AM, "Tutkowski, Mike" <Mike.Tutkowski@xxxxxxxxxx> wrote:
> While running managed-storage regression tests tonight, I noticed a
> problem that is not related to managed storage.
> CLOUDSTACK-10240 is a ticket asking that we allow the migration of a
> virtual disk that’s on local storage to shared storage. In the process of
> enabling this feature, the VirtualMachineManagerImpl.
> getPoolListForVolumesForMigration method was re-written in a way that
> completely breaks at least one use case: Migrating a VM across compute
> clusters (at least supported in XenServer). If, say, a virtual disk resides
> on shared storage in the source compute cluster, we must be able to copy
> this virtual disk to shared storage in the destination compute cluster.
> As the code is currently written, this is no longer possible. It also
> seems that the managed-storage logic has been dropped for some reason in
> the new implementation.
> Rafael – It seems that you worked on this feature. Would you be able
> to look into this and create a PR?