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

bugzilla at redhat.com bugzilla at redhat.com
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.


More information about the Bugs mailing list