Bug 745174

Summary: Bad sys.path.append in fencing agents
Product: [Retired] Red Hat Cluster Suite Reporter: Lon Hohberger <lhh>
Component: fenceAssignee: Lon Hohberger <lhh>
Status: CLOSED WONTFIX QA Contact: Cluster QE <mspqa-list>
Severity: urgent Docs Contact:
Priority: urgent    
Version: 4CC: cfeist, cluster-maint, edamato, mbrodeur, mschick
Target Milestone: rcKeywords: Regression
Target Release: ---   
Hardware: All   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-03-05 15:29:29 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
Patch, part 1
none
Patch, part 2
none
Patch, part 3 none

Description Lon Hohberger 2011-10-11 14:48:53 UTC
Description of problem:

On RHEL 4.9, the new RSA agent and the sanbox2 agent have:

  sys.path.append("@FENCEAGENTSLIBDIR@")

... in the shipped release.  This should be a pointer to /usr/lib/fence, as is consistent with other fencing agents.  This prevents use of RSA and sanbox2 fencing agents, and no workaround exists.

Additionally, the fence_rsa agent has an extraneous parameter to 'fence_action' on RHEL-4:

  fence_action(conn, options, set_power_status, get_power_status, None)

Other agents (ilo, wti) have:

  fence_action(conn, options, set_power_status, get_power_status)

Version-Release number of selected component (if applicable): fence-1.32.68-5.el4


How reproducible: 100%
  
Actual results: Fencing agents fail to run.

Comment 2 Lon Hohberger 2011-10-11 14:53:25 UTC
(In reply to comment #0)
> no workaround exists.

Actually, users can edit the scripts to work around this for now.

Comment 3 Lon Hohberger 2011-10-11 15:36:39 UTC
Upon further investigation, fence-1.32.67 does not have this problem with the fence agents library path.

Comment 6 Lon Hohberger 2011-10-11 17:45:50 UTC
Created attachment 527508 [details]
Patch, part 1

Comment 7 Lon Hohberger 2011-10-11 17:46:22 UTC
Created attachment 527509 [details]
Patch, part 2

Comment 9 Lon Hohberger 2011-10-11 18:26:24 UTC
Created attachment 527515 [details]
Patch, part 3