Bug 1440742 - Test files clean up for tier during 3.10
Summary: Test files clean up for tier during 3.10
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: tiering
Version: 3.10
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: ---
Assignee: hari gowtham
QA Contact: bugs@gluster.org
URL:
Whiteboard:
Depends On: 1421600
Blocks: glusterfs-3.10.2
TreeView+ depends on / blocked
 
Reported: 2017-04-10 12:12 UTC by hari gowtham
Modified: 2017-05-31 20:47 UTC (History)
2 users (show)

Fixed In Version: glusterfs-3.10.2
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1421600
Environment:
Last Closed: 2017-05-31 20:47:07 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

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.


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