----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24936/#review51250 -----------------------------------------------------------
client/tomcatconf/server-nonssl.xml.in <https://reviews.apache.org/r/24936/#comment89351> Aren't we loosing the jmx support on tomcat6 by commenting this? client/tomcatconf/server-nonssl.xml.in <https://reviews.apache.org/r/24936/#comment89350> Was there any performance impact on tomcat6 due to this change? Dont we need separate files for tomcat6 and 7? - Rajani Karuturi On Aug. 21, 2014, 4:15 p.m., Damodar Reddy Talakanti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24936/ > ----------------------------------------------------------- > > (Updated Aug. 21, 2014, 4:15 p.m.) > > > Review request for cloudstack, Rayees Namathponnan and Hugo Trippaers. > > > Bugs: https://issues.apache.org/jira/browse/CLOUDSTACK-7106 > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/CLOUDSTACK-7106 > > > Repository: cloudstack-git > > > Description > ------- > > RPM Build was failing on RHEL7 due to the packages changed in RHEL7. > Refactored the cloud.spec to make it specific to OS builds. > > > Diffs > ----- > > client/tomcatconf/server-nonssl.xml.in 847197c > client/tomcatconf/server-ssl.xml.in 37bc53d > packaging/centos63/cloud-management.rc 2f04793 > packaging/centos63/cloud.spec 7565f95 > packaging/centos63/default/macros.spec PRE-CREATION > packaging/centos63/default/tomcat.sh PRE-CREATION > packaging/centos63/package.sh 07f95fc > packaging/centos63/rhel7/cloud-management.service PRE-CREATION > packaging/centos63/rhel7/macros.spec PRE-CREATION > packaging/centos63/rhel7/tomcat.sh PRE-CREATION > python/lib/cloudutils/serviceConfigServer.py 2c19d7e > > Diff: https://reviews.apache.org/r/24936/diff/ > > > Testing > ------- > > Tested on RHEL7 and Centos 6.3 > > > Thanks, > > Damodar Reddy Talakanti > >
