Marcus, that's the problem, this information is passed on regardless of iso
attachment state, which is not the case with nfs. So, probably a bug submission
is in order.
Andrei
----- Original Message -----
From: "Marcus Sorensen" <shadow...@gmail.com>
To: dev@cloudstack.apache.org
Sent: Friday, 20 December, 2013 4:38:11 PM
Subject: Re: [VOTE] 3rd round of voting for ASF 4.2.1 RC
Really quick, Andrei, the command looks like it's passing both the
root volume and an iso template as disks. I'm assuming you still have
the iso attached to the VM. If this is not the case, we probably need
to know about it in the bug submission as well. With the iso detached,
it should not show up in the VM config being passed and the migration
should work.
On Fri, Dec 20, 2013 at 9:32 AM, Marcus Sorensen <shadow...@gmail.com> wrote:
> Andrei,
> There are one or two spots in the KVM code that still assume that
> secondary storage is always NFS. In this particular example, it seems
> like maybe you're running into this:
>
> if (volume.getType() == Volume.Type.ISO && data.getPath() != null) {
> NfsTO nfsStore = (NfsTO)store;
>
> In this case, isos are assumed to be on NFS. Since we can't mount S3
> storage and use an iso directly, I assume it should instead be passing
> the NFS backing store location rather than the S3 origin of the
> template. Please open a ticket for it if one hasn't been already.
>
> Please keep in mind that when a new feature emerges, it's usually not
> perfect with the broad range of platforms and things you can do with
> each. In general, bugfix releases are only held up for regressions
> (since we don't want to make existing environments worse) or high
> priority/security fixes. Blocker bugs. Any other fixes that make it
> into a particular release are just nice to have. As such, new feature
> issues generally aren't deemed blockers since they're no worse than
> they started out. That said, if you raise an issue like this it can be
> fairly high priority for 4.3, and the fix can be backported for a
> future bugfix release for 4.2.
>
>
> On Fri, Dec 20, 2013 at 8:53 AM, Chip Childers <chipchild...@apache.org>
> wrote:
>> On Tue, Dec 17, 2013 at 01:49:14PM +0000, Abhinandan Prateek wrote:
>>> The 4.2.1 is re-spun mainly because the commit that was used to generate
>>> the previous RC did not get pushed to repo.
>>>
>>> Following are the particulars to vote for this time around:
>>>
>>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.2
>>>
>>> commit: 1b2b58fe352a19aee1721bd79b9d023d36e80ec5
>>>
>>> List of changes are available in Release Notes, a summary can be accessed
>>> here:
>>> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blob_plain;f=CHANGES;hb=4.2
>>>
>>>
>>> Source release revision 3911 (checksums and signatures are available at the
>>> same location):
>>> https://dist.apache.org/repos/dist/dev/cloudstack/4.2.1/
>>>
>>> PGP release keys (signed using RSA Key ID = 42443AA1):
>>> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>>>
>>> Vote will be open for 72 hours (until 20 Dec 2013 End of day PST).
>>>
>>> For sanity in tallying the vote, can PMC members please be sure to indicate
>>> "(binding)" with their vote?
>>>
>>> [ ] +1 approve
>>> [ ] +0 no opinion
>>> [ ] -1 disapprove (and reason why)
>>>
>>
>> +1 (binding)
>>
>> Testing limited to validation of the artifact checksums and signatures,
>> as well as a licensing check:
>>
>> AUTOMATED TESTING RESULTS:
>> [PASS] rm -Rf /tmp/cloudstack
>> [PASS] rm -Rf ~/.m2
>> [PASS] mkdir /tmp/cloudstack
>> [PASS] wget --no-check-certificate -q
>> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>> [PASS] wget --no-check-certificate -q
>> https://dist.apache.org/repos/dist/dev/cloudstack/4.2.1/apache-cloudstack-4.2.1-src.tar.bz2
>>
>> [PASS] wget --no-check-certificate -q
>> https://dist.apache.org/repos/dist/dev/cloudstack/4.2.1/apache-cloudstack-4.2.1-src.tar.bz2.asc
>>
>> [PASS] wget --no-check-certificate -q
>> https://dist.apache.org/repos/dist/dev/cloudstack/4.2.1/apache-cloudstack-4.2.1-src.tar.bz2.md5
>>
>> [PASS] wget --no-check-certificate -q
>> https://dist.apache.org/repos/dist/dev/cloudstack/4.2.1/apache-cloudstack-4.2.1-src.tar.bz2.sha
>>
>> [PASS] gpg --verify apache-cloudstack-4.2.1-src.tar.bz2.asc
>> [PASS] gpg --print-md MD5 apache-cloudstack-4.2.1-src.tar.bz2 | diff -
>> apache-cloudstack-4.2.1-src.tar.bz2.md5
>> [PASS] gpg --print-md SHA512 apache-cloudstack-4.2.1-src.tar.bz2 | diff -
>> apache-cloudstack-4.2.1-src.tar.bz2.sha
>> [PASS] mkdir /tmp/cloudstack/git
>> [PASS] mkdir /tmp/cloudstack/tree
>> [PASS] git clone -q https://git-wip-us.apache.org/repos/asf/cloudstack.git
>> /tmp/cloudstack/git
>> [PASS] git archive --prefix=/tmp/cloudstack/tree/
>> 1b2b58fe352a19aee1721bd79b9d023d36e80ec5 | tar Pxf -
>> [PASS] tar xvfj apache-cloudstack-4.2.1-src.tar.bz2
>> [PASS] diff -r /tmp/cloudstack/apache-cloudstack-4.2.1-src
>> /tmp/cloudstack/tree
>> [PASS] mvn --projects='org.apache.cloudstack:cloudstack'
>> org.apache.rat:apache-rat-plugin:0.8:check
