Bug 1374167 - disperse: Integrate important events with events framework
Summary: disperse: Integrate important events with events framework
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: disperse
Version: 3.9
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Ashish Pandey
QA Contact:
URL:
Whiteboard:
Depends On: 1371470 1376805
Blocks: 1361084
TreeView+ depends on / blocked
 
Reported: 2016-09-08 07:24 UTC by Ashish Pandey
Modified: 2017-03-08 08:34 UTC (History)
1 user (show)

Fixed In Version: 3.9.1
Clone Of: 1371470
Environment:
Last Closed: 2017-03-08 08:34:14 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Ashish Pandey 2016-09-08 07:24:45 UTC
+++ This bug was initially created as a clone of Bug #1371470 +++

Description of problem:

Add following events for EC translator

1- EVENT_EC_DATA_BRICKS_NOT_UP -
   When atlest "K" number of bricks, required
   for any ec fop, are not up.
2- EVENT_EC_DATA_BRICKS_UP
   When atlest "K" number of bricks, required
   for any ec fop, are up.

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-08-30 06:07:41 EDT ---

REVIEW: http://review.gluster.org/15348 (cluster/ec: Add events for EC translator) posted (#1) for review on master by Ashish Pandey (aspandey)

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

REVIEW: http://review.gluster.org/15348 (cluster/ec: Add events for EC translator) posted (#2) for review on master by Ashish Pandey (aspandey)

--- Additional comment from Worker Ant on 2016-08-31 02:01:47 EDT ---

REVIEW: http://review.gluster.org/15348 (cluster/ec: Add events for EC translator) posted (#3) for review on master by Ashish Pandey (aspandey)

--- Additional comment from Worker Ant on 2016-08-31 04:22:02 EDT ---

REVIEW: http://review.gluster.org/15348 (cluster/ec: Add events for EC translator) posted (#4) for review on master by Ashish Pandey (aspandey)

--- Additional comment from Worker Ant on 2016-08-31 08:29:01 EDT ---

REVIEW: http://review.gluster.org/15348 (cluster/ec: Add events for EC translator) posted (#5) for review on master by Ashish Pandey (aspandey)

--- Additional comment from Worker Ant on 2016-09-06 18:31:38 EDT ---

REVIEW: http://review.gluster.org/15348 (cluster/ec: Add events for EC translator) posted (#6) for review on master by Pranith Kumar Karampuri (pkarampu)

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

COMMIT: http://review.gluster.org/15348 committed in master by Pranith Kumar Karampuri (pkarampu) 
------
commit e1584b1c1df656612f6233ac40ba55f29591b0c3
Author: Ashish Pandey <aspandey>
Date:   Tue Aug 30 14:17:58 2016 +0530

    cluster/ec: Add events for EC translator
    
    This patch will generates events in following
    cases which will be consumed by new event
    framework.
    
    Consider an EC volume with (K+M) configuration
    K = Data bricks
    M = Redundancy bricks
    
    1- EVENT_EC_MIN_BRICKS_NOT_UP -
       When minimum "K" number of bricks, required
       for any ec fop, are not up.
    2- EVENT_EC_MIN_BRICKS_UP
       When minimum "K" number of bricks, required
       for any ec fop, are up.
    
    Change-Id: I0414b8968c39740a171e5aa14b087afd524d574f
    BUG: 1371470
    Signed-off-by: Ashish Pandey <aspandey>
    Reviewed-on: http://review.gluster.org/15348
    Tested-by: Pranith Kumar Karampuri <pkarampu>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Smoke: Gluster Build System <jenkins.org>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>

Comment 1 Worker Ant 2016-09-08 07:28:31 UTC
REVIEW: http://review.gluster.org/15425 (cluster/ec: Add events for EC translator) posted (#1) for review on release-3.9 by Ashish Pandey (aspandey)

Comment 2 Worker Ant 2016-09-08 12:21:55 UTC
COMMIT: http://review.gluster.org/15425 committed in release-3.9 by Xavier Hernandez (xhernandez) 
------
commit 23e3204489151009f87100fd4eb98a33210a4565
Author: Ashish Pandey <aspandey>
Date:   Tue Aug 30 14:17:58 2016 +0530

    cluster/ec: Add events for EC translator
    
    This patch will generates events in following
    cases which will be consumed by new event
    framework.
    
    Consider an EC volume with (K+M) configuration
    K = Data bricks
    M = Redundancy bricks
    
    1- EVENT_EC_MIN_BRICKS_NOT_UP -
       When minimum "K" number of bricks, required
       for any ec fop, are not up.
    2- EVENT_EC_MIN_BRICKS_UP
       When minimum "K" number of bricks, required
       for any ec fop, are up.
    
    >Change-Id: I0414b8968c39740a171e5aa14b087afd524d574f
    >BUG: 1371470
    >Signed-off-by: Ashish Pandey <aspandey>
    >Reviewed-on: http://review.gluster.org/15348
    >Tested-by: Pranith Kumar Karampuri <pkarampu>
    >NetBSD-regression: NetBSD Build System <jenkins.org>
    >CentOS-regression: Gluster Build System <jenkins.org>
    >Smoke: Gluster Build System <jenkins.org>
    >Reviewed-by: Pranith Kumar Karampuri <pkarampu>
    
    Change-Id: I0414b8968c39740a171e5aa14b087afd524d574f
    BUG: 1374167
    Signed-off-by: Ashish Pandey <aspandey>
    Reviewed-on: http://review.gluster.org/15425
    Smoke: Gluster Build System <jenkins.org>
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>


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