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 and make 
> it a much more viable product for end users. 
> KVM is usually chosen because 1: it's free 2: it has many features that make 
> it a much better choice for large installations. 
> Many have ended up either writing their own VM snapshot solution, or simply 
> hiding the icon from the customer. 
> I'm surprised that after years of waiting for this feature to be added, it's 
> still missing. 
> I know it's not totally the Cloudstack Dev team's fault that VM Snapshots are 
> missing from KVM support but I do think this is a list opportunity for a big 
> win if they did something about it. 
> 
> Just my 2 penneth on this subject that has plagued me on a daily basis 
> supporting Cloudstack installs. 
> 

The problem is that KVM ONLY supports full VM snapshots with QCOW2 and not with 
RBD. You have to store the memory contents somewhere and only QCOW2 supports it.

An true, it's a lack of time. A lot of stuff needs to be done.

Wido

> Regards
> Paul
> 
> 
> > On Jul 12, 2016, at 05:19, ilya  wrote:
> > 
> > Just FYI:
> > 
> > You can ask on this list for sponsored feature development if this is
> > something that needs prompt resolution.
> > 
> > Regards
> > ilya
> > 
> >> On 7/8/16 7:07 PM, 吕海蛟 wrote:
> >> Hi,  Developers
> >> 
> >> 
> >> 
> >> We deployed ACS+KVM+Ceph in our environment.  Everything looks fine
> >> except the VM snapshot feature (not volume backup) is missing in current
> >> ACS version which seems only supported by XenServer/ESXi.
> >> 
> >> 
> >> 
> >> Wondering if we can have this on the roadmap as well.  @Wido, do you
> >> have any plan ?
> >> 
> >> 
> >> 
> >> Thanks !
> >> 
> >> 
> >> 
> >> 
> >> 
> >> 
> >> 
> >> 
> >> 
> >> private cloud2  *We Deliver Enterprise-Grade Cloud !*
> >> 
> >> --
> >> 
> >> *吕海蛟*
> >> 
> >> *Product Engineering & Innovation Center (PEIC)*
> >> 
> >> *华胜蓝泰科技(天津)有限责任公司*
> >> 
> >> * *
> >> 
> >> *上海闵行区合川路**2679**号虹桥国际商务广场**B**栋**601**室*
> >> 
> >> *邮编:**201103*
> >> 
> >> *Office: +86-21-62351222*
> >> 
> >> *Mobile: +86-18602198181*
> >> 
> >> 
> >> 
> >> 
> >> 
>


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 would be a huge plus for Cloudstack if this feature was added and
> make it a much more viable product for end users.
> KVM is usually chosen because 1: it's free 2: it has many features that
> make it a much better choice for large installations.
> Many have ended up either writing their own VM snapshot solution, or
> simply hiding the icon from the customer.
> I'm surprised that after years of waiting for this feature to be added,
> it's still missing.
> I know it's not totally the Cloudstack Dev team's fault that VM Snapshots
> are missing from KVM support but I do think this is a list opportunity for
> a big win if they did something about it.
>
> Just my 2 penneth on this subject that has plagued me on a daily basis
> supporting Cloudstack installs.
>
> Regards
> Paul
>
>
> > On Jul 12, 2016, at 05:19, ilya  wrote:
> >
> > Just FYI:
> >
> > You can ask on this list for sponsored feature development if this is
> > something that needs prompt resolution.
> >
> > Regards
> > ilya
> >
> >> On 7/8/16 7:07 PM, 吕海蛟 wrote:
> >> Hi,  Developers
> >>
> >>
> >>
> >> We deployed ACS+KVM+Ceph in our environment.  Everything looks fine
> >> except the VM snapshot feature (not volume backup) is missing in current
> >> ACS version which seems only supported by XenServer/ESXi.
> >>
> >>
> >>
> >> Wondering if we can have this on the roadmap as well.  @Wido, do you
> >> have any plan ?
> >>
> >>
> >>
> >> Thanks !
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >> private cloud2  *We Deliver Enterprise-Grade Cloud !*
> >>
> >>
> --
> >>
> >> *吕海蛟*
> >>
> >> *Product Engineering & Innovation Center (PEIC)*
> >>
> >> *华胜蓝泰科技(天津)有限责任公司*
> >>
> >> * *
> >>
> >> *上海闵行区合川路**2679**号虹桥国际商务广场**B**栋**601**室*
> >>
> >> *邮编:**201103*
> >>
> >> *Office: +86-21-62351222*
> >>
> >> *Mobile: +86-18602198181*
> >>
> >>
> >>
> >>
> >>
>
>


