[Bugs] [Bug 1478275] New: separating attach tier and add brick

Fri Aug 4 07:28:18 UTC 2017

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

            Bug ID: 1478275
           Summary: separating attach tier and add brick
           Product: GlusterFS
           Version: 3.11
         Component: tiering
          Severity: low
          Assignee: bugs at gluster.org
          Reporter: hgowtham at redhat.com
        QA Contact: bugs at gluster.org
                CC: bugs at gluster.org
        Depends On: 1376326
            Blocks: 1478273

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

Description of problem:

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-09-15 02:36:47 EDT ---

REVIEW: http://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#1) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from hari gowtham on 2016-09-19 05:35:45 EDT ---

Description of problem:
currently attach tier uses the same code as add brick. it has be separated so
that we can support add brick on tiered volume.

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

How reproducible:
no change in the testing part.both add brick and attach tier command has to
work same as before. 

Steps to Reproduce:
1.create a volume.
2.try adding a brick/ attaching a hot tier
3.

Actual results:
add brick/attach tier successful 

Expected results:
add brick/attach tier successful (same result)

Additional info:
changes are in code level. functionality is not affected.

--- Additional comment from Worker Ant on 2016-09-20 06:08:14 EDT ---

REVIEW: http://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#2) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2016-09-20 06:53:07 EDT ---

REVIEW: http://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#3) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2016-09-22 07:54:50 EDT ---

REVIEW: http://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#4) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2016-09-30 07:30:11 EDT ---

REVIEW: http://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#5) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2016-10-12 08:10:09 EDT ---

REVIEW: http://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#6) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2016-10-13 06:08:31 EDT ---

REVIEW: http://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#7) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2016-10-20 08:46:57 EDT ---

REVIEW: http://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#8) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2016-10-26 08:14:04 EDT ---

REVIEW: http://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#9) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2016-10-27 03:17:20 EDT ---

REVIEW: http://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#10) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2017-03-07 05:24:54 EST ---

REVIEW: https://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#13) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2017-04-03 05:06:13 EDT ---

REVIEW: https://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#14) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2017-04-11 09:00:49 EDT ---

REVIEW: https://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#15) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2017-05-19 06:09:09 EDT ---

REVIEW: https://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#16) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2017-05-22 05:32:22 EDT ---

REVIEW: https://review.gluster.org/15503 (tier: separation of attach tier from
add brick) posted (#17) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2017-05-22 06:32:47 EDT ---

REVIEW: https://review.gluster.org/15503 (tier: separation of attach-tier from
add brick) posted (#18) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2017-05-22 07:28:10 EDT ---

REVIEW: https://review.gluster.org/15503 (tier: separation of attach-tier from
add-brick) posted (#19) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2017-05-26 04:05:35 EDT ---

REVIEW: https://review.gluster.org/15503 (tier: separation of attach-tier from
add-brick) posted (#20) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2017-07-04 07:57:26 EDT ---

REVIEW: https://review.gluster.org/17693 (tier: separation of attach-tier from
add-brick) posted (#1) for review on experimental by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2017-07-05 06:56:03 EDT ---

COMMIT: https://review.gluster.org/17693 committed in experimental by Amar
Tumballi (amarts at redhat.com) 
------
commit 0b3f47372ffbe1c5c07e217bc1730933b8c00b7c
Author: hari gowtham <hgowtham at redhat.com>
Date:   Tue Jul 4 16:16:37 2017 +0530

    tier: separation of attach-tier from add-brick

    PROBLEM: Both attach tier and add brick have the same RPC
    and set of code. This becomes a hurdle while tring to implement
    add brick on a tiered volume.

    FIX: This patch separates the add brick and attach tier
    giving them separate RPCs.

    Conflicts:

        xlators/mgmt/glusterd/src/glusterd-messages.h

    Change-Id: Iec57e972be968a9ff00b15b507e56a4f6dc398a2
    BUG: 1376326
    Signed-off-by: hari gowtham <hgowtham at redhat.com>
    Reviewed-on: https://review.gluster.org/17693
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    Tested-by: hari gowtham <hari.gowtham005 at gmail.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>
    Reviewed-by: Amar Tumballi <amarts at redhat.com>

--- Additional comment from Worker Ant on 2017-07-31 06:16:43 EDT ---

REVIEW: https://review.gluster.org/15503 (tier: separation of attach-tier from
add-brick) posted (#21) for review on master by hari gowtham
(hari.gowtham005 at gmail.com)

--- Additional comment from Worker Ant on 2017-08-01 07:08:09 EDT ---

COMMIT: https://review.gluster.org/15503 committed in master by Atin Mukherjee
(amukherj at redhat.com) 
------
commit e5db980504bc4ace037ea806ad89a1ca897150d5
Author: hari gowtham <hgowtham at redhat.com>
Date:   Mon Jan 23 16:38:00 2017 +0530

    tier: separation of attach-tier from add-brick

    PROBLEM: Both attach tier and add brick have the same RPC
    and set of code. This becomes a hurdle while tring to implement
    add brick on a tiered volume.

    FIX: This patch separates the add brick and attach tier
    giving them separate RPCs.

    Change-Id: Iec57e972be968a9ff00b15b507e56a4f6dc398a2
    BUG: 1376326
    Signed-off-by: hari gowtham <hgowtham at redhat.com>
    Reviewed-on: https://review.gluster.org/15503
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    Tested-by: hari gowtham <hari.gowtham005 at gmail.com>
    Reviewed-by: Samikshan Bairagya <samikshan at gmail.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>

Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1376326
[Bug 1376326] separating attach tier and add brick
https://bugzilla.redhat.com/show_bug.cgi?id=1478273
[Bug 1478273] separating attach tier and add brick
-- 
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.