Re: [Feature Request] VM Snapshot based on KVM+Ceph

2016-07-12 Thread Wido den Hollander
> Op 12 juli 2016 om 5:58 schreef Paul Shadwell : > > > This is a feature that has been missing from Cloudstack since the beginning > when using KVM as the hypervisor. > It's been asked for many times but still no support. > This would be a huge plus for Cloudstack if this feature was added a

Re: [Feature Request] VM Snapshot based on KVM+Ceph

2016-07-12 Thread Wei ZHOU
Paul, If you mean VM snapshot on QCOW2, there is a PR on github for it. -Wei 2016-07-12 5:58 GMT+02:00 Paul Shadwell : > This is a feature that has been missing from Cloudstack since the > beginning when using KVM as the hypervisor. > It's been asked for many times but still no support. > This

[GitHub] cloudstack issue #1545: CLOUDSTACK-8715: Add channel to Instances for Qemu G...

2016-07-12 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1545 We have multiple LGTM. Are we good to merge it? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this fe

[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...

2016-07-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1610 @wido lgtm, though we would also want the same change in the centos6's spec file. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] cloudstack issue #1572: CLOUDSTACK-9395: Add Virtio RNG device to Instances ...

2016-07-12 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1572 No LGTM on this one yet. Are we good @ustcweizhou and @remibergsma ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...

2016-07-12 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1610 Ah, good point @rhtyd! Added it to CentOS 6 as well. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have th

[GitHub] cloudstack issue #1610: packaging: Add db.X.driver=jdbc:mysql to db.properti...

2016-07-12 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1610 👍LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

Re: XenServer 7

2016-07-12 Thread Erik Weber
I'm interested in knowing more about any efforts as well, there are several things in XS7 that we really want to use. -- Erik On Wed, May 25, 2016 at 2:59 PM, Paul Angus wrote: > Is anyone here working on XenServer 7 support for CloudStack? > > > Kind regards, > > Paul Angus > > > paul.an...@s

[GitHub] cloudstack pull request #1607: SSVM downloader now handles redirects properl...

2016-07-12 Thread ProjectMoon
GitHub user ProjectMoon reopened a pull request: https://github.com/apache/cloudstack/pull/1607 SSVM downloader now handles redirects properly. Previously it was using the HttpClient to make an initial request to an ISO. This would follow redirects. Then it would make another reques

[GitHub] cloudstack pull request #1607: SSVM downloader now handles redirects properl...

2016-07-12 Thread ProjectMoon
Github user ProjectMoon closed the pull request at: https://github.com/apache/cloudstack/pull/1607 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featu

[GitHub] cloudstack issue #1607: SSVM downloader now handles redirects properly.

2016-07-12 Thread ProjectMoon
Github user ProjectMoon commented on the issue: https://github.com/apache/cloudstack/pull/1607 Testing is a bit difficult without running a local web server (which I think is overkill for a unit test). Maybe a test at the integration level (Marvin) would work? Also, closed an

[GitHub] cloudstack issue #1607: SSVM downloader now handles redirects properly.

2016-07-12 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1607 Yes, I agree, hard to test this one with a Unit Test. Code changes look good to me. LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] cloudstack issue #1607: SSVM downloader now handles redirects properly.

2016-07-12 Thread ProjectMoon
Github user ProjectMoon commented on the issue: https://github.com/apache/cloudstack/pull/1607 Looks like all checks have passed successfully now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] cloudstack pull request #1600: Support Backup of Snapshots for Managed Stora...

2016-07-12 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1600#discussion_r70425148 --- Diff: api/src/org/apache/cloudstack/api/command/user/snapshot/CreateSnapshotCmd.java --- @@ -195,24 +196,37 @@ public void create() throws Resourc

[GitHub] cloudstack pull request #1609: [CLOUDSTACK-9430] Added fix for adding/editin...

2016-07-12 Thread pdube
Github user pdube closed the pull request at: https://github.com/apache/cloudstack/pull/1609 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request #1609: [CLOUDSTACK-9430] Added fix for adding/editin...

2016-07-12 Thread pdube
GitHub user pdube reopened a pull request: https://github.com/apache/cloudstack/pull/1609 [CLOUDSTACK-9430] Added fix for adding/editing Network ACL rule ordering BUG: https://issues.apache.org/jira/browse/CLOUDSTACK-9430 The issue occurred because all of the ACL rules get i

[GitHub] cloudstack issue #1609: [CLOUDSTACK-9430] Added fix for adding/editing Netwo...

2016-07-12 Thread pdube
Github user pdube commented on the issue: https://github.com/apache/cloudstack/pull/1609 The issue was with the teardown of a snapshots test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have th

Re: [Feature Request] VM Snapshot based on KVM+Ceph

