[GitHub] cloudstack issue #1938: CLOUDSTACK-9780: Fixed the default JAVA_HOME value t...

2017-02-10 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1938 So the short answer is that I ran into this issue when I upgraded and since `jenv` did not correctly set the `JAVA_HOME` directory, I ran into this, so I don't have a deep understanding for all

Re: Upgrade fails on Master

2017-02-10 Thread Will Stevens
New PR opened to solve this issue. Jira: https://issues.apache.org/jira/browse/CLOUDSTACK-9780 Github: https://github.com/apache/cloudstack/pull/1938 Please review... *Will STEVENS* Lead Developer On Fri, Feb 3, 2017 at 12:32 PM, Will Stevens

[GitHub] cloudstack issue #1938: CLOUDSTACK-9780: Fixed the default JAVA_HOME value t...

2017-02-10 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1938 @rhtyd, @wido, @nvazquez, @rafaelweingartner please review... This fixes an issue with #1888. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack-www issue #34: Remove link to “planet.apache.org” and update ...

2017-02-10 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack-www/pull/34 Guys (@pdion891, @terbolous) any extra comment here? --- 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 #1938: CLOUDSTACK-9780: Fixed the default JAVA_HOME value t...

2017-02-10 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1938 Ok, got it. Maybe in CentOS logical links get created during the installation of Java. Or maybe during the installation of ACS. The computer I am using is not used to run ACS,

[GitHub] cloudstack issue #1938: CLOUDSTACK-9780: Fixed the default JAVA_HOME value t...

2017-02-10 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1938 Thanks @swill! 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

[GitHub] cloudstack-www issue #25: Updated list with makewholist.py

2017-02-10 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack-www/pull/25 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

[GitHub] cloudstack pull request #1938: CLOUDSTACK-9780: Fixed the default JAVA_HOME ...

2017-02-10 Thread swill
GitHub user swill opened a pull request: https://github.com/apache/cloudstack/pull/1938 CLOUDSTACK-9780: Fixed the default JAVA_HOME value to be Java8 if not set Now that PR-1888 is merged, Java8 is required. Unfortunately, the file pushed to

[GitHub] cloudstack issue #1938: CLOUDSTACK-9780: Fixed the default JAVA_HOME value t...

2017-02-10 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1938 @Swill, I have a question (actually two) :) If “JAVA_HOME” is not set, or if “JAVA_HOME” content does not exist, we check if there is a java installation. I have a

ONE DAY LEFT!!! Submit a CCC Talk for Miami TODAY!

2017-02-10 Thread Will Stevens
Hello Everyone, We are now down to the wire. Tomorrow the talk submissions close for the CloudStack Collaboration Conference in Miami. Go submit your talk... It is going to be a great event with a lot of participation, so it is a great opportunity to network and get exposure for your work.

[GitHub] cloudstack pull request #1935: CLOUDSTACK-9764: Delete domain failure due to...

2017-02-10 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1935#discussion_r100575787 --- Diff: server/src/com/cloud/user/DomainManagerImpl.java --- @@ -273,79 +274,97 @@ public boolean deleteDomain(long domainId, Boolean

[GitHub] cloudstack pull request #1935: CLOUDSTACK-9764: Delete domain failure due to...

2017-02-10 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1935#discussion_r100574038 --- Diff: server/src/com/cloud/user/DomainManagerImpl.java --- @@ -273,79 +274,97 @@ public boolean deleteDomain(long domainId, Boolean

[GitHub] cloudstack pull request #1935: CLOUDSTACK-9764: Delete domain failure due to...

2017-02-10 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1935#discussion_r100575967 --- Diff: server/src/com/cloud/user/DomainManagerImpl.java --- @@ -273,79 +274,97 @@ public boolean deleteDomain(long domainId, Boolean