[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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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...@shapeblue.com
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> @shapeblue
>
>
>


[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 request using 
built-in Java URL and InputStream, which doesn't follow redirects. This results 
in the ISO getting stuck at 0% forever and also causing DOS effects.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/greenqloud/cloudstack pr-iso-follow-redirects

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1607.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1607


commit 14803529705f17194a6eee92187cc572e8741f47
Author: jeff 
Date:   2016-07-08T09:59:08Z

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 request using built-in Java URL and InputStream, which
doesn't follow redirects. This results in the ISO getting stuck
at 0% forever and also causing DOS effects.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 and reopened the PR to get it to build/validate again.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 
ResourceAllocationException {
 
 @Override
 public void execute() {
-s_logger.info("VOLSS: createSnapshotCmd starts:" + 
System.currentTimeMillis());
-CallContext.current().setEventDetails("Volume Id: " + 
getVolumeUuid());
 Snapshot snapshot;
 try {
 snapshot =
-_volumeService.takeSnapshot(getVolumeId(), getPolicyId(), 
getEntityId(), _accountService.getAccount(getEntityOwnerId()), getQuiescevm());
+_volumeService.takeSnapshot(getVolumeId(), getPolicyId(), 
getEntityId(), _accountService.getAccount(getEntityOwnerId()), getQuiescevm(), 
getLocationType());
+
 if (snapshot != null) {
 SnapshotResponse response = 
_responseGenerator.createSnapshotResponse(snapshot);
 response.setResponseName(getCommandName());
 setResponseObject(response);
 } else {
-throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR, 
"Failed to create snapshot due to an internal error creating snapshot for 
volume " + volumeId);
+throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR, 
"Failed to create snapshot due to an internal error creating snapshot for 
volume " + getVolumeId());
 }
 } catch (Exception e) {
-throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR, 
"Failed to create snapshot due to an internal error creating snapshot for 
volume " + volumeId);
+throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR, 
"Failed to create snapshot due to an internal error creating snapshot for 
volume " + getVolumeId());
 }
 }
 
+private Snapshot.LocationType getLocationType() {
+if (Snapshot.LocationType.values() == null || 
Snapshot.LocationType.values().length == 0) {
+return null;
+}
+
+Snapshot.LocationType locationTypeToReturn = 
Snapshot.LocationType.values()[0];
+
+if (locationType != null && locationType >= 1 && locationType <= 
Snapshot.LocationType.values().length) {
+locationTypeToReturn = 
Snapshot.LocationType.values()[locationType-1];
--- End diff --

I would suggest keeping the enum values all upper case, make locationType 
string in cmd and use LocationType.valueOf(cmd.locationType.toUpperCase())

https://books.google.co.in/books?id=ka2VUBqHiWkC&lpg=PA158&dq=Never%20derive%20a%20value%20associated%20with%20an%20enum%20from%20its%20ordinal&pg=PA158#v=onepage&q&f=false


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 inserted before the old 
ones. Then, the cleanup deletes the duplicate rows, and leaves any new rule in 
front of the old ones.

Here is an example with a simplified iptables view for ACL
Ex: adding a rule 4
before add:
1,2,3

during add:
1',2',3',4',1,2,3

after add:
4',1,2,3

After fix:
before add:
1,2,3

during add:
1,2,3,1',2',3',4'

after add:
1',2',3',4'


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/pdube/cloudstack network-acl-add-order

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1609.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1609


commit 6dd6ef0c9a30f128657de22fa8b6ea8ab5daa133
Author: Patrick Dube 
Date:   2016-07-11T19:12:41Z

Added fix for adding/editing Network ACL rule ordering




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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 detach of volumes on KVM (It has been merged into 
4.7-forward, 4.8-forward and 4.9 RC).

We use it to emulate a snapshot revert with Ceph by using createVolume sourced 
from a snapshot, then detaching and reattach the root volume of a VM with 
device id of 0.
This preserves the previous volume history and allows the user to switch back 
and forth between different snapshots.


We would like a fully featured "VM + memory" Ceph based snapshot as well, but 
as Wido has indicated, a lot has to happen before that will be possible and we 
have more research to do in order to understand the challenges. I would guess 
Redhat is going to be very interested in getting this into libvirt, as they are 
obviously pushing Ceph aggressively.


- Si


From: Wei ZHOU 
Sent: Tuesday, July 12, 2016 2:25 AM
To: dev@cloudstack.apache.org
Subject: Re: [Feature Request] VM Snapshot based on KVM+Ceph

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 would be a huge plus for Cloudstack if this feature was added and
> make it a much more viable product for end users.
> KVM is usually chosen because 1: it's free 2: it has many features that
> make it a much better choice for large installations.
> Many have ended up either writing their own VM snapshot solution, or
> simply hiding the icon from the customer.
> I'm surprised that after years of waiting for this feature to be added,
> it's still missing.
> I know it's not totally the Cloudstack Dev team's fault that VM Snapshots
> are missing from KVM support but I do think this is a list opportunity for
> a big win if they did something about it.
>
> Just my 2 penneth on this subject that has plagued me on a daily basis
> supporting Cloudstack installs.
>
> Regards
> Paul
>
>
> > On Jul 12, 2016, at 05:19, ilya  wrote:
> >
> > Just FYI:
> >
> > You can ask on this list for sponsored feature development if this is
> > something that needs prompt resolution.
> >
> > Regards
> > ilya
> >
> >> On 7/8/16 7:07 PM, ??? wrote:
> >> Hi,  Developers
> >>
> >>
> >>
> >> We deployed ACS+KVM+Ceph in our environment.  Everything looks fine
> >> except the VM snapshot feature (not volume backup) is missing in current
> >> ACS version which seems only supported by XenServer/ESXi.
> >>
> >>
> >>
> >> Wondering if we can have this on the roadmap as well.  @Wido, do you
> >> have any plan ?
> >>
> >>
> >>
> >> Thanks !
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >>
> >> private cloud2  *We Deliver Enterprise-Grade Cloud !*
> >>
> >>
> --
> >>
> >> *???*
> >>
> >> *Product Engineering & Innovation Center (PEIC)*
> >>
> >> *??(??)??*
> >>
> >> * *
> >>
> >> ***2679**?**B**?**601**?*
> >>
> >> *??:**201103*
> >>
> >> *Office: +86-21-62351222*
> >>
> >> *Mobile: +86-18602198181*
> >>
> >>
> >>
> >>
> >>
>
>


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 12, 2016 1:48 AM
To: Sean Lair ; dev@cloudstack.apache.org
Subject: RE: [VOTE] Apache Cloudstack 4.9.0 RC1


> 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 the PFS variable when 
> determining whether or not to enable PFS for the site-to-site VPN.
> 
> Here is the 1-line of code that is broken.  You can see how it refers to dpd 
> to set pfs.
> 
> file.addeq(" pfs=%s" % CsHelper.bool_to_yn(obj['dpd']))
> 
> This pull request fixes the issue, but was not merged since we were going to 
> strongSwan.  It would be nice if this bug fix was put into 4.9.0
> 
> https://github.com/apache/cloudstack/pull/1480
> 

Would it make you a -1 for you without this PR? If so, please vote -1 :)

Wido

> 
> Thanks!
> Sean
> 
> -Original Message-
> From: Will Stevens [mailto:williamstev...@gmail.com] 
> Sent: Wednesday, July 6, 2016 3:52 PM
> To: dev@cloudstack.apache.org
> Subject: [VOTE] Apache Cloudstack 4.9.0 RC1
> 
> Hi All,
> 
> I've created a 4.9.0 release, with the following artifacts up for a vote:
> 
> Git Branch and Commit SH:
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.9.0-RC20160706T1546
> Commit: 643f75aa9150156b1fb05f339a338614fc7ad3fb
> 
> I will be updating the Release Notes with the changes in this release 
> tomorrow.  If the RC changes, I can adapt the release notes after.
> 
> Source release (checksums and signatures are available at the same
> location):
> https://dist.apache.org/repos/dist/dev/cloudstack/4.9.0/
> 
> PGP release keys (signed using CB818F64):
> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
> 
> Vote will be open for 72 hours.
> 
> 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)
> 
> Thanks,
> 
> Will


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 Weber"  wrote:

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...@shapeblue.com
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> @shapeblue
>
>
>





DISCLAIMER
==
This e-mail may contain privileged and confidential information which is the 
property of Accelerite, a Persistent Systems business. It is intended only for 
the use of the individual or entity to which it is addressed. If you are not 
the intended recipient, you are not authorized to read, retain, copy, print, 
distribute or use this message. If you have received this communication in 
error, please notify the sender and delete all copies of this message. 
Accelerite, a Persistent Systems business does not accept any liability for 
virus infected mails.


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 speak
now so we can streamline this next RC.

Thanks,

*Will STEVENS*
Lead Developer

*CloudOps* *| *Cloud Solutions Experts
420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
w cloudops.com *|* tw @CloudOps_

On Tue, Jul 12, 2016 at 10:53 AM, Sean Lair  wrote:

> 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 12, 2016 1:48 AM
> To: Sean Lair ; dev@cloudstack.apache.org
> Subject: RE: [VOTE] Apache Cloudstack 4.9.0 RC1
>
>
> > 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 the PFS variable
> when determining whether or not to enable PFS for the site-to-site VPN.
> >
> > Here is the 1-line of code that is broken.  You can see how it refers to
> dpd to set pfs.
> >
> > file.addeq(" pfs=%s" % CsHelper.bool_to_yn(obj['dpd']))
> >
> > This pull request fixes the issue, but was not merged since we were
> going to strongSwan.  It would be nice if this bug fix was put into 4.9.0
> >
> > https://github.com/apache/cloudstack/pull/1480
> >
>
> Would it make you a -1 for you without this PR? If so, please vote -1 :)
>
> Wido
>
> >
> > Thanks!
> > Sean
> >
> > -Original Message-
> > From: Will Stevens [mailto:williamstev...@gmail.com]
> > Sent: Wednesday, July 6, 2016 3:52 PM
> > To: dev@cloudstack.apache.org
> > Subject: [VOTE] Apache Cloudstack 4.9.0 RC1
> >
> > Hi All,
> >
> > I've created a 4.9.0 release, with the following artifacts up for a vote:
> >
> > Git Branch and Commit SH:
> >
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.9.0-RC20160706T1546
> > Commit: 643f75aa9150156b1fb05f339a338614fc7ad3fb
> >
> > I will be updating the Release Notes with the changes in this release
> tomorrow.  If the RC changes, I can adapt the release notes after.
> >
> > Source release (checksums and signatures are available at the same
> > location):
> > https://dist.apache.org/repos/dist/dev/cloudstack/4.9.0/
> >
> > PGP release keys (signed using CB818F64):
> > https://dist.apache.org/repos/dist/release/cloudstack/KEYS
> >
> > Vote will be open for 72 hours.
> >
> > 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)
> >
> > Thanks,
> >
> > 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&s=400]

