Bug 901126 (JBEWS-8)

Summary: Update postinstall script to check for check for conf file before editing
Product: [JBoss] JBoss Enterprise Web Server 2 Reporter: Permaine Cheung <pcheung>
Component: httpdAssignee: Weinan Li <weli>
Status: CLOSED CURRENTRELEASE QA Contact: Libor Fuka <lfuka>
Severity: high Docs Contact:
Priority: high    
Version: 2.0.0CC: jclere, jdoyle, lfuka, majoshi, mhasko, mhusnain, mturk, pcheung, pslavice, rhatlapa, rsvoboda, weli
Target Milestone: ---   
Target Release: 2.0.1   
Hardware: Unspecified   
OS: Unspecified   
URL: http://jira.jboss.org/jira/browse/JBEWS-8
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
JBoss Enterprise Web Server 2.0 contained <classname>mod_snmp</classname> and <classname>mod_cluster</classname>. However, JBoss Enterprise Application Platform does not require these dependencies and therefore the commands that manipulate <classname>mod_snmp</classname> and <classname>mod_cluster</classname> in the <filename>.postinstall</filename> file are not suitable for JBoss Enterprise Application Platform. As a result, the <filename>.postinstall</filename> file included with JBoss Enterprise Application Platform 2.0 did not work as expected. This is fixed so that the <filename>.postinstall</filename> file now includes condition tests in <classname>mod_cluster</classname> and <classname>mod_snmp</classname>. As a result, the <filename>.postinstall</filename> file shipped with JBoss Enterprise Application Platform works as expected.
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-01-03 12:58:30 UTC Type: Feature Request
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
add test condition in post install none

Description Permaine Cheung 2012-10-30 14:00:54 UTC
project_key: JBEWS

Currently, when we include the httpd zips from EWS 2 in EAP, we need to remove a couple of lines for the editing of .conf files that we will be removing from the httpd zips for EAP.
If we can add a check before the editing, we do not need to update the postinstall file.
As EWS 2.0.0 is close to GA, if we can get this in for EWS 2.0.1, that would be great.
(You can see which lines in the postinstall are removed if you look at the jb-eap-6-rhel-6 branch of jboss-eap6-httpd-natives in distgit)

Comment 1 Permaine Cheung 2012-10-30 14:01:35 UTC
Link: Added: This issue is related to JBPAPP-9993


Comment 2 Libor Fuka 2012-11-05 12:01:44 UTC
Link: Added: This issue is related to JBPAPP-9551


Comment 3 Libor Fuka 2012-11-05 12:39:24 UTC
Link: Added: This issue relates to JBPAPP-9551


Comment 4 Libor Fuka 2012-11-05 12:39:35 UTC
Link: Removed: This issue is related to JBPAPP-9551 


Comment 5 Weinan Li 2013-04-25 10:46:07 UTC
Created attachment 739811 [details]
add test condition in post install

Comment 6 Weinan Li 2013-04-25 10:56:51 UTC
Hi Mladen, could you please patch the win/sol postinstall file (for zip bundle) as I did in attachment? Thanks!

Comment 7 Mladen Turk 2013-04-25 11:34:31 UTC
We are doing this differently for windows/solaris.
For windows we already have that rewrite to absolute dir path.
For solaris we use /var/cache/ and that cannot be changed.
BTW, you should no do that for RHEL as well because its a security risk if
you have log and cache in userdir.

Comment 9 Jean-frederic Clere 2013-05-31 14:06:03 UTC
The DocText looks wrong...

Comment 10 Weinan Li 2013-06-01 14:18:20 UTC
The current doc is wrong. Here are some suggestions:

Cause:JBoss Enterprise Web Server 2 ships mod_snmp and mod_jk but JBoss Enterprise Application Platform don't have these dependencies. So the commands that manipulate mod_snmp and mod_jk in .postinstall are not suitable for JBoss Enterprise Application Platform.

Consequence: The .postinstall shipped in JBoss Enterprise Application Platform will not work properly. 

Fix: Add condition tests in mod_jk and mod_snmp parts of .postinstall.

Result: The .postinstall shipped in JBoss Enterprise Application Platform will work properly.

Comment 11 Misha H. Ali 2013-06-03 00:34:04 UTC
Thank you, Wei Nan and Jean-Frederic. New proposed text is now in the doc_text field.

Comment 12 Weinan Li 2013-06-03 07:14:51 UTC
Sorry Misha, I've made a mistake on the doc: The components affected should be "mod_cluster and mod_snmp" instead of "mod_jk and mod_snmp".

Comment 13 Michal Haško 2013-06-10 09:50:33 UTC
VERIFIED on:

jboss-ews-httpd-2.0.1-CR1-RHEL5-i386.zip
jboss-ews-httpd-2.0.1-CR1-RHEL5-x86_64.zip
jboss-ews-httpd-2.0.1-CR1-RHEL6-i386.zip
jboss-ews-httpd-2.0.1-CR1-RHEL6-x86_64.zip

Comment 14 Libor Fuka 2013-06-10 11:53:11 UTC
Verified on 2.0.1 CR1 for Solaris and Windows, regarding comm 7