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
