Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
From: rsafonseca <git@git.apache.org>
To: dev@cloudstack.apache.org
Reply-To: dev@cloudstack.apache.org
References: <git-pr-238-cloudstack@git.apache.org>
In-Reply-To: <git-pr-238-cloudstack@git.apache.org>
Subject: [GitHub] cloudstack pull request: Build and create tomcat,
 fix several othe...
Content-Type: text/plain
Message-Id: <20150529093257.05C1DDFF7A@git1-us-west.apache.org>
Date: Fri, 29 May 2015 09:32:56 +0000 (UTC)

Github user rsafonseca commented on the pull request:

    https://github.com/apache/cloudstack/pull/238#issuecomment-106758023
  
    Indeed.. it's only setup on centos 6 and not 7, sorry for misleading you there :)
    cloudstack-setup-management will link the server.xml in any environment anyway.
    
    The current version on master drops a server.xml in the folder (for Centos7 only), but it doesn't support SSL. I just restored the behaviour to be the same as in other OSs (you get a non-ssl and ssl version and link is handled by the setup scripts).
    
    Configuring SSL by default will remove the need to have a separate server-ssl.xml and server-nonssl.xml and will enable to just drop the server.xml config file in /etc/cloudstack/management .
    
    The behaviour should be consistent across distros
    
    Thanks for taking the time to test it out :)


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---