Bug 1405450

Summary:	tests/bugs/snapshot/bug-1316437.t test is causing spurious failure
Product:	[Community] GlusterFS	Reporter:	rjoseph
Component:	snapshot	Assignee:	rjoseph
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	medium	Docs Contact:
Priority:	medium
Version:	3.8	CC:	bugs
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.8.8	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:	1404573	Environment:
Last Closed:	2017-01-16 12:27:19 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:	1404573
Bug Blocks:

Description rjoseph 2016-12-16 14:40:56 UTC

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

Description of problem:
https://build.gluster.org/job/centos6-regression/1624/consoleText

================================================================================
[13:51:48] Running tests in file ./tests/bugs/snapshot/bug-1316437.t
./tests/bugs/snapshot/bug-1316437.t .. 
1..7
ok 1, LINENUM:9
ok 2, LINENUM:13
ok 3, LINENUM:15
ok 4, LINENUM:17
ok 5, LINENUM:19
not ok 6 , LINENUM:24
FAILED COMMAND: ! [ 14775 -gt 0 ]
ok 7, LINENUM:28
Failed 1/7 subtests 

Test Summary Report
-------------------
./tests/bugs/snapshot/bug-1316437.t (Wstat: 0 Tests: 7 Failed: 1)
  Failed test:  6
Files=1, Tests=7, 22 wallclock secs ( 0.02 usr  0.01 sys +  2.44 cusr  1.55 csys =  4.02 CPU)
Result: FAIL
End of test ./tests/bugs/snapshot/bug-1316437.t
================================================================================

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 Worker Ant on 2016-12-16 04:08:49 EST ---

REVIEW: http://review.gluster.org/16162 (tests: Fix spurious test failure in bug-1316437.t) posted (#1) for review on master by Rajesh Joseph (rjoseph)

--- Additional comment from Worker Ant on 2016-12-16 09:36:49 EST ---

COMMIT: http://review.gluster.org/16162 committed in master by Rajesh Joseph (rjoseph) 
------
commit e9d8525a0d34130ba2a582109937b8e79eecf6ab
Author: Rajesh Joseph <rjoseph>
Date:   Thu Dec 15 20:51:30 2016 +0530

    tests: Fix spurious test failure in bug-1316437.t
    
    After sending SIGTERM to gluster process we immediately
    check if process exited. We should wait for some time
    before checking process state.
    
    BUG: 1404573
    Change-Id: Iaba0067f6e880a7fe38e11b9fa0fe9bd103b19e2
    Signed-off-by: Rajesh Joseph <rjoseph>
    Reviewed-on: http://review.gluster.org/16162
    Smoke: Gluster Build System <jenkins.org>
    Reviewed-by: Avra Sengupta <asengupt>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    Reviewed-by: N Balachandran <nbalacha>
    CentOS-regression: Gluster Build System <jenkins.org>

Comment 1 Worker Ant 2016-12-16 14:43:03 UTC

REVIEW: http://review.gluster.org/16164 (tests: Fix spurious test failure in bug-1316437.t) posted (#1) for review on release-3.8 by Rajesh Joseph (rjoseph)

Comment 2 Worker Ant 2016-12-16 15:06:40 UTC

REVIEW: http://review.gluster.org/16164 (tests: Fix spurious test failure in bug-1316437.t) posted (#2) for review on release-3.8 by Rajesh Joseph (rjoseph)

Comment 3 Worker Ant 2016-12-20 07:25:53 UTC

COMMIT: http://review.gluster.org/16164 committed in release-3.8 by Rajesh Joseph (rjoseph) 
------
commit e5e9de4bd7c0b9421de5095207a58b9372b1bd7d
Author: Rajesh Joseph <rjoseph>
Date:   Thu Dec 15 20:51:30 2016 +0530

    tests: Fix spurious test failure in bug-1316437.t
    
    After sending SIGTERM to gluster process we immediately
    check if process exited. We should wait for some time
    before checking process state.
    
    
    > Reviewed-on: http://review.gluster.org/16162
    > Smoke: Gluster Build System <jenkins.org>
    > Reviewed-by: Avra Sengupta <asengupt>
    > NetBSD-regression: NetBSD Build System <jenkins.org>
    > Reviewed-by: N Balachandran <nbalacha>
    > CentOS-regression: Gluster Build System <jenkins.org>
    (cherry picked from commit e9d8525a0d34130ba2a582109937b8e79eecf6ab)
    
    BUG: 1405450
    Change-Id: Iaba0067f6e880a7fe38e11b9fa0fe9bd103b19e2
    Signed-off-by: Rajesh Joseph <rjoseph>
    Reviewed-on: http://review.gluster.org/16164
    Smoke: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Avra Sengupta <asengupt>

Comment 4 Worker Ant 2016-12-21 23:22:27 UTC

REVIEW: http://review.gluster.org/16264 (tests: Fix spurious test failure in bug-1316437.t) posted (#1) for review on release-3.8-fb by Kevin Vigor (kvigor)

Comment 5 Niels de Vos 2017-01-16 12:27:19 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.8.8, please open a new bug report.

glusterfs-3.8.8 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] https://lists.gluster.org/pipermail/announce/2017-January/000064.html
[2] https://www.gluster.org/pipermail/gluster-users/