Bug 640995 - Cman doesn't allow user to select infiniband (iba) and udpu (UDP unicast) corosync transport mechanism
Summary: Cman doesn't allow user to select infiniband (iba) and udpu (UDP unicast) cor...
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: cluster
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Fabio Massimo Di Nitto
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 657041
TreeView+ depends on / blocked
 
Reported: 2010-10-07 13:37 UTC by Jan Friesse
Modified: 2010-11-25 06:58 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
: 657041 (view as bug list)
Environment:
Last Closed: 2010-11-25 06:58:35 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)
Proposed patch (7.64 KB, patch)
2010-10-07 13:37 UTC, Jan Friesse 		no flags Details | Diff
Proposed patch - try2 (7.66 KB, patch)
2010-10-08 07:38 UTC, Jan Friesse 		no flags Details | Diff
Show Obsolete (1) View All

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.
Note You need to log in before you can comment on or make changes to this bug.