[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2017-02-10 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1741 Here is another set of tests run to validate all the settings (in addition to the normal tests). ```

[GitHub] cloudstack issue #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

2017-02-10 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1935 Trillian test result (tid-810) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 32603 seconds Marvin logs:

[GitHub] cloudstack issue #1938: CLOUDSTACK-9780: Fixed the default JAVA_HOME value t...

2017-02-10 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1938 @blueorangutan package --- 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

[GitHub] cloudstack issue #1938: CLOUDSTACK-9780: Fixed the default JAVA_HOME value t...

2017-02-10 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1938 @BlueOrangutan package --- 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

[GitHub] cloudstack issue #1938: CLOUDSTACK-9780: Fixed the default JAVA_HOME value t...

2017-02-10 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1938 @borisstoyanov 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

[GitHub] cloudstack issue #1727: CLOUDSTACK-9539: Support changing Service offering f...

2017-02-10 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1727 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1727: CLOUDSTACK-9539: Support changing Service offering f...

2017-02-10 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1727 Sure @serg38 @blueorangutan test centos7 xenserver-65sp1 --- 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

RE: [GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2017-02-10 Thread Daan Hoogland
sorry to keep you guys waiting, I am struggling my way back in and am working on this (and a lot) will keep you posted. @Rajani, what is the ETA that I must meet to get this in 4.10? daan.hoogl...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, Utrecht Utrecht 3531

[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2017-02-10 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1741 @swill I have tested this PR with systemvm template with strongswan installed long back. The s2s vpn worked fine. LGTM from my side. --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request #1937: CLOUDSTACK-9779 : Releasing secondary guest I...

2017-02-10 Thread ustcweizhou
Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1937#discussion_r100509837 --- Diff: server/src/com/cloud/network/NetworkServiceImpl.java --- @@ -852,7 +852,8 @@ public boolean releaseSecondaryIpFromNic(long ipAddressId) {

[GitHub] cloudstack issue #1934: [CLOUDSTACK-9772] Template: perform a HEAD request t...

2017-02-10 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1934 @marcaurele there are some marvin tests in extract and delete template, could you please have a look? --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1937: CLOUDSTACK-9779 : Releasing secondary guest IP fails...

2017-02-10 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1937 files are also changed from 100644 → 100755 which is not necessary --- 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 #1936: [CLOUDSTACK-9773] API: don't break API output with n...

2017-02-10 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1936 This is a really small change, why's Travis build failing? @marcaurele @koushik-das --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

2017-02-10 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1935 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-478 --- 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 #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

2017-02-10 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1935 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

2017-02-10 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1935 @blueorangutan 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

[GitHub] cloudstack pull request #1937: CLOUDSTACK-9779 : Releasing secondary guest I...

2017-02-10 Thread niteshsarda
GitHub user niteshsarda opened a pull request: https://github.com/apache/cloudstack/pull/1937 CLOUDSTACK-9779 : Releasing secondary guest IP fails with error VM nic Ip x.x.x.x is mapped to load balancing rule ISSUE = Releasing secondary guest IP fails with

[GitHub] cloudstack issue #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

2017-02-10 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1935 @blueorangutan package --- 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

[GitHub] cloudstack issue #1935: CLOUDSTACK-9764: Delete domain failure due to Accoun...

2017-02-10 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1935 @borisstoyanov 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

[GitHub] cloudstack issue #1933: CLOUDSTACK-9569: add router.aggregation.command.each...

2017-02-10 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1933 Testing has PASSED There's a way to override the hardcoded timeout of 10 mins now by setting the "router.aggregation.command.each.timeout" in the

Re: [GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2017-02-10 Thread Rajani Karuturi
I planning to create first RC end of feb(in 20 days). ~ Rajani http://cloudplatform.accelerite.com/ On February 10, 2017 at 2:24 PM, Daan Hoogland (daan.hoogl...@shapeblue.com) wrote: sorry to keep you guys waiting, I am struggling my way back in and am working on this (and a lot) will keep

[GitHub] cloudstack pull request #1937: CLOUDSTACK-9779 : Releasing secondary guest I...

2017-02-10 Thread ustcweizhou
Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1937#discussion_r100511377 --- Diff: server/src/com/cloud/network/NetworkServiceImpl.java --- @@ -852,7 +852,8 @@ public boolean releaseSecondaryIpFromNic(long ipAddressId) {

[GitHub] cloudstack pull request #1908: CLOUDSTACK-9317: Fixed disable static nat on ...

2017-02-10 Thread ProjectMoon
Github user ProjectMoon commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1908#discussion_r100535848 --- Diff: setup/db/db/schema-4920to41000.sql --- @@ -62,3 +62,4 @@ INSERT INTO `cloud`.`role_permissions` (`uuid`, `role_id`, `rule`, `permission`,

[GitHub] cloudstack pull request #1913: CLOUDSTACK-9752: [Vmware] Optimization of vol...

2017-02-10 Thread nvazquez
Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1913#discussion_r100543231 --- Diff: plugins/hypervisors/vmware/src/com/cloud/storage/resource/VmwareStorageProcessor.java --- @@ -1577,11 +1577,15 @@ public Answer

[GitHub] cloudstack issue #1727: CLOUDSTACK-9539: Support changing Service offering f...

2017-02-10 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1727 Trillian test result (tid-809) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 31586 seconds Marvin logs:

[GitHub] cloudstack issue #1727: CLOUDSTACK-9539: Support changing Service offering f...

2017-02-10 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1727 @borisstoyanov @DaanHoogland @rhtyd KVM tests passed but key smoke test is skipped test_change_service_offering_for_vm_with_snapshots Skipped To be on the safe side can we test it on