Bug 1226167

Summary:	tiering: use sperate log/socket/pid file for tiering
Product:	[Red Hat Storage] Red Hat Gluster Storage	Reporter:	Mohammed Rafi KC <rkavunga>
Component:	tier	Assignee:	Mohammed Rafi KC <rkavunga>
Status:	CLOSED ERRATA	QA Contact:	Nag Pavan Chilakam <nchilaka>
Severity:	unspecified	Docs Contact:
Priority:	medium
Version:	rhgs-3.1	CC:	annair, asrivast, bugs, josferna, nsathyan, rhs-bugs, rkavunga, storage-qa-internal, trao, vagarwal
Target Milestone:	---	Keywords:	Triaged
Target Release:	RHGS 3.1.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	TIERING
Fixed In Version:	glusterfs-3.7.1-1	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1221970	Environment:
Last Closed:	2015-07-29 04:54:04 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:	1221970
Bug Blocks:	1202842, 1221969

Description Mohammed Rafi KC 2015-05-29 06:18:46 UTC

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

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

Description of problem:



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


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

--- Additional comment from Anand Avati on 2015-05-15 07:27:32 EDT ---

REVIEW: http://review.gluster.org/10792 (tiering/rebalance: Use separate pid/socket file for tiering) posted (#1) for review on master by mohammed rafi  kc (rkavunga)

--- Additional comment from Anand Avati on 2015-05-28 01:30:28 EDT ---

REVIEW: http://review.gluster.org/10792 (tiering/rebalance: Use separate pid/socket file for tiering) posted (#2) for review on master by mohammed rafi  kc (rkavunga)

--- Additional comment from Anand Avati on 2015-05-28 10:03:25 EDT ---

COMMIT: http://review.gluster.org/10792 committed in master by Kaushal M (kaushal) 
------
commit ffc15b7ef0631b571be00cdb7d091cac84e314f5
Author: Mohammed Rafi KC <rkavunga>
Date:   Fri May 15 16:03:00 2015 +0530

    tiering/rebalance: Use separate pid/socket file for tiering
    
    When promotion/demotion daemon starts, it uses the same pidfile
    as rebalance. This patch will introduce a different pid file
    for the same.
    
    Change-Id: Ic484c53f51e00ae6b2d697748a9600b14829e23b
    BUG: 1221970
    Signed-off-by: Mohammed Rafi KC <rkavunga>
    Reviewed-on: http://review.gluster.org/10792
    Reviewed-by: Atin Mukherjee <amukherj>
    Tested-by: Gluster Build System <jenkins.com>
    Tested-by: NetBSD Build System

Comment 4 Triveni Rao 2015-06-11 17:39:05 UTC

how to validate this? any particular command to execute?

description?

Comment 5 Joseph Elwin Fernandes 2015-06-16 08:46:52 UTC

when tiering rebalancer runs you will find this file 

/var/lib/glusterd/vols/test/tier/<uuid>.pid

thatz the validation of this bug.

It will not the pid file of dht rebalancer.

Comment 6 Nag Pavan Chilakam 2015-06-30 07:15:41 UTC

Noting this snippet from parent bug for QE reference and understanding:

============
tiering/rebalance: Use separate pid/socket file for tiering    
          
    
    When promotion/demotion daemon starts, it uses the same pidfile
    as rebalance. This patch will introduce a different pid file
    for the same.
=============

Comment 7 Nag Pavan Chilakam 2015-06-30 10:44:57 UTC

I have verified the bug in following scenarios:
1)attach-tier ---> tier deamon kicks off
2)detach-tier start------->tier deamon stops
3)rebalance volumename tier start------->tier deamon kicks off
4)stop tier volume--->tier deamon dies
5)New volume has new tier deamon kicked of per node per volume seperately

The Process socket was available at /var/lib/glusterd/vols/<volname>/tier/<pidname>.pid

hence moving the bug to verified
Logs and sosreports are attached

Note: But for a detach-tier start, a seperate PID under rebalance folder is getting created to move files to cold tier

Comment 8 Nag Pavan Chilakam 2015-06-30 11:37:57 UTC

QE verified logs available at rhsqe-repo.lab.eng.blr.redhat.com:/home/repo/sosreports/bug.1226167

Comment 9 Mohammed Rafi KC 2015-07-06 11:02:23 UTC

removing the needinfo

Comment 10 errata-xmlrpc 2015-07-29 04:54:04 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHSA-2015-1495.html