2016-07-12 Thread Simon Weller
Wei's KVM QCOW2 snapshot PR works very well. I tested it a few months back - It would be great to get this into 4.9.10. In regards to the Ceph snapshot challenges, it is possible to emulate the behaviour you want (sans the RAM component). We pushed a PR a couple of months ago to enable root d

RE: [VOTE] Apache Cloudstack 4.9.0 RC1

2016-07-12 Thread Sean Lair
Hi all, I vote -1 and would like to see the jdbc:mysql and site-to-site vpn fixed in 4.9. https://github.com/apache/cloudstack/pull/1610 https://github.com/apache/cloudstack/pull/1480 Thanks! Sean -Original Message- From: Wido den Hollander [mailto:w...@widodh.nl] Sent: Tuesday, July 1

Re: XenServer 7

2016-07-12 Thread Raja Pullela
we are going to use XS7 for our testing and will be running BVTs/smoke tests against the same. Best, Raja Pullela Senior Manager, Product Development Accelerate, www.accelerite.com,@accelerite 2055, Laurelwood Road, Santa Clara, CA 95054, USA Phone: 1-408-216-7010 On 7/12/16, 2:17 PM, "Erik W

Re: [VOTE] Apache Cloudstack 4.9.0 RC1

2016-07-12 Thread Will Stevens
Thank you, so this RC has officially failed. Can you guys please review those PRs and give me code review so I can be confident with those PRs. I will run testing on those two PRs to make sure things don't break with them. Is there anything else that needs to be added to the next RC, please spea

Re: [VOTE] Apache Cloudstack 4.9.0 RC1

2016-07-12 Thread Simon Weller
Will, Here's another PR you might want to consider: https://github.com/apache/cloudstack/pull/1608 [https://avatars3.githubusercontent.com/u/1444686?v=3&s=400] Cleanup RBD contexts after exceptions to prevent potential agent crash by leprechau ·

Re: [VOTE] Apache Cloudstack 4.9.0 RC1

2016-07-12 Thread Wido den Hollander
Hi Will, These are on my list: - https://github.com/apache/cloudstack/pull/1610 - https://github.com/apache/cloudstack/pull/1480 - https://github.com/apache/cloudstack/pull/1608 Those 3 should fix all the -1 votes and discussion on the list. Wido > Op 12 juli 2016 om 18:14 schreef Will Stevens

Re: [VOTE] Apache Cloudstack 4.9.0 RC1

2016-07-12 Thread Will Stevens
Thank you gentlemen. I have been following the 1608 PR from afar and I will review your list Wido. I will get my CI chugging with some of those PRs today. Thx, *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|

Re: [Feature Request] VM Snapshot based on KVM+Ceph

2016-07-12 Thread ilya
Hi Paul Are you talking about Ceph + KVM snapshot or non Ceph filesystem using qemu/libvirtd? KVM Snapshot with non-ceph using qemu/libvirtd apis will be released in upcoming versions. We have a code written for it - but to release it to open source community takes a bit of time and might need a

Re: [Feature Request] VM Snapshot based on KVM+Ceph

2016-07-12 Thread ilya
I guess ENA beat us to it :) hats of to ENA folks! On 7/12/16 6:39 AM, Simon Weller wrote: > Wei's KVM QCOW2 snapshot PR works very well. I tested it a few months back - > It would be great to get this into 4.9.10. > > > In regards to the Ceph snapshot challenges, it is possible to emulate the

[GitHub] cloudstack pull request #1600: Support Backup of Snapshots for Managed Stora...

2016-07-12 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1600#discussion_r70563575 --- Diff: api/src/org/apache/cloudstack/api/response/SnapshotResponse.java --- @@ -82,6 +82,10 @@ @Param(description = "valid types are hourly,

Re: JIRA Permission

2016-07-12 Thread Rajani Karuturi
I added you to the contributors list. you should have the required permissions now. ~Rajani On Fri, Jul 8, 2016 at 2:37 PM, Priyank Parihar < priyank.pari...@accelerite.com> wrote: > Hi, > I am unable to assign issues (e.g. - CLOUDSTACK-9104) to me. Please give > me permission. > > -- > Thanks &

Re: XenServer 7

2016-07-12 Thread Erik Weber
Would you mind elaborating a bit what that means? Does it mean that you have already done some fixing to get it to work and will start testing it? Or that you plan on testing it now, to figure out what needs fixing? Or something else? I'm trying to figure out when I can expect to use ACS (or a co

Re: [VOTE] Apache Cloudstack 4.9.0 RC1

2016-07-12 Thread Milamber
Please add this PR on your list. That will fix the spanish localization display. Thanks https://github.com/apache/cloudstack/pull/1583 On 12/07/2016 19:17, Will Stevens wrote: Thank you gentlemen. I have been following the 1608 PR from afar and I will review your list Wido. I will get my C