You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Chris Suich <ch...@netapp.com> on 2013/10/17 20:59:39 UTC
Review Request 14715: [CLOUDSTACK-4887] CLVM broken
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14715/
-----------------------------------------------------------
Review request for cloudstack, edison su and Marcus Sorensen.
Repository: cloudstack-git
Description
-------
Fixed a bug with backup snapshot condition logic causing snapshots to not be backed up and the API to fail.
Diffs
-----
core/src/com/cloud/storage/resource/StorageSubsystemCommandHandlerBase.java 002143f
Diff: https://reviews.apache.org/r/14715/diff/
Testing
-------
Thanks,
Chris Suich
Re: Review Request 14715: [CLOUDSTACK-4887] CLVM broken
Posted by edison su <ed...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14715/#review27164
-----------------------------------------------------------
Ship it!
Ship It!
- edison su
On Oct. 17, 2013, 6:59 p.m., Chris Suich wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14715/
> -----------------------------------------------------------
>
> (Updated Oct. 17, 2013, 6:59 p.m.)
>
>
> Review request for cloudstack, edison su and Marcus Sorensen.
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> Fixed a bug with backup snapshot condition logic causing snapshots to not be backed up and the API to fail.
>
>
> Diffs
> -----
>
> core/src/com/cloud/storage/resource/StorageSubsystemCommandHandlerBase.java 002143f
>
> Diff: https://reviews.apache.org/r/14715/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Chris Suich
>
>
Re: Review Request 14715: [CLOUDSTACK-4887] CLVM broken
Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14715/#review27157
-----------------------------------------------------------
Commit a504c004bf10555e5ea67ec89fe7bf6f00fe4622 in branch refs/heads/master from Marcus Sorensen
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=a504c00 ]
CLOUDSTACK-4887: Fix CLVM template download from storage refactoring work
- ASF Subversion and Git Services
On Oct. 17, 2013, 6:59 p.m., Chris Suich wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14715/
> -----------------------------------------------------------
>
> (Updated Oct. 17, 2013, 6:59 p.m.)
>
>
> Review request for cloudstack, edison su and Marcus Sorensen.
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> Fixed a bug with backup snapshot condition logic causing snapshots to not be backed up and the API to fail.
>
>
> Diffs
> -----
>
> core/src/com/cloud/storage/resource/StorageSubsystemCommandHandlerBase.java 002143f
>
> Diff: https://reviews.apache.org/r/14715/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Chris Suich
>
>