Bug 147828

Summary: cman initscript in_cluster function needs rework after latest cman changes
Product: [Retired] Red Hat Cluster Suite Reporter: Derek Anderson <danderso>
Component: cmanAssignee: Christine Caulfield <ccaulfie>
Status: CLOSED CURRENTRELEASE QA Contact: Cluster QE <mspqa-list>
Severity: medium Docs Contact:
Priority: medium    
Version: 4CC: cluster-maint
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-04-24 14:32:19 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 149236    
Bug Blocks:    

Description Derek Anderson 2005-02-11 18:57:29 UTC
Description of problem:
The latest changes in cman regarding node naming have made it
necessary to change how the in_cluster() function works.  Currently it
looks for the node's shortname in /proc/cluster/nodes.  I was testing
private network stuff so had link-10-pvt in the cluster.conf.  They
joined just fine, but I was sitting around waiting 5 minutes for it to
fail because link-10-pvt != link-10.  I can't think of a clean way to
implement this check.  Hope someone can.

Version-Release number of selected component (if applicable):
cman-1.0-0.pre21.0

How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Derek Anderson 2005-02-11 19:03:47 UTC
Makes stopping a beeeyatch too.

Comment 2 Adam "mantis" Manthei 2005-02-11 20:05:51 UTC
Well, I definately can't use uname or shortname now since Dave's
recent changes break cman's behavior, and I'm not about to implelement
any ip resolving hacks to determine hostname.  Thankfully, Patrick
added the wait feauture to the `cman_tool join` command, so we're not
totally hosed.  Another option would be to force cman_tool join to use
a predetermined string (such as "hostname" or "uname -n") instead of
allowing cman_tool to try to guess it's value.  That way the startup
script will know what to look for in /proc/cluster/nodes.

I saw something in one of Patrick's commits about `cman_tool leave` 
now having a wait option too.  I'll try to pull that into the init
script refresh as well.

Comment 3 Adam "mantis" Manthei 2005-02-18 23:34:43 UTC
patrial fix has been checked in.  leaving this still assigned because
I think I need to use the wait option, but am not getting it to work
like I expect at the moment.

Comment 4 Christine Caulfield 2009-04-24 14:32:19 UTC
Closing because the current initscript has been in the packages for ages and no-one has complained ;-)