Cleanup RBD contexts after exceptions to prevent potential agent crash by 
leprechau · Pull Request #1608 · 
apache/cloudstack
github.com
We noticed that when an exception occurs within the cleanup loop inside the 
deletePhysicalDisk routine that the previously allocated contexts are not 
cleaned up. This seemed to cause an eventual c...

It fixes a segfault in the agent that we discovered when we had a misconfigured 
ceph ACL and also greatly improves logging.

I see pdube also posted this PR: https://github.com/apache/cloudstack/pull/1609

We've pulled it in for testing.

- Si





- Si



From: williamstev...@gmail.com  on behalf of Will 
Stevens 
Sent: Tuesday, July 12, 2016 11:14 AM
To: dev@cloudstack.apache.org
Subject: Re: [VOTE] Apache Cloudstack 4.9.0 RC1

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 speak
now so we can streamline this next RC.

Thanks,

*Will STEVENS*
Lead Developer

*CloudOps* *| *Cloud Solutions Experts
420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
w cloudops.com *|* tw @CloudOps_

On Tue, Jul 12, 2016 at 10:53 AM, Sean Lair  wrote:

> 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 12, 2016 1:48 AM
> To: Sean Lair ; dev@cloudstack.apache.org
> Subject: RE: [VOTE] Apache Cloudstack 4.9.0 RC1
>
>
> > 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 the PFS variable
> when determining whether or not to enable PFS for the site-to-site VPN.
> >
> > Here is the 1-line of code that is broken.  You can see how it refers to
> dpd to set pfs.
> >
> > file.addeq(" pfs=%s" % CsHelper.bool_to_yn(obj['dpd']))
> >
> > This pull request fixes the issue, but was not merged since we were
> going to strongSwan.  It would be nice if this bug fix was put into 4.9.0
> >
> > https://github.com/apache/cloudstack/pull/1480
> >
>
> Would it make you a -1 for you without this PR? If so, please vote -1 :)
>
> Wido
>
> >
> > Thanks!
> > Sean
> >
> > -Original Message-
> > From: Will Stevens [mailto:williamstev...@gmail.com]
> > Sent: Wednesday, July 6, 2016 3:52 PM
> > To: dev@cloudstack.apache.org
> > Subject: [VOTE] Apache Cloudstack 4.9.0 RC1
> >
> > Hi All,
> >
> > I've created a 4.9.0 release, with the following artifacts up for a vote:
> >
> > Git Branch and Commit SH:
> >
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.9.0-RC20160706T1546
> > Commit: 643f75aa9150156b1fb05f339a338614fc7ad3fb
> >
> > I will be updating the Release Notes with the changes in this release
> tomorrow.  If the RC changes, I can adapt the release notes after.
> >
> > Source release (checksums and signatures are available at the same
> > location):
> > https://dist.apache.org/repos/dist/dev/cloudstack/4.9.0/
> >
> > PGP release keys (signed using CB818F64):
> > https://dist.apache.org/repos/dist/release/cloudstack/KEYS
> >
> > Vote will be open for 72 hours.
> >
> > 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)
> >
> > Thanks,
> >
> > Will
>


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 :
> 
> 
> 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 speak
> now so we can streamline this next RC.
> 
> Thanks,
> 
> *Will STEVENS*
> Lead Developer
> 
> *CloudOps* *| *Cloud Solutions Experts
> 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
> w cloudops.com *|* tw @CloudOps_
> 
> On Tue, Jul 12, 2016 at 10:53 AM, Sean Lair  wrote:
> 
> > 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 12, 2016 1:48 AM
> > To: Sean Lair ; dev@cloudstack.apache.org
> > Subject: RE: [VOTE] Apache Cloudstack 4.9.0 RC1
> >
> >
> > > 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 the PFS variable
> > when determining whether or not to enable PFS for the site-to-site VPN.
> > >
> > > Here is the 1-line of code that is broken.  You can see how it refers to
> > dpd to set pfs.
> > >
> > > file.addeq(" pfs=%s" % CsHelper.bool_to_yn(obj['dpd']))
> > >
> > > This pull request fixes the issue, but was not merged since we were
> > going to strongSwan.  It would be nice if this bug fix was put into 4.9.0
> > >
> > > https://github.com/apache/cloudstack/pull/1480
> > >
> >
> > Would it make you a -1 for you without this PR? If so, please vote -1 :)
> >
> > Wido
> >
> > >
> > > Thanks!
> > > Sean
> > >
> > > -Original Message-
> > > From: Will Stevens [mailto:williamstev...@gmail.com]
> > > Sent: Wednesday, July 6, 2016 3:52 PM
> > > To: dev@cloudstack.apache.org
> > > Subject: [VOTE] Apache Cloudstack 4.9.0 RC1
> > >
> > > Hi All,
> > >
> > > I've created a 4.9.0 release, with the following artifacts up for a vote:
> > >
> > > Git Branch and Commit SH:
> > >
> > https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.9.0-RC20160706T1546
> > > Commit: 643f75aa9150156b1fb05f339a338614fc7ad3fb
> > >
> > > I will be updating the Release Notes with the changes in this release
> > tomorrow.  If the RC changes, I can adapt the release notes after.
> > >
> > > Source release (checksums and signatures are available at the same
> > > location):
> > > https://dist.apache.org/repos/dist/dev/cloudstack/4.9.0/
> > >
> > > PGP release keys (signed using CB818F64):
> > > https://dist.apache.org/repos/dist/release/cloudstack/KEYS
> > >
> > > Vote will be open for 72 hours.
> > >
> > > 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)
> > >
> > > Thanks,
> > >
> > > Will
> >


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 *|* tw @CloudOps_

