Bug 1261744

Summary:	Tier/shd: Tracker bug for tier and shd compatibility
Product:	[Community] GlusterFS	Reporter:	Mohammed Rafi KC <rkavunga>
Component:	tiering	Assignee:	Mohammed Rafi KC <rkavunga>
Status:	CLOSED CURRENTRELEASE	QA Contact:	bugs <bugs>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	3.7.5	CC:	bugs, dlambrig, vagarwal
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.7.6	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1261276
Clones:	1271733 (view as bug list)		Environment:
Last Closed:	2015-11-17 05:58:48 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:	1261276
Bug Blocks:	1260923, 1271733, 1275914

Description Mohammed Rafi KC 2015-09-10 06:06:54 UTC

+++ This bug was initially created as a clone of Bug #1261276 +++

Description of problem:

Tier and shd are not compatible, the problems include

* afr requires unified name across the grapsh switch
* shd is not starting on a tiered volume.
* volfiles are created in a wrong way
* multiple afr/disperse stacks are possible in a single graph
* heal commands are not compatible with tiering
* self-heal-daemon enable commands need to be supported 


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


How reproducible:

100

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

--- Additional comment from Vijay Bellur on 2015-09-09 02:07:19 EDT ---

REVIEW: http://review.gluster.org/11962 (tier/shd: create shd volfile for tiering) posted (#2) for review on master by mohammed rafi  kc (rkavunga)

--- Additional comment from Vijay Bellur on 2015-09-09 02:07:21 EDT ---

REVIEW: http://review.gluster.org/12134 (tiering/volgen: keep afr/ec xlators name constant) posted (#1) for review on master by mohammed rafi  kc (rkavunga)

--- Additional comment from Vijay Bellur on 2015-09-09 02:07:28 EDT ---

REVIEW: http://review.gluster.org/12135 (tier/shd: make shd commands compatible with tiering) posted (#1) for review on master by mohammed rafi  kc (rkavunga)

--- Additional comment from Vijay Bellur on 2015-09-09 03:01:32 EDT ---

REVIEW: http://review.gluster.org/12134 (tiering/volgen: keep afr/ec xlators name constant) posted (#2) for review on master by mohammed rafi  kc (rkavunga)

--- Additional comment from Vijay Bellur on 2015-09-09 03:05:13 EDT ---

REVIEW: http://review.gluster.org/12135 (tier/shd: make shd commands compatible with tiering) posted (#3) for review on master by mohammed rafi  kc (rkavunga)

--- Additional comment from Vijay Bellur on 2015-09-09 04:43:15 EDT ---

REVIEW: http://review.gluster.org/12134 (tiering/volgen: keep afr/ec xlators name constant) posted (#3) for review on master by mohammed rafi  kc (rkavunga)

--- Additional comment from Vijay Bellur on 2015-09-09 04:43:17 EDT ---

REVIEW: http://review.gluster.org/12135 (tier/shd: make shd commands compatible with tiering) posted (#4) for review on master by mohammed rafi  kc (rkavunga)

Comment 1 Vijay Bellur 2015-10-12 06:31:53 UTC

REVIEW: http://review.gluster.org/12333 (tier/shd: create shd volfile for tiering) posted (#1) for review on release-3.7 by mohammed rafi  kc (rkavunga)

Comment 2 Vijay Bellur 2015-10-12 18:24:29 UTC

COMMIT: http://review.gluster.org/12333 committed in release-3.7 by Dan Lambright (dlambrig) 
------
commit 0f5c3e88f145005f963172e3ada10dbf6d6a607c
Author: Mohammed Rafi KC <rkavunga>
Date:   Thu Aug 20 12:19:51 2015 +0530

    tier/shd: create shd volfile for tiering
    
    Currently shd graph will only start if it is replicate
    or disperse volume. But in case of tiering, volume type
    will be tier. So we need to start shd if any of the cold
    or hot is compatible with shd volume.
    
    Back port of>
    >Change-Id: Ic689746ac7d2fc6a9eccdabd8518dc9139829de2
    >BUG: 1261276
    >Signed-off-by: Mohammed Rafi KC <rkavunga>
    >Reviewed-on: http://review.gluster.org/11962
    >Tested-by: Gluster Build System <jenkins.com>
    >Tested-by: NetBSD Build System <jenkins.org>
    >Reviewed-by: Pranith Kumar Karampuri <pkarampu>
    
    (cherry picked from commit fdff192b918ca9cd237f3f784c627102377e3661)
    
    Change-Id: I236a31e7dcefb3dad64881e0b007144bd826b840
    BUG: 1261744
    Signed-off-by: Mohammed Rafi KC <rkavunga>
    Reviewed-on: http://review.gluster.org/12333
    Tested-by: Gluster Build System <jenkins.com>
    Tested-by: NetBSD Build System <jenkins.org>
    Reviewed-by: Dan Lambright <dlambrig>
    Tested-by: Dan Lambright <dlambrig>

Comment 3 Vijay Bellur 2015-10-13 06:11:33 UTC

REVIEW: http://review.gluster.org/12344 (tier/shd: make shd commands compatible with tiering) posted (#1) for review on release-3.7 by mohammed rafi  kc (rkavunga)

Comment 4 Mohammed Rafi KC 2015-10-13 11:45:02 UTC

moving back to post since one more patches need to be merged.

Comment 5 Vijay Bellur 2015-10-14 02:47:43 UTC

COMMIT: http://review.gluster.org/12344 committed in release-3.7 by Dan Lambright (dlambrig) 
------
commit 613f5c0120068a117f7337eeda293512bf106ef8
Author: Mohammed Rafi KC <rkavunga>
Date:   Tue Sep 8 13:04:45 2015 +0530

    tier/shd: make shd commands compatible with tiering
    
    tiering volfiles may contain afr and disperse together
    or multiple time based on configuration. And the
    informations for those configurations are stored in
    tier_info. So most of the volgen code generation
    need to be changed to make compatible with it.
    
    Back port of>
    >Change-Id: I563d1ca6f281f59090ebd470b7fda1cc4b1b7e1d
    >BUG: 1261276
    >Signed-off-by: Mohammed Rafi KC <rkavunga>
    >Reviewed-on: http://review.gluster.org/12135
    >Tested-by: NetBSD Build System <jenkins.org>
    >Tested-by: Gluster Build System <jenkins.com>
    >Reviewed-by: Pranith Kumar Karampuri <pkarampu>
    
    (cherry picked from commit 0ef62933649392051e73fe01c028e41baddec489)
    
    BUG: 1261744
    Change-Id: Iff1b27ae8ce61f1f38fbbd6c92894b3d3516e4d4
    Signed-off-by: Mohammed Rafi KC <rkavunga>
    Reviewed-on: http://review.gluster.org/12344
    Tested-by: NetBSD Build System <jenkins.org>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Joseph Fernandes
    Reviewed-by: Dan Lambright <dlambrig>
    Tested-by: Dan Lambright <dlambrig>

Comment 6 Raghavendra Talur 2015-11-17 05:58:48 UTC

This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.7.6, please open a new bug report.

glusterfs-3.7.6 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://www.gluster.org/pipermail/gluster-users/2015-November/024359.html
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user