Bug 1388470 - throw warning to show that older tier commands are depricated and will be removed.
Summary: throw warning to show that older tier commands are depricated and will be rem...
Keywords:
Status: CLOSED EOL
Alias: None
Product: GlusterFS
Classification: Community
Component: tiering
Version: 3.9
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
Assignee: hari gowtham
QA Contact: bugs@gluster.org
URL:
Whiteboard:
Depends On: 1388062
Blocks: 1388464
TreeView+ depends on / blocked
 
Reported: 2016-10-25 12:38 UTC by hari gowtham
Modified: 2017-03-08 12:31 UTC (History)
3 users (show)

Fixed In Version:
Clone Of: 1388062
Environment:
Last Closed: 2017-03-08 12:31:30 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

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/


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