Bug 745174 - Bad sys.path.append in fencing agents
Summary: Bad sys.path.append in fencing agents
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Cluster Suite
Classification: Retired
Component: fence
Version: 4
Hardware: All
OS: Unspecified
urgent
urgent
Target Milestone: rc
Assignee: Lon Hohberger
QA Contact: Cluster QE
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-10-11 14:48 UTC by Lon Hohberger
Modified: 2012-03-05 15:29 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-03-05 15:29:29 UTC


Attachments (Terms of Use)
Patch, part 1 (1.17 KB, patch)
2011-10-11 17:45 UTC, Lon Hohberger
no flags Details | Diff
Patch, part 2 (828 bytes, patch)
2011-10-11 17:46 UTC, Lon Hohberger
no flags Details | Diff
Patch, part 3 (1.24 KB, patch)
2011-10-11 18:26 UTC, Lon Hohberger
no flags Details | Diff

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


Note You need to log in before you can comment on or make changes to this bug.