Bug 1376667 - gfapi: deprecate the rdma support for management connections
Summary: gfapi: deprecate the rdma support for management connections
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Gluster Storage
Classification: Red Hat Storage
Component: libgfapi
Version: rhgs-3.1
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: ---
Assignee: Prasanna Kumar Kalever
QA Contact: Vivek Das
URL:
Whiteboard:
Depends On: 1360647
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-09-16 06:51 UTC by Prasanna Kumar Kalever
Modified: 2018-11-13 12:26 UTC (History)
8 users (show)

Fixed In Version: glusterfs-3.12.2-18.2
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1360647
Environment:
Last Closed: 2018-11-13 12:26:55 UTC
Embargoed:


Attachments (Terms of Use)

Description Prasanna Kumar Kalever 2016-09-16 06:51:31 UTC
+++ This bug was initially created as a clone of Bug #1360647 +++

Description of problem:
Improve glfs_set_volfile_server();

RDMA transport was never supported for management connection before, but glfs_set_volfile_server() allowed it and the gluster code silently fall back to tcp with out a warning, this gave a chance to people assume that mgmt connection supports rdma along with tcp and unix.

We need to deprecate the rdma support by stating to splash warning at the usage.

Also add better defense code on port and transport;


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

COMMIT: http://review.gluster.org/15026 committed in master by Niels de Vos (ndevos) 
------
commit 6e78a0dda9343d75674a12b7e1a1e8e59e17c78c
Author: Prasanna Kumar Kalever <prasanna.kalever>
Date:   Wed Jul 27 14:23:23 2016 +0530

    gfapi: deprecate rdma support for management connections
    
    RDMA transport was never supported for management connection before, but
    glfs_set_volfile_server() allowed rdma transport type due to lack of defense
    code and the glusterfs_mgmt_init() code silently fall back to tcp with out
    any warnings, this gave a chance for assumtions that mgmt connection also
    supports rdma along with tcp and unix.
    
    This patch deprecates the rdma support by warning at its usage.
    
    Change-Id: I0ad99d9851e05ff84ba4b6a3534a984bcc7e2ed7
    BUG: 1360647
    Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever>
    Reviewed-on: http://review.gluster.org/15026
    Tested-by: Prasanna Kumar Kalever <pkalever>
    Smoke: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Jeff Darcy <jdarcy>
    Reviewed-by: Niels de Vos <ndevos>

--- Additional comment from Niels de Vos on 2016-08-02 08:19:55 EDT ---


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