[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1542 Great. You have done a lot of work already; but, just to make sure things do not get lost. What about opening a Jira ticket to expose that customization through the ACS API? --- If

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1542 @rafaelweingartner That's correct. At this point advanced VM details and advanced template details have no UI exposure. At some point we can bring in PR to show this in UI and allow API based

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1542 Great.I am ok with the code. I give my LGTM here. I have one extra question, though. How do you intend to change that parameter (NESTED_VIRTUALIZATION_FLAG) to tell which VM

[GitHub] cloudstack pull request #1542: CLOUDSTACK-9379: Support nested virtualizatio...

Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1542#discussion_r74667063 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java --- @@ -2184,7 +2184,7 @@ int

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1542 I had removed it, I saw your comment in that file and I included it in last refactor, I forgot to mention it in my last comment :) --- If your project is set up for it, you can reply to this

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1542 There is still one point I would like to highlight. The method VmwareResource.configNestedHVSupport does not need to be static. --- If your project is set up for it, you can reply to

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1542 Sure, done --- 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

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1542 @nvazquez, it seems that there was a problem with the Jenkins Job. Can you force another build? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request #1542: CLOUDSTACK-9379: Support nested virtualizatio...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1542#discussion_r74655041 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java --- @@ -2184,7 +2184,7 @@ int

[ANNOUNCE] Apache CloudStack 4.9.0 Released!

Hello Everyone, I am happy to announce the 4.9.0 release of Apache CloudStack. As the release manager for 4.9.0, I would like to say a big Thank You to the community for supporting this release. We were able to merge over 180 PRs and fixed more than 75 bugs with this version. We had a period

Re: CLOUDSTACK-8682 StrongSwan

Link done in Jira as comment in case we end up with some other PR related the strongswan. On Fri, Aug 12, 2016 at 9:10 AM, Matthew Smart wrote: > Can someone with a little more knowledge or access update Jira for this > ticket so that it links to the existing PR

[GitHub] cloudstack issue #872: Strongswan vpn feature

Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/872 JIRA reference: https://issues.apache.org/jira/browse/CLOUDSTACK-8682 --- 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 #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1542 Thanks @rafaelweingartner, I agree with you, I refactored 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

[GitHub] cloudstack issue #1603: Vrouter fixes pylint

Github user romain-dartigues commented on the issue: https://github.com/apache/cloudstack/pull/1603 Done. Le me know what you think about it (I don't think Jenkins failed because of my edits?). --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1542 I understand the need to create a test. And I agree that it deserves a test. What I meant was the following. IMHO, you could simply remove the final keyword of the variable

[GitHub] cloudstack pull request #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

Github user marcaurele commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1638#discussion_r74588212 --- Diff: pom.xml --- @@ -92,22 +85,22 @@ 1.5.1 1.2.8 2.0.4 -2.5 +3.1.0 1.2 1.2.1

Re: CLOUDSTACK-8682 StrongSwan

