Bug 640995

Summary: Cman doesn't allow user to select infiniband (iba) and udpu (UDP unicast) corosync transport mechanism
Product: [Fedora] Fedora Reporter: Jan Friesse <jfriesse>
Component: clusterAssignee: Fabio Massimo Di Nitto <fdinitto>
Status: CLOSED UPSTREAM QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: agk, cfeist, fdinitto, jfriesse, lhh, sdake, swhiteho
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 657041 (view as bug list) Environment:
Last Closed: 2010-11-25 06:58:35 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:    
Bug Blocks: 657041    
Attachments:
Description Flags
Proposed patch
none
Proposed patch - try2 none

Description Jan Friesse 2010-10-07 13:37:25 UTC
Created attachment 452105 [details]
Proposed patch

Description of problem:
Corosync has support for infiniband and udpu (future version). Cman doesn't. Add  support for that.

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

How reproducible:
100%

Steps to Reproduce:
1. Try to configure cluster.conf with udpu
  
Actual results:
Isn't possible

Expected results:
Be possible

Additional info:
Depends on corosync 2.x

Comment 1 Lon Hohberger 2010-10-07 17:26:24 UTC
+		[TX_MECH_UDP] = "udp",
+		[TX_MECH_UDPB] = "udp",
+		[TX_MECH_UDPU] = "udpu",
+		[TX_MECH_RDMA] = "iba",


Shouldn't the TX_MECH_UDPB = "udbp" ?

Comment 2 Steven Dake 2010-10-07 17:31:52 UTC
Not sure what udbp is, maybe an extra cut and paste.

Regards
-steve

Comment 3 Lon Hohberger 2010-10-07 18:03:18 UTC
UDP broadcast, according to the patch.

Comment 4 Lon Hohberger 2010-10-07 18:04:58 UTC
i.e.

udp = udp multicast
udpb = broadcast
udpu = udp unicast
iba = infiniband

Comment 5 Steven Dake 2010-10-07 18:09:52 UTC
In that case, udp is the correct transport.  The broadcast support is implemented in the udp multicast transport driver.

Comment 6 Steven Dake 2010-10-07 18:12:42 UTC
+     <attribute name="transport" rha:description="Specifies transport mechanism to use. Available values are udp (multicast default), udpb (broadcast), rdma (Infiniband).  corosync.conf(5)" rha:sample="">

should list udpu as well.

Regards
-steve

Comment 7 Jan Friesse 2010-10-08 07:38:37 UTC
Created attachment 452288 [details]
Proposed patch - try2

Same as previous patch but fixes relax NG description.

Comment 8 Jan Friesse 2010-11-10 14:39:15 UTC
Patch committed to git branch STABLE31 as c399f4c0f0d7cc4467a68c41715c404b2afb9425

Comment 9 Fabio Massimo Di Nitto 2010-11-25 06:58:35 UTC
This is done and tested. Pending corosync 1.3.0 for release and distribution.