[GitHub] cloudstack issue #1632: Honor the forced parameter for stopVirtualMachine ap...

2016-08-11 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1632 @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 wishes

[GitHub] cloudstack issue #1632: Honor the forced parameter for stopVirtualMachine ap...

2016-08-11 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1632 @karuturi @jburwell test keyword has been disabled due to infrastructural issues, I'll try to get this fixed today. Also, we need to run package first, to evaluate build/test/package checks before

[GitHub] cloudstack issue #1632: Honor the forced parameter for stopVirtualMachine ap...

2016-08-11 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1632 @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 appe

[GitHub] cloudstack issue #1632: Honor the forced parameter for stopVirtualMachine ap...

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

[GitHub] cloudstack issue #1632: Honor the forced parameter for stopVirtualMachine ap...

2016-08-11 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1632 @nathanejohnson LGTM, but this bug affects 4.9 as well; please close this PR and reopen against 4.9. Your PR when accepted will be merged on 4.9 and fwd-merged on master. Thanks. --- If your pro

[GitHub] cloudstack issue #1632: Honor the forced parameter for stopVirtualMachine ap...

2016-08-11 Thread ProjectMoon
Github user ProjectMoon commented on the issue: https://github.com/apache/cloudstack/pull/1632 It goes all the way back to 4.7. Isn't the oldest supported release currently 4.8 at the moment? If so, it should be opened against 4.8, right? --- If your project is set up for it, you can

[GitHub] cloudstack issue #1632: Honor the forced parameter for stopVirtualMachine ap...

2016-08-11 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1632 @ProjectMoon With our current branch support policy, we should merge bugfixes/changes on master and the last release branch (i.e. 4.9 and also since 4.9 is current/to-be lts branch). I'm fine with

[GitHub] cloudstack issue #1632: Honor the forced parameter for stopVirtualMachine ap...

2016-08-11 Thread ProjectMoon
Github user ProjectMoon commented on the issue: https://github.com/apache/cloudstack/pull/1632 I thought it was: new features -> master, bug fixes -> oldest supported release branch? 4.7 is no longer supported but as far as I know 4.8 is still supported (I could be wrong on this). -

[GitHub] cloudstack issue #1632: Honor the forced parameter for stopVirtualMachine ap...

2016-08-11 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1632 @ProjectMoon Thats correct. bugfixes should go to 4.8, the oldest supported branch. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

VPN User/Pass printed in plain text during boot of VRouter

2016-08-11 Thread Matthew Smart
Sorry. My subject line was incorrect. Hey guys, I am new to Cloudstack and Apache (from a dev perspective) so forgive me if this is not the proper place for this... and let me know where I should be reporting issues like this. I am getting Remote Access VPN configured in my test environment

[GitHub] cloudstack pull request #1632: Honor the forced parameter for stopVirtualMac...

2016-08-11 Thread nathanejohnson
Github user nathanejohnson closed the pull request at: https://github.com/apache/cloudstack/pull/1632 --- 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 fe

[GitHub] cloudstack issue #1632: Honor the forced parameter for stopVirtualMachine ap...

2016-08-11 Thread nathanejohnson
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1632 @karaturi @ProjectMoon @rthyd I will close and reopen against 4.8, thanks --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] cloudstack pull request #1635: CLOUDSTACK-9451

2016-08-11 Thread nathanejohnson
GitHub user nathanejohnson opened a pull request: https://github.com/apache/cloudstack/pull/1635 CLOUDSTACK-9451 https://issues.apache.org/jira/browse/CLOUDSTACK-9451 Re-doing against 4.8 since this is a bug, not a feature. You can merge this pull request into a Git reposit

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

2016-08-11 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1542 @rafaelweingartner thanks a lot for your help as always! I refactored based on your comments. I really liked your review, especially your refactor on `shouldEnableNestedVirtualization`,

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-08-11 Thread kiwiflyer
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1635 References: https://github.com/apache/cloudstack/pull/1632 --- 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

Re: VPN User/Pass printed in plain text during boot of VRouter

2016-08-11 Thread John Burwell
Matthew, Thank you for the report. I have forwarded this report to security@ for further investigation. Thanks, -John > john.burw...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, London VA WC2N 4HSUK @shapeblue On Aug 11, 2016, at 8:14 AM, Matthew Smart wrote: > >

CLOUDSTACK-8682 StrongSwan

2016-08-11 Thread Matthew Smart
Hey guys, I have been banging my head against VPN for awhile now on 4.8.0. I am testing on OSX which according to bug Cloudstack-7821 is a won't fix until the Virtual Router is altered to use StronSwan instead of OpenSwan. I am looking to make the auth system for VPN more sophisticated but be

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