On Tue, Jul 12, 2016 at 1:07 PM, Wido den Hollander  wrote:

> 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 :
> >
> >
> > 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
> speak
> > now so we can streamline this next RC.
> >
> > Thanks,
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > *CloudOps* *| *Cloud Solutions Experts
> > 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
> > w cloudops.com *|* tw @CloudOps_
> >
> > On Tue, Jul 12, 2016 at 10:53 AM, Sean Lair 
> wrote:
> >
> > > 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 12, 2016 1:48 AM
> > > To: Sean Lair ; dev@cloudstack.apache.org
> > > Subject: RE: [VOTE] Apache Cloudstack 4.9.0 RC1
> > >
> > >
> > > > 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 the PFS
> variable
> > > when determining whether or not to enable PFS for the site-to-site VPN.
> > > >
> > > > Here is the 1-line of code that is broken.  You can see how it
> refers to
> > > dpd to set pfs.
> > > >
> > > > file.addeq(" pfs=%s" % CsHelper.bool_to_yn(obj['dpd']))
> > > >
> > > > This pull request fixes the issue, but was not merged since we were
> > > going to strongSwan.  It would be nice if this bug fix was put into
> 4.9.0
> > > >
> > > > https://github.com/apache/cloudstack/pull/1480
> > > >
> > >
> > > Would it make you a -1 for you without this PR? If so, please vote -1
> :)
> > >
> > > Wido
> > >
> > > >
> > > > Thanks!
> > > > Sean
> > > >
> > > > -Original Message-
> > > > From: Will Stevens [mailto:williamstev...@gmail.com]
> > > > Sent: Wednesday, July 6, 2016 3:52 PM
> > > > To: dev@cloudstack.apache.org
> > > > Subject: [VOTE] Apache Cloudstack 4.9.0 RC1
> > > >
> > > > Hi All,
> > > >
> > > > I've created a 4.9.0 release, with the following artifacts up for a
> vote:
> > > >
> > > > Git Branch and Commit SH:
> > > >
> > >
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.9.0-RC20160706T1546
> > > > Commit: 643f75aa9150156b1fb05f339a338614fc7ad3fb
> > > >
> > > > I will be updating the Release Notes with the changes in this release
> > > tomorrow.  If the RC changes, I can adapt the release notes after.
> > > >
> > > > Source release (checksums and signatures are available at the same
> > > > location):
> > > > https://dist.apache.org/repos/dist/dev/cloudstack/4.9.0/
> > > >
> > > > PGP release keys (signed using CB818F64):
> > > > https://dist.apache.org/repos/dist/release/cloudstack/KEYS
> > > >
> > > > Vote will be open for 72 hours.
> > > >
> > > > 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)
> > > >
> > > > Thanks,
> > > >
> > > > Will
> > >
>


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

