[Bugs] [Bug 1252903] New: Fix invalid logic in tier.t

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1252903

            Bug ID: 1252903
           Summary: Fix invalid logic in tier.t
           Product: GlusterFS
           Version: 3.7.3
         Component: tiering
          Priority: high
          Assignee: bugs at gluster.org
          Reporter: dlambrig at redhat.com
        QA Contact: bugs at gluster.org
                CC: bugs at gluster.org
        Depends On: 1231268
            Blocks: 1244290



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

The tier.t logic did not properly check for corrupted data after migration.

--- Additional comment from Anand Avati on 2015-06-17 11:15:57 EDT ---

REVIEW: http://review.gluster.org/11285 (cluster/tier : fixed logical bug in
tier.t) posted (#1) for review on master by Pamela Ousley (pousley at redhat.com)

--- Additional comment from Anand Avati on 2015-06-22 10:48:19 EDT ---

REVIEW: http://review.gluster.org/11285 (cluster/tier : fixed logical bug in
tier.t) posted (#2) for review on master by Pamela Ousley (pousley at redhat.com)

--- Additional comment from Anand Avati on 2015-06-22 13:56:08 EDT ---

REVIEW: http://review.gluster.org/11285 (cluster/tier : fixed logical bug in
tier.t) posted (#3) for review on master by Pamela Ousley (pousley at redhat.com)

--- Additional comment from Anand Avati on 2015-06-26 19:25:40 EDT ---

REVIEW: http://review.gluster.org/11285 (cluster/tier : fixed logical bug in
tier.t) posted (#4) for review on master by Pamela Ousley (pousley at redhat.com)

--- Additional comment from Anand Avati on 2015-06-29 11:32:15 EDT ---

REVIEW: http://review.gluster.org/11285 (cluster/tier : fixed logical bug in
tier.t) posted (#5) for review on master by Pamela Ousley (pousley at redhat.com)

--- Additional comment from Anand Avati on 2015-06-29 11:45:30 EDT ---

REVIEW: http://review.gluster.org/11285 (cluster/tier : fixed logical bug in
tier.t) posted (#6) for review on master by Pamela Ousley (pousley at redhat.com)

--- Additional comment from Anand Avati on 2015-07-07 15:45:36 EDT ---

REVIEW: http://review.gluster.org/11285 (cluster/tier : fix for logical
bugs/timing errors in tier.t) posted (#7) for review on master by Pamela Ousley
(pousley at redhat.com)

--- Additional comment from Anand Avati on 2015-07-07 15:58:14 EDT ---

REVIEW: http://review.gluster.org/11285 (cluster/tier : fix for logical
bugs/timing errors in tier.t) posted (#8) for review on master by Pamela Ousley
(pousley at redhat.com)

--- Additional comment from Anand Avati on 2015-07-13 13:14:56 EDT ---

REVIEW: http://review.gluster.org/11285 (cluster/tier : fix for logical
bugs/timing errors in tier.t) posted (#9) for review on master by Pamela Ousley
(pousley at redhat.com)

--- Additional comment from Anand Avati on 2015-07-13 17:12:30 EDT ---

COMMIT: http://review.gluster.org/11285 committed in master by Dan Lambright
(dlambrig at redhat.com) 
------
commit f7b9379a0a03fb1d2df56bbfc8dca06f5f3b8437
Author: Pamela Ousley <pousley at redhat.com>
Date:   Wed Jun 17 15:07:55 2015 -0400

    cluster/tier : fix for logical bugs/timing errors in tier.t

    The md5sum fingerprints were not correctly compared after moving
    files between the hot and cold tiers.

    This version of tier.t uses a new function, "check_counters", to
    ensure that the number of promotions/demotions is as expected.
    This is intended to avoid spurious timing-related errors that were
    seen with the old script.

    Change-Id: I4a0ae7315493bfd307a0f68f21fa3ea33c88b08f
    BUG: 1231268
    Signed-off-by: Pamela Ousley <pousley at redhat.com>
    Reviewed-on: http://review.gluster.org/11285
    Tested-by: NetBSD Build System <jenkins at build.gluster.org>
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Dan Lambright <dlambrig at redhat.com>

--- Additional comment from Anand Avati on 2015-08-11 15:38:21 EDT ---

REVIEW: http://review.gluster.org/11885 (cluster/tier: fix 64 bit issue with
sql query using times) posted (#1) for review on master by Dan Lambright
(dlambrig at redhat.com)


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1231268
[Bug 1231268] Fix invalid logic in tier.t
https://bugzilla.redhat.com/show_bug.cgi?id=1244290
[Bug 1244290] Fix invalid logic in tier.t
-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are on the CC list for the bug.
You are the assignee for the bug.