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

[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

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

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: [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: [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

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=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 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

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

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

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

[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

[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

[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 #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

[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

[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

[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

[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

[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

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, > >

[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

[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

[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 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.

[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

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

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

RE: [VOTE] Apache Cloudstack 4.9.0 RC1

2016-07-12 Thread Wido den Hollander
> Op 11 juli 2016 om 22:40 schreef Sean Lair : > > > Hi all, > > One small comment since strongSwan didn't make it into 4.9. There is still a > very simple bug in enabling PFS for site-to-site VPNs. The code checks the > Dead Peer Detection (DPD) variable instead of

RE: [VOTE] Apache Cloudstack 4.9.0 RC1

2016-07-12 Thread Wido den Hollander
> Op 11 juli 2016 om 20:33 schreef Paul Angus : > > > Hey Will, > > I was referring to the need to add: > > > db.cloud.driver=jdbc:mysql > > db.usage.driver=jdbc:mysql > > db.simulator.driver=jdbc:mysql > > to db.properties when upgrading. > > Asking admins to

[GitHub] cloudstack pull request #1610: packaging: Add db.X.driver=jdbc:mysql to db.p...

2016-07-12 Thread wido
GitHub user wido opened a pull request: https://github.com/apache/cloudstack/pull/1610 packaging: Add db.X.driver=jdbc:mysql to db.properties on upgrade This is required afther the upgrade to 4.9.0 and for convience we add this to the configuration so our users do not have to.