Bug 639252

+++ This bug was initially created as a clone of Bug #637154 +++

Description of problem:


Version-Release number of selected component (if applicable):



How reproducible:

Always

Steps to Reproduce:

1.Build a cluster using current packages versions and create services using at least a resource of type SAPInstance and SAPDatabase.

2. Create directory /u

3. Try to start SAP resources
  
Actual results:

Resources fail to start / stop / status.

Following message is found in logs file for SAPDatabase:

Sep 23 00:08:53 srv00201 clurgmgrd: [12875]: <err> Oracle Listener LISTENER_EPC start failed: su: user ora does not exist
Sep 23 00:08:53 srv00201 clurgmgrd: [12875]: <err> SAP database EPC start failed: This account is currently not available.

Similar messages for SAPInstance resources.

Expected results:

SAP Resources are successfully started.

Additional info:

The problem is related with "tr [:upper:] [:lower:]" commands found in SAPInstance and SAPDatabase resources. These commands fail when a file or directory exist named with one of the characters in "upper" or "lower" as it's not properly interpreted by shell, i.e.:

[amoralej@faemino tmp]$ echo TEST| tr [:upper:] [:lower:]
test
[amoralej@faemino tmp]$ touch u
[amoralej@faemino tmp]$ echo TEST| tr [:upper:] [:lower:]
tr: misaligned [:upper:] and/or [:lower:] construct
[amoralej@faemino tmp]$ rm u
[amoralej@faemino tmp]$ echo TEST| tr [:upper:] [:lower:]
test
[amoralej@faemino tmp]$ touch o
[amoralej@faemino tmp]$ echo TEST| tr [:upper:] [:lower:]
oooo
[amoralej@faemino tmp]$ rm o
[amoralej@faemino tmp]$ echo TEST| tr [:upper:] [:lower:]
test
[amoralej@faemino tmp]$ 

This can be fixed by escaping or using quotes:

[amoralej@faemino tmp]$ echo TEST| tr [:upper:] [:lower:]
tr: misaligned [:upper:] and/or [:lower:] construct
[amoralej@faemino tmp]$ echo TEST| tr [\:upper:\] [\:lower:\]
test
[amoralej@faemino tmp]$ echo TEST| tr '[:upper:]' '[:lower:]'
test
[amoralej@faemino tmp]$ 



I tested it works fine by modifying resource files as follows:




[root@srv00200 cluster]# diff SAPInstance  /etc/cluster/22-sept-2010-backup/SAPInstance.ori-redhat 
571c571 
< sidadm="`echo $SID | tr [\:upper:\] [:lower:]`adm" 
--- 
> sidadm="`echo $SID | tr [:upper:] [:lower:]`adm"  




[root@srv00200 cluster]# diff SAPDatabase  /etc/cluster/22-sept-2010-backup/SAPDatabase.ori-redhat 
191c191 
<   orasid="ora`echo $SID | tr [\:upper:\] [:lower:]`" 
--- 
>   orasid="ora`echo $SID | tr [:upper:] [:lower:]`" 
209c209 
<   orasid="ora`echo $SID | tr [\:upper:\] [:lower:]`" 
--- 
>   orasid="ora`echo $SID | tr [:upper:] [:lower:]`" 
233c233 
<   orasid="ora`echo $SID | tr [\:upper:\] [:lower:]`" 
--- 
>   orasid="ora`echo $SID | tr [:upper:] [:lower:]`" 
492c492 
<   db2sid="db2`echo $SID | tr [\:upper:\] [:lower:]`" 
--- 
>   db2sid="db2`echo $SID | tr [:upper:] [:lower:]`" 
742c742 
<          SUSER="ora`echo $SID | tr [\:upper:\] [:lower:]`" 
--- 
>          SUSER="ora`echo $SID | tr [:upper:] [:lower:]`" 
746c746 
<          SUSER="db2`echo $SID | tr [\:upper:\] [:lower:]`" 
--- 
>          SUSER="db2`echo $SID | tr [:upper:] [:lower:]`" 
982c982 
< sidadm="`echo $SID | tr [\:upper:\] [:lower:]`adm" 
--- 
> sidadm="`echo $SID | tr [:upper:] [:lower:]`adm"

--- Additional comment from amoralej on 2010-09-24 09:57:53 EDT ---

I only escaped :upper: (in my case the file name was /u), however to fix it completely I should have applied it to lower too:

tr [\:upper:\] [\:lower:\]

Best regards,

Alfredo