Regards
ilya

On 7/11/16 8:58 PM, Paul Shadwell wrote:
> 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 and make 
> it a much more viable product for end users. 
> KVM is usually chosen because 1: it's free 2: it has many features that make 
> it a much better choice for large installations. 
> Many have ended up either writing their own VM snapshot solution, or simply 
> hiding the icon from the customer. 
> I'm surprised that after years of waiting for this feature to be added, it's 
> still missing. 
> I know it's not totally the Cloudstack Dev team's fault that VM Snapshots are 
> missing from KVM support but I do think this is a list opportunity for a big 
> win if they did something about it. 
> 
> Just my 2 penneth on this subject that has plagued me on a daily basis 
> supporting Cloudstack installs. 
> 
> Regards
> Paul
> 
> 
>> On Jul 12, 2016, at 05:19, ilya  wrote:
>>
>> Just FYI:
>>
>> You can ask on this list for sponsored feature development if this is
>> something that needs prompt resolution.
>>
>> Regards
>> ilya
>>
>>> On 7/8/16 7:07 PM, 吕海蛟 wrote:
>>> Hi,  Developers
>>>
>>>
>>>
>>> We deployed ACS+KVM+Ceph in our environment.  Everything looks fine
>>> except the VM snapshot feature (not volume backup) is missing in current
>>> ACS version which seems only supported by XenServer/ESXi.
>>>
>>>
>>>
>>> Wondering if we can have this on the roadmap as well.  @Wido, do you
>>> have any plan ?
>>>
>>>
>>>
>>> Thanks !
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> private cloud2  *We Deliver Enterprise-Grade Cloud !*
>>>
>>> --
>>>
>>> *吕海蛟*
>>>
>>> *Product Engineering & Innovation Center (PEIC)*
>>>
>>> *华胜蓝泰科技(天津)有限责任公司*
>>>
>>> * *
>>>
>>> *上海闵行区合川路**2679**号虹桥国际商务广场**B**栋**601**室*
>>>
>>> *邮编:**201103*
>>>
>>> *Office: +86-21-62351222*
>>>
>>> *Mobile: +86-18602198181*
>>>
>>>
>>>
>>>
>>>
> 


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 
> behaviour you want (sans the RAM component). We pushed a PR a couple of 
> months ago to enable root detach of volumes on KVM (It has been merged into 
> 4.7-forward, 4.8-forward and 4.9 RC).
> 
> We use it to emulate a snapshot revert with Ceph by using createVolume 
> sourced from a snapshot, then detaching and reattach the root volume of a VM 
> with device id of 0.
> This preserves the previous volume history and allows the user to switch back 
> and forth between different snapshots.
> 
> 
> We would like a fully featured "VM + memory" Ceph based snapshot as well, but 
> as Wido has indicated, a lot has to happen before that will be possible and 
> we have more research to do in order to understand the challenges. I would 
> guess Redhat is going to be very interested in getting this into libvirt, as 
> they are obviously pushing Ceph aggressively.
> 
> 
> - Si
> 
> 
> From: Wei ZHOU 
> Sent: Tuesday, July 12, 2016 2:25 AM
> To: dev@cloudstack.apache.org
> Subject: Re: [Feature Request] VM Snapshot based on KVM+Ceph
> 
> 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 would be a huge plus for Cloudstack if this feature was added and
>> make it a much more viable product for end users.
>> KVM is usually chosen because 1: it's free 2: it has many features that
>> make it a much better choice for large installations.
>> Many have ended up either writing their own VM snapshot solution, or
>> simply hiding the icon from the customer.
>> I'm surprised that after years of waiting for this feature to be added,
>> it's still missing.
>> I know it's not totally the Cloudstack Dev team's fault that VM Snapshots
>> are missing from KVM support but I do think this is a list opportunity for
>> a big win if they did something about it.
>>
>> Just my 2 penneth on this subject that has plagued me on a daily basis
>> supporting Cloudstack installs.
>>
>> Regards
>> Paul
>>
>>
>>> On Jul 12, 2016, at 05:19, ilya  wrote:
>>>
>>> Just FYI:
>>>
>>> You can ask on this list for sponsored feature development if this is
>>> something that needs prompt resolution.
>>>
>>> Regards
>>> ilya
>>>
 On 7/8/16 7:07 PM, ??? wrote:
 Hi,  Developers



 We deployed ACS+KVM+Ceph in our environment.  Everything looks fine
 except the VM snapshot feature (not volume backup) is missing in current
 ACS version which seems only supported by XenServer/ESXi.



 Wondering if we can have this on the roadmap as well.  @Wido, do you
 have any plan ?



 Thanks !









 private cloud2  *We Deliver Enterprise-Grade Cloud !*