Can someone with a little more knowledge or access update Jira for this ticket so that it links to the existing PR in some way? Should I just put a comment in the ticket linking to it? It seems, but correct me if I am wrong, that work on Strongswan has stalled (last comment by original

CLOUDSTACK-7982 - KVM live migration

Need some Jira love... Can someone give me the rights to assign myself to this JIRA issue, or to assign myself directly. Thanks https://issues.apache.org/jira/browse/CLOUDSTACK-7982 There are new configuration items and some explainations to be documented. I thought to write it in the

[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1638 Packaging result: ✔centos6 ✔centos7 ✔debian repo: http://packages.shapeblue.com/cloudstack/pr/1638 Job ID-85 --- If your project is set up for it, you can reply to this email

Re: CLOUDSTACK-8682 StrongSwan

We are also keeping an eye on the remote management VPN. Last time I've tested the StrongSwan PR, it was not working for the remote MGT vpn, site-to-site was working. Look like some config would be missing in strongswan configuration. I'm willing to test/provide configuration needed in

Request for comments: Moving to Spring 4 and Java 8

All, It's about time to migrate master to Java 8 JDK and Spring 4. Please help review the following PRs: https://github.com/apache/cloudstack/pull/1638 There are three outstanding issues that I need help and feedback on: 1. F5 plugin fails to build due to issue likely in the axis

[GitHub] cloudstack pull request #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1638#discussion_r74580238 --- Diff: pom.xml --- @@ -92,22 +85,22 @@ 1.5.1 1.2.8 2.0.4 -2.5 +3.1.0 1.2 1.2.1

[GitHub] cloudstack pull request #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

Github user marcaurele commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1638#discussion_r74575182 --- Diff: pom.xml --- @@ -92,22 +85,22 @@ 1.5.1 1.2.8 2.0.4 -2.5 +3.1.0 1.2 1.2.1

[GitHub] cloudstack pull request #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1638#discussion_r74574871 --- Diff: pom.xml --- @@ -92,22 +85,22 @@ 1.5.1 1.2.8 2.0.4 -2.5 +3.1.0 1.2 1.2.1

[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1638 Packaging result: ✔centos6 ✖centos7 ✖debian repo: http://packages.shapeblue.com/cloudstack/pr/1638 Job ID-83 --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

Github user marcaurele commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1638#discussion_r74574588 --- Diff: pom.xml --- @@ -92,22 +85,22 @@ 1.5.1 1.2.8 2.0.4 -2.5 +3.1.0 1.2 1.2.1

[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1638 @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 pull request #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

Github user rhtyd closed the pull request at: https://github.com/apache/cloudstack/pull/1638 --- 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 #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

GitHub user rhtyd reopened a pull request: https://github.com/apache/cloudstack/pull/1638 CLOUDSTACK-9456: Use Spring 4 and Java 8 This changes makes CloudStack use spring 4 and Java 8: - Bumps jdk version to 1.8 - Use jdk8 for Travis CI - Bump

[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1638 Packaging result: ✖centos6 ✖centos7 ✖debian repo: http://packages.shapeblue.com/cloudstack/pr/1638 Job ID-82 --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1542 @rafaelweingartner I agree it wasn't the best solution but I meant to include it for the sake of testing `configureNestedVirtualization` method. Which approach can I take to test this method,

[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1638 @blueorangutan I see, slaves need Java8 installed. Fixing that. --- 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 #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1638 Packaging result: ✖centos6 ✖centos7 ✖debian repo: http://packages.shapeblue.com/cloudstack/pr/1638 Job ID-81 --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

GitHub user rhtyd opened a pull request: https://github.com/apache/cloudstack/pull/1638 CLOUDSTACK-9456: Use Spring 4 and Java 8 This changes makes CloudStack use spring 4 and Java 8: - Bumps jdk version to 1.8 - Use jdk8 for Travis CI - Bump

[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1638 @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 #1546: [WIP][4.10+/long-term] Moving towards Java8 and Spri...

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1546 I'll split 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 feature enabled and wishes

[GitHub] cloudstack pull request #1637: Command route not available on CentOS 7

GitHub user lmrv opened a pull request: https://github.com/apache/cloudstack/pull/1637 Command route not available on CentOS 7 Hi guys, The command route is not available on a fresh install of CentOS 7. This one should be replace by ip route. Command route make

[GitHub] cloudstack-docs-install pull request #28: corrected -d value while building ...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack-docs-install/pull/28 --- 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: [VOTE] Apache Cloudstack 4.8.1 RC2

Hello, My vote +1 (binding) Tests are passed on a virtual topology of servers (CS over CS) (1mgr+2nodes+1nfs) : 1/ Fresh install of 4.8.1RC2 (adv net) on Ubuntu 14.04.5 + KVM + NFS : OK Some standard tests with success (create vm, migration, HA, create networks, create user, create ssh

[GitHub] cloudstack pull request #1636: Fix a quote issue with Spanish L10N (from tra...

/cloudstack L10N-update-4.8-20160812-fix-spanish-quote Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1636.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit