Bug 1440742

Summary:	Test files clean up for tier during 3.10
Product:	[Community] GlusterFS	Reporter:	hari gowtham <hgowtham>
Component:	tiering	Assignee:	hari gowtham <hgowtham>
Status:	CLOSED CURRENTRELEASE	QA Contact:	bugs <bugs>
Severity:	low	Docs Contact:
Priority:	medium
Version:	3.10	CC:	bugs, rtalur
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.10.2	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:	1421600	Environment:
Last Closed:	2017-05-31 20:47:07 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:	1421600
Bug Blocks:	1442603

Description hari gowtham 2017-04-10 12:12:18 UTC

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

Description of problem:
there have been more than 5 bad tests in tier. these test have to be fixed

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

How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:
regression tests werent running on these tests

Expected results:
they have to run on these tests.

Additional info:

--- Additional comment from Worker Ant on 2017-02-13 04:00:38 EST ---

REVIEW: https://review.gluster.org/16605 (TIER/TESTS: improving regression test for tier) posted (#1) for review on master by hari gowtham (hari.gowtham005)

--- Additional comment from Worker Ant on 2017-02-14 06:59:26 EST ---

REVIEW: https://review.gluster.org/16605 (TIER/TESTS: improving regression test for tier) posted (#2) for review on master by hari gowtham (hari.gowtham005)

--- Additional comment from Worker Ant on 2017-03-10 00:37:57 EST ---

REVIEW: https://review.gluster.org/16605 (TIER/TESTS: improving regression test for tier) posted (#3) for review on master by hari gowtham (hari.gowtham005)

--- Additional comment from Worker Ant on 2017-03-10 02:26:16 EST ---

REVIEW: https://review.gluster.org/16605 (TIER/TESTS: improving regression test for tier) posted (#4) for review on master by hari gowtham (hari.gowtham005)

--- Additional comment from Worker Ant on 2017-03-15 05:07:02 EDT ---

REVIEW: https://review.gluster.org/16605 (TIER/TESTS: improving regression test for tier) posted (#5) for review on master by hari gowtham (hari.gowtham005)

--- Additional comment from Worker Ant on 2017-03-22 03:21:26 EDT ---

REVIEW: https://review.gluster.org/16605 (TIER/TESTS: improving regression test for tier) posted (#6) for review on master by hari gowtham (hari.gowtham005)

--- Additional comment from Worker Ant on 2017-04-10 05:30:47 EDT ---

COMMIT: https://review.gluster.org/16605 committed in master by Raghavendra Talur (rtalur) 
------
commit 44b1a68bc9bc1605d208d50fbafae85967085e58
Author: hari gowtham <hgowtham>
Date:   Fri Feb 10 13:00:16 2017 +0530

    TIER/TESTS: improving regression test for tier
    
    The test files that were marked as bad test were checked and
    updated for centos. The tests that had issue were fixed.
    Tests that aren't needed anymore are removed.
    
    REASON:
    
    tests/basic/tier/tier-file-create.t
    This test checks one line after creating a tiered volume (which
    is done in every tier test). So this line is moved along with
    other test in tier and the file is deleted.
    
    tests/bugs/tier/bug-1286974.t
    This bug checks for the tier as a task and tier has been moved
    from a task to service as a part of the tier as a service patch
    https://review.gluster.org/#/c/13365/
    So it is removed from bad tests.
    
    tests/basic/tier/record-metadata-heat.t
    This test had a bug and has been fixed.
    
    tests/basic/tier/bug-1214222-directories_missing_after_attach_tier.t
    tests/basic/tier/fops-during-migration.t
    tests/basic/tier/tier-snapshot.t
    tests/basic/tier/tier_lookup_heal.t
    
    These test seem to work fine on centos now.
    
    Change-Id: I05537f4bbb91584410177ce43543897eff8761a1
    BUG: 1421600
    Signed-off-by: hari gowtham <hgowtham>
    Reviewed-on: https://review.gluster.org/16605
    Smoke: Gluster Build System <jenkins.org>
    Tested-by: hari gowtham <hari.gowtham005>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    Reviewed-by: Jeff Darcy <jeff.us>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Raghavendra Talur <rtalur>

Comment 1 Worker Ant 2017-04-10 12:12:58 UTC

REVIEW: https://review.gluster.org/17027 (TIER/TESTS: improving regression test for tier) posted (#1) for review on release-3.10 by hari gowtham (hari.gowtham005)

Comment 2 Worker Ant 2017-05-03 09:05:55 UTC

REVIEW: https://review.gluster.org/17027 (TIER/TESTS: improving regression test for tier) posted (#2) for review on release-3.10 by hari gowtham (hari.gowtham005)

Comment 3 Worker Ant 2017-05-10 06:29:09 UTC

COMMIT: https://review.gluster.org/17027 committed in release-3.10 by Raghavendra Talur (rtalur) 
------
commit fd7c58880344477441995fd541853e8041506319
Author: hari gowtham <hgowtham>
Date:   Fri Feb 10 13:00:16 2017 +0530

    TIER/TESTS: improving regression test for tier
    
    The test files that were marked as bad test were checked and
    updated for centos. The tests that had issue were fixed.
    Tests that aren't needed anymore are removed.
    
    REASON:
    
    tests/basic/tier/tier-file-create.t
    This test checks one line after creating a tiered volume (which
    is done in every tier test). So this line is moved along with
    other test in tier and the file is deleted.
    
    tests/bugs/tier/bug-1286974.t
    This bug checks for the tier as a task and tier has been moved
    from a task to service as a part of the tier as a service patch
    https://review.gluster.org/#/c/13365/
    So it is removed from bad tests.
    
    tests/basic/tier/record-metadata-heat.t
    This test had a bug and has been fixed.
    
    tests/basic/tier/bug-1214222-directories_missing_after_attach_tier.t
    tests/basic/tier/fops-during-migration.t
    tests/basic/tier/tier-snapshot.t
    tests/basic/tier/tier_lookup_heal.t
    
    These test seem to work fine on centos now.
    
    >Change-Id: I05537f4bbb91584410177ce43543897eff8761a1
    >BUG: 1421600
    >Signed-off-by: hari gowtham <hgowtham>
    >Reviewed-on: https://review.gluster.org/16605
    >Smoke: Gluster Build System <jenkins.org>
    >Tested-by: hari gowtham <hari.gowtham005>
    >NetBSD-regression: NetBSD Build System <jenkins.org>
    >Reviewed-by: Jeff Darcy <jeff.us>
    >CentOS-regression: Gluster Build System <jenkins.org>
    >Reviewed-by: Raghavendra Talur <rtalur>
    
    Change-Id: I05537f4bbb91584410177ce43543897eff8761a1
    BUG: 1440742
    Signed-off-by: hari gowtham <hgowtham>
    Change-Id: I9402312608de1ede28009ec52f7385e45678ed75
    Reviewed-on: https://review.gluster.org/17027
    Tested-by: hari gowtham <hari.gowtham005>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Raghavendra Talur <rtalur>
    Smoke: Gluster Build System <jenkins.org>

Comment 4 Raghavendra Talur 2017-05-31 20:47:07 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.10.2, please open a new bug report.