[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 Packaging result: ✔centos6 ✔centos7 ✖debian. JID-62 --- 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 #1709: CLOUDSTACK-7982 - KVM live migration

2016-10-17 Thread marcaurele
Github user marcaurele commented on the issue: https://github.com/apache/cloudstack/pull/1709 Is there a way to get the management server log of the travis job? --- 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

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-17 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 Thanks @syed for your PR. I'll kick some tests following merge of PR #1708. @PaulAngus can add support for XenServer 7 in Trillian, thanks. @blueorangutan package --- If your project

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-10-17 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1659 LGTM. @murali-reddy can you squash the commits to a single commit and rebase against latest 4.9 branch. Following merge of PR #1703 I'll kick a new trillian test run and merge based on the test

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-10-17 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1692 @karuturi can you help run BVT tests on this PR? --- 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

[GitHub] cloudstack issue #1703: Trillian testrun on PR-1692 fwd-merged on 4.9 branch

2016-10-17 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1703 @karuturi can you help run BVT tests on this PR? --- 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

[GitHub] cloudstack issue #1708: Fwd merge 4.8->4.9 marvintestfixes to master

2016-10-17 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1708 @karuturi can you help run BVT tests on this PR? --- 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: XenServer 7

2016-10-17 Thread Syed Ahmed
@Raj I'd be more interested in knowing if there are any corner cases that We've missed. The fix seems to be relatively simple but I wonder if there is something that I've missed as I cannot tests most of the scenarios On Mon, Oct 17, 2016 at 4:00 PM, Syed Ahmed wrote: > PR

Re: XenServer 7

2016-10-17 Thread Syed Ahmed
PR Created at https://github.com/apache/cloudstack/pull/1711 On Mon, Oct 17, 2016 at 3:20 PM, Syed Ahmed wrote: > Agree with Paul. If you can point to your public repo we can collaborate > easily. > > As per testing. I have manually verified the following use cases below.

[GitHub] cloudstack pull request #1711: Xenserver7

2016-10-17 Thread syed
GitHub user syed opened a pull request: https://github.com/apache/cloudstack/pull/1711 Xenserver7 This PR adds support for XenServer 7. I have manually done the following tests * Create a new cluster with XenServer7 * Add Primary storage: Should create an SR on XS7

Re: XenServer 7

2016-10-17 Thread Syed Ahmed
Agree with Paul. If you can point to your public repo we can collaborate easily. As per testing. I have manually verified the following use cases below. I am very happy with the results so far. I will open a PR with this and we can run regression tests. I also have qcow2 images of XenServer7 and

RE: XenServer 7

2016-10-17 Thread Paul Angus
Hey Rajesh, Could you submit a pull request to Syed's branch containing where you've got to (or let us know where your work is publicly accessible) so that we can combine work..? Kind regards, Paul Angus paul.an...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, London

Re: XenServer 7

2016-10-17 Thread Rajesh Ramchandani
Syed - we are working with Citrix on XS7 as well, please drop a note if you need help. Raj On Mon, Oct 17, 2016 at 8:30 AM -0700, "Syed Ahmed" > wrote: Hi Guys, I have some free time on my hands and was trying to add XenServer 7 support.

Re: XenServer 7

2016-10-17 Thread Syed Ahmed
Thanks Paul, I am still trying to see what changes are needed. I am seeing problems when adding an new host, sometimes the NFS doesn't get mounted. I will try to sort these things today. I would need most of the help in testing and making sure that we don't have any regressions. It would be

RE: XenServer 7

2016-10-17 Thread Paul Angus
Awesome Syed, What can we do to help? @rohit can we point blueorangutan at any repo to build packages for testing? Kind regards, Paul Angus paul.an...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, London WC2N 4HSUK @shapeblue -Original Message- From: Syed

RE: XenServer 7

2016-10-17 Thread Marty Godsey
That is amazing. I cannot wait to see this hit mainstream. The performance difference in XenServer 7 is considerably different than 6.5. I have my "core services" cluster running 7 and it works nicely. Core services like DNS, AD, and even ACS management servers but not compute nodes. Regards,

Re: XenServer 7

2016-10-17 Thread Haijiao
So cool, thats the feature lots of people awaiting and makes 4.10 shinning Sent from my mobile On 2016-10-17 23:28 , Syed Ahmed Wrote: Hi Guys, I have some free time on my hands and was trying to add XenServer 7 support. There seems to be only one line change that was required to add it. I

XenServer 7

2016-10-17 Thread Syed Ahmed
Hi Guys, I have some free time on my hands and was trying to add XenServer 7 support. There seems to be only one line change that was required to add it. I am able to create VMs and do basic stuff like NAT/port forwarding. If anyone is interested in trying this out, I have a branch [1] which I

[GitHub] cloudstack pull request #1559: CLOUDSTACK-9280: System VM volumes can be exp...

2016-10-17 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1559#discussion_r83595364 --- Diff: engine/storage/src/org/apache/cloudstack/storage/endpoint/DefaultEndPointSelector.java --- @@ -345,15 +354,34 @@ public EndPoint

[GitHub] cloudstack pull request #1559: CLOUDSTACK-9280: System VM volumes can be exp...

2016-10-17 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1559#discussion_r83628876 --- Diff: engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeDataFactoryImpl.java --- @@ -77,22 +80,51 @@ public VolumeInfo

[GitHub] cloudstack pull request #1559: CLOUDSTACK-9280: System VM volumes can be exp...

2016-10-17 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1559#discussion_r83647727 --- Diff: engine/storage/test/org/apache/cloudstack/storage/endpoint/DefaultEndPointSelectorTest.java --- @@ -0,0 +1,167 @@ +// Licensed to

[GitHub] cloudstack pull request #1559: CLOUDSTACK-9280: System VM volumes can be exp...

2016-10-17 Thread GabrielBrascher
Github user GabrielBrascher commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1559#discussion_r83600411 --- Diff: engine/storage/test/org/apache/cloudstack/storage/endpoint/DefaultEndPointSelectorTest.java --- @@ -0,0 +1,167 @@ +// Licensed to

[GitHub] cloudstack issue #872: Strongswan vpn feature

2016-10-17 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/872 @swill My setup is up with the strongswan template. Configured s2s VPN connection, I could observe that once that 'ipsec restart' or reload/rereadsecrets brought up the tunnels. I

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

2016-10-17 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1681 @murali-reddy could you please squash your commits? --- 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

Re: ACS 4.9 + VMware: Unable to remove one of the NICs of a multi-nic VM

2016-10-17 Thread Prashanth Manthena
Hi Paul, First of all, thank you for your time and investigation. I have tried with timeouts up to 180 seconds, and that didn't help either :) Yes, it doesn't explain the difference between 4.7 and 4.9, because I am using the same vcenter/vmware setup (i.e. no vmware tools installed). Let me

[GitHub] cloudstack issue #1703: Trillian testrun on PR-1692 fwd-merged on 4.9 branch

2016-10-17 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1703 @karuturi @swill @wido @NuxRo @nvazquez we believe this PR is ready to merge. However, we need to cross check the test results. Would one (or more of you) be able to test this PR in order to

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-10-17 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1692 @karuturi @swill @wido @NuxRo @nvazquez we believe this PR is ready to merge. However, we need to cross check the test results. Would one (or more of you) be able to test this PR in order to

[GitHub] cloudstack issue #873: CLOUDSTACK-8896: allocated percentage of storage pool...

2016-10-17 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/873 @karuturi can you please rebase this against latest master, let's get this reviewed and merged. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request #1710: CLOUDSTACK-9538: FIX failure in Deleting Snap...

2016-10-17 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1710 CLOUDSTACK-9538: FIX failure in Deleting Snapshot From Primary Storage RBD Storage if vm has been removed You can merge this pull request into a Git repository by running: $ git

[GitHub] cloudstack pull request #1709: CLOUDSTACK-7982 - KVM live migration

2016-10-17 Thread marcaurele
GitHub user marcaurele opened a pull request: https://github.com/apache/cloudstack/pull/1709 CLOUDSTACK-7982 - KVM live migration This is the forward port of our KVM live migration implementation in 4.4.2 to the master branch. Feedback is welcome. I still have to write the