2016-08-11 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1542 @nvazquez the code is great now. I have just a concern about that hack to change the field declared as final. In my opinion, the final there does not bring benefits. So, I

Re: CLOUDSTACK-8682 StrongSwan

2016-08-11 Thread Simon Weller
Matthew, There is an open PR for Strongswan here: https://github.com/apache/cloudstack/pull/872 [https://avatars3.githubusercontent.com/u/3789214?v=3&s=400] Strongswan vpn feature by jayapalu · Pull Request #872 · apache/cloudstack

[GitHub] cloudstack pull request #1593: CLOUDSTACK-9417: Usage module refactoring

2016-08-11 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1593#discussion_r74468365 --- Diff: usage/src/com/cloud/usage/UsageSanityChecker.java --- @@ -72,7 +72,9 @@ protected boolean checkItemCountByPstmt(CheckCase checkCase)

[GitHub] cloudstack-docs issue #18: Removed support for Citrix XenServer

2016-08-11 Thread FFincannon
Github user FFincannon commented on the issue: https://github.com/apache/cloudstack-docs/pull/18 Yes, Rafael, I did mean support of the Nuage plugin for XenServer. In answer to Erik Weber's question, "How should people with xenserver + nuage that upgrades handle it?" the pr

[GitHub] cloudstack pull request #1599: Marvin: Fix codegenerator to work with API di...

2016-08-11 Thread rhtyd
Github user rhtyd closed the pull request at: https://github.com/apache/cloudstack/pull/1599 --- 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

Re: CLOUDSTACK-8682 StrongSwan

2016-08-11 Thread Matthew Smart
Thanks. I figured I was looking in the wrong place. For my future reference did you just find this by searching strongswan in the git repo? Is there any direct link between a Jira ticket and a git PR? Matthew Smart President Smart Software Solutions Inc. 108 S Pierre St. Pierre, SD 57501 Phone

[GitHub] cloudstack pull request #1599: Marvin: Fix codegenerator to work with API di...

2016-08-11 Thread rhtyd
GitHub user rhtyd reopened a pull request: https://github.com/apache/cloudstack/pull/1599 Marvin: Fix codegenerator to work with API discovery This fixes Marvin cloudstackAPI generator to work with a live running mgmt server's api discovery. You can merge this pull request into a G

Re: CLOUDSTACK-8682 StrongSwan

2016-08-11 Thread Simon Weller
Matthew, If the PR has the jira issue in the PR subject, any updates to the PR will be added to the Jira issue. See this one for an example: From: Matthew Smart Sent: Thursday, August 11, 2016 1:40 PM To: dev@cloudstack.apache.org Subject: Re: CLOUDSTACK-868

Re: CLOUDSTACK-8682 StrongSwan

2016-08-11 Thread Simon Weller
Oops, hit the send button too soon. https://github.com/apache/cloudstack/pull/1635 [https://avatars0.githubusercontent.com/u/1360240?v=3&s=400] CLOUDSTACK-9451 by nathanejohnson · Pull Request #1635 · apache/cloudstack

Re: CLOUDSTACK-8682 StrongSwan

2016-08-11 Thread Matthew Smart
Thanks. Been years since I touched Jira/Confluence. Matthew Smart President Smart Software Solutions Inc. 108 S Pierre St. Pierre, SD 57501 Phone: (605) 280-0383 Skype: msmart13 Email: msm...@smartsoftwareinc.com On 08/11/2016 01:50 PM, Simon Weller wrote: Matthew, If the PR has the jira iss

[GitHub] cloudstack-docs-install pull request #29: corrected the list of rpms

2016-08-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack-docs-install/pull/29 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] cloudstack-docs-install pull request #30: update for 4.9.0

2016-08-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack-docs-install/pull/30 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] cloudstack-docs-admin issue #36: OSPF: WIP formatting documentation for ospf

2016-08-11 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack-docs-admin/pull/36 @agneya2001 , any update on the merge of your code in the upstream ? I'm keeping this PR open until the actual code get merge into master Regards, --- If your project is s

[GitHub] cloudstack-www pull request #28: website update for 4.9 and page template ch...

2016-08-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack-www/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 the feature

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

2016-08-11 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/1636 Fix a quote issue with Spanish L10N (from transifex translation) You can merge this pull request into a Git repository by running: $ git pull https://github.com/milamberspace/clouds

Re: [VOTE] Apache Cloudstack 4.8.1 RC2

2016-08-11 Thread Milamber
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 key,