[Bugs] [Bug 1219027] New: DHT/Tiering/Rebalancer: The Client PID set by tiering migration is getting reset by dht migration

[bugzilla at redhat.com]

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

            Bug ID: 1219027
           Summary: DHT/Tiering/Rebalancer: The Client PID set by tiering
                    migration is getting reset by dht migration
           Product: GlusterFS
           Version: 3.7.0
         Component: distribute
          Assignee: bugs at gluster.org
          Reporter: josferna at redhat.com
                CC: bugs at gluster.org, dlambrig at redhat.com,
                    gluster-bugs at redhat.com, nbalacha at redhat.com,
                    spalai at redhat.com, srangana at redhat.com
        Depends On: 1217937



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

Description of problem:

    dht/tier/rebalancer: Fix set of client pid for migration.

    In the patch http://review.gluster.org/#/c/9657
    the client pid set by tiering migration was getting over-
    written in dht_start_rebalance_task(). Just corrected it
    in dht_setxattr() before calling dht_start_rebalance_task()
    and removed it from dht_start_rebalance_task().


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-03 03:22:08 EDT ---

REVIEW: http://review.gluster.org/10502 (dht/tier/rebalancer: Fix set of client
pid for migration) posted (#1) for review on master by Joseph Fernandes
(josferna at redhat.com)

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

REVIEW: http://review.gluster.org/10502 (dht/tier/rebalancer: Fix reset of
tiering client pid) posted (#2) for review on master by Joseph Fernandes
(josferna at redhat.com)

--- Additional comment from Anand Avati on 2015-05-05 08:52:30 EDT ---

REVIEW: http://review.gluster.org/10502 (dht/tier/rebalancer: Fix reset of
tiering client pid) posted (#3) for review on master by Joseph Fernandes
(josferna at redhat.com)

--- Additional comment from Anand Avati on 2015-05-05 12:03:14 EDT ---

COMMIT: http://review.gluster.org/10502 committed in master by Kaleb KEITHLEY
(kkeithle at redhat.com) 
------
commit a5fe0f594d41e1a11661d9074bb19e9c2e2c4776
Author: Joseph Fernandes <josferna at redhat.com>
Date:   Sun May 3 12:41:40 2015 +0530

    dht/tier/rebalancer: Fix reset of tiering client pid

    In the patch http://review.gluster.org/#/c/9657
    the client pid set by tiering migration was getting over-
    written in dht_start_rebalance_task(). Just corrected it
    in dht_setxattr() before calling dht_start_rebalance_task()
    and removed it from dht_start_rebalance_task().

    Change-Id: I37cfa111f83a4e5d498042575c93799f60b49870
    BUG: 1217937
    Signed-off-by: Joseph Fernandes <josferna at redhat.com>
    Reviewed-on: http://review.gluster.org/10502
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Susant Palai <spalai at redhat.com>
    Reviewed-by: Dan Lambright <dlambrig at redhat.com>


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1217937
[Bug 1217937] DHT/Tiering/Rebalancer: The Client PID set by tiering
migration is getting reset by dht migration
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.