Bug 1388470

Summary: throw warning to show that older tier commands are depricated and will be removed.
Product: [Community] GlusterFS Reporter: hari gowtham <hgowtham>
Component: tieringAssignee: hari gowtham <hgowtham>
Status: CLOSED EOL QA Contact: bugs <bugs>
Severity: low Docs Contact:
Priority: low    
Version: 3.9CC: amukherj, bugs, dlambrig
Target Milestone: ---Keywords: Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1388062 Environment:
Last Closed: 2017-03-08 12:31:30 UTC Type: Bug
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: 1388062    
Bug Blocks: 1388464    

Description hari gowtham 2016-10-25 12:38:47 UTC
+++ This bug was initially created as a clone of Bug #1388062 +++

Description of problem:
The older tier commands are needed to be removed to aid maintaining the code.
before removing we need to a warning that these commands are depricated, use the new ones.

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


How reproducible:
100%

Steps to Reproduce:
1.create any volume.
2.try attach brick using the old tier command "gluster v attach-tier <volname> <brick>..
3.same thing happens for detach tier related older conmands.

Actual results:
it works

Expected results:
should throw a warning and then continue.

Additional info:
future features might not work for this command.

--- Additional comment from Worker Ant on 2016-10-24 06:58:30 EDT ---

REVIEW: http://review.gluster.org/15713 (CLI/TIER: throw warning regarding the removal of the older commands.) posted (#1) for review on master by hari gowtham (hari.gowtham005)

--- Additional comment from Worker Ant on 2016-10-25 00:23:46 EDT ---

REVIEW: http://review.gluster.org/15713 (CLI/TIER: throw warning regarding the removal of the older commands.) posted (#2) for review on master by hari gowtham (hari.gowtham005)

--- Additional comment from Worker Ant on 2016-10-25 07:55:43 EDT ---

COMMIT: http://review.gluster.org/15713 committed in master by Dan Lambright (dlambrig) 
------
commit 62238f9fb3511dc854651e694bb27b9ebce1e19e
Author: hari <hgowtham>
Date:   Mon Oct 24 16:24:23 2016 +0530

    CLI/TIER: throw warning regarding the removal of the older commands.
    
    The older tier commands for attach tier and detach tier have to be
    removed from code. This patch sends a warning asking to use new
    command as older ones are depricated and will be removed.
    
    Change-Id: Ie1c62947bad6ff106f40331ff6134838a6c72a7a
    BUG: 1388062
    Signed-off-by: hari <hgowtham>
    Reviewed-on: http://review.gluster.org/15713
    Tested-by: hari gowtham <hari.gowtham005>
    Smoke: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Dan Lambright <dlambrig>

Comment 1 Worker Ant 2016-10-25 13:06:43 UTC
REVIEW: http://review.gluster.org/15723 (CLI/TIER: throw warning regarding the removal of the older commands.) posted (#1) for review on release-3.9 by hari gowtham (hari.gowtham005)

Comment 2 Worker Ant 2016-10-26 07:04:34 UTC
COMMIT: http://review.gluster.org/15723 committed in release-3.9 by Atin Mukherjee (amukherj) 
------
commit a55a780349615a0e5dc2d97fec61c143d5cd8256
Author: hari <hgowtham>
Date:   Mon Oct 24 16:24:23 2016 +0530

    CLI/TIER: throw warning regarding the removal of the older commands.
    
    The older tier commands for attach tier and detach tier have to be
    removed from code. This patch sends a warning asking to use new
    command as older ones are depricated and will be removed.
    
    >Change-Id: Ie1c62947bad6ff106f40331ff6134838a6c72a7a
    >BUG: 1388062
    >Signed-off-by: hari <hgowtham>
    >Reviewed-on: http://review.gluster.org/15713
    >Tested-by: hari gowtham <hari.gowtham005>
    >Smoke: Gluster Build System <jenkins.org>
    >NetBSD-regression: NetBSD Build System <jenkins.org>
    >CentOS-regression: Gluster Build System <jenkins.org>
    >Reviewed-by: Dan Lambright <dlambrig>
    
    Change-Id: I28bcb0b0d2f362fbdd28aab9bea95e3a2947caa6
    BUG: 1388470
    Signed-off-by: hari <hgowtham>
    Reviewed-on: http://review.gluster.org/15723
    Tested-by: hari gowtham <hari.gowtham005>
    Smoke: Gluster Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    Reviewed-by: Atin Mukherjee <amukherj>

Comment 3 Kaushal 2017-03-08 12:31:30 UTC
This bug is getting closed because GlusterFS-3.9 has reached its end-of-life [1].

Note: This bug is being closed using a script. No verification has been performed to check if it still exists on newer releases of GlusterFS.
If this bug still exists in newer GlusterFS releases, please open a new bug against the newer release.

[1]: https://www.gluster.org/community/release-schedule/