>> --

 *???*

 *Product Engineering & Innovation Center (PEIC)*

 *??(??)??*

 * *

 ***2679**?**B**?**601**?*

 *??:**201103*

 *Office: +86-21-62351222*

 *Mobile: +86-18602198181*





>>
>>
> 


[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, daily, weekly, monthy, 
template, and none.")
 private String intervalType;
 
+@SerializedName(ApiConstants.LOCATION_TYPE)
+@Param(description = "valid location types are primary and archive.")
+private String locationType;
--- End diff --

LocationType is a String in response. It makes sense for it be String in 
request 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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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 & Regards
> Priyank  Parihar
>
>
>
>
> DISCLAIMER
> ==
> This e-mail may contain privileged and confidential information which is
> the property of Accelerite, a Persistent Systems business. It is intended
> only for the use of the individual or entity to which it is addressed. If
> you are not the intended recipient, you are not authorized to read, retain,
> copy, print, distribute or use this message. If you have received this
> communication in error, please notify the sender and delete all copies of
> this message. Accelerite, a Persistent Systems business does not accept any
> liability for virus infected mails.
>


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 commercial
distribution) with XS7

-- 
Erik

On Tue, Jul 12, 2016 at 6:01 PM, Raja Pullela 
wrote:

> 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 Weber"  wrote:
>
> 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...@shapeblue.com
> > www.shapeblue.com
> > 53 Chandos Place, Covent Garden, London  WC2N 4HSUK
> > @shapeblue
> >
> >
> >
>
>
>
>
>
> DISCLAIMER
> ==
> This e-mail may contain privileged and confidential information which is
> the property of Accelerite, a Persistent Systems business. It is intended
> only for the use of the individual or entity to which it is addressed. If
> you are not the intended recipient, you are not authorized to read, retain,
> copy, print, distribute or use this message. If you have received this
> communication in error, please notify the sender and delete all copies of
> this message. Accelerite, a Persistent Systems business does not accept any
> liability for virus infected mails.
>


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 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 *|* tw @CloudOps_

