Bug 980541 - GlusterFS testing framework needs rdma support added
Summary: GlusterFS testing framework needs rdma support added
Keywords:
Status: CLOSED EOL
Alias: None
Product: GlusterFS
Classification: Community
Component: unclassified
Version: pre-release
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-07-02 16:20 UTC by Justin Clift
Modified: 2015-10-22 15:40 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-10-22 15:40:20 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Justin Clift 2013-07-02 16:20:26 UTC
Description of problem:

  At present, the GlusterFS testing framework uses tcp for all tests, as there
  is no "protocol $PROTOCOL" line in any of the volume creation commands.

  We should add a protocol specifier to the volume creation commands in our
  testing framework, which would enable us to test RDMA as well.


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

  Upstream GlusterFS git, master branch as of Tue 2nd July 2013.


Additional info:

  Initial support should be very easy to add without breaking the existing
  testing framework.

  Likely starting by adding the PROTOCOL definition to tests/include.rc:

    PROTOCOL=${PROTOCOL:=tcp};    # transport protocol to use

  Then adding "protocol $PROTOCOL" to the volume creation commands in:

    tests/basic/*.t

  Adding "protocol $PROTOCOL" to other tests could be done at a future point.

Comment 1 Justin Clift 2013-07-02 19:25:36 UTC
s/protocol $PROTOCOL/transport $PROTOCOL/

Comment 2 Anand Avati 2013-07-02 21:50:25 UTC
REVIEW: http://review.gluster.org/5275 (Add transport keyword and variable to our test framework) posted (#1) for review on master by Justin Clift (jclift)

Comment 4 Kaleb KEITHLEY 2015-10-22 15:40:20 UTC
pre-release version is ambiguous and about to be removed as a choice.

If you believe this is still a bug, please change the status back to NEW and choose the appropriate, applicable version for it.


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