On Tue, Jul 12, 2016 at 1:07 PM, Wido den Hollander  wrote:


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 :


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

speak

now so we can streamline this next RC.

Thanks,

*Will STEVENS*
Lead Developer

*CloudOps* *| *Cloud Solutions Experts
420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
w cloudops.com *|* tw @CloudOps_

On Tue, Jul 12, 2016 at 10:53 AM, Sean Lair 

wrote:

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 12, 2016 1:48 AM
To: Sean Lair ; dev@cloudstack.apache.org
Subject: RE: [VOTE] Apache Cloudstack 4.9.0 RC1



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 the PFS

variable

when determining whether or not to enable PFS for the site-to-site VPN.

Here is the 1-line of code that is broken.  You can see how it

refers to

dpd to set pfs.

file.addeq(" pfs=%s" % CsHelper.bool_to_yn(obj['dpd']))

This pull request fixes the issue, but was not merged since we were

going to strongSwan.  It would be nice if this bug fix was put into

4.9.0

https://github.com/apache/cloudstack/pull/1480


Would it make you a -1 for you without this PR? If so, please vote -1

:)

Wido


Thanks!
Sean

-Original Message-
From: Will Stevens [mailto:williamstev...@gmail.com]
Sent: Wednesday, July 6, 2016 3:52 PM
To: dev@cloudstack.apache.org
Subject: [VOTE] Apache Cloudstack 4.9.0 RC1

Hi All,

I've created a 4.9.0 release, with the following artifacts up for a

vote:

Git Branch and Commit SH:


https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.9.0-RC20160706T1546

Commit: 643f75aa9150156b1fb05f339a338614fc7ad3fb

I will be updating the Release Notes with the changes in this release

tomorrow.  If the RC changes, I can adapt the release notes after.

Source release (checksums and signatures are available at the same
location):
https://dist.apache.org/repos/dist/dev/cloudstack/4.9.0/

PGP release keys (signed using CB818F64):
https://dist.apache.org/repos/dist/release/cloudstack/KEYS

Vote will be open for 72 hours.

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)

Thanks,

Will