Bug 1396901

Summary:	Scheduler : Scheduler should not depend on glusterfs-events package
Product:	[Community] GlusterFS	Reporter:	Avra Sengupta <asengupt>
Component:	snapshot	Assignee:	Avra Sengupta <asengupt>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	urgent	Docs Contact:
Priority:	unspecified
Version:	3.9	CC:	ashah, bugs, rhs-bugs, rjoseph, storage-qa-internal
Target Milestone:	---
Target Release:	---
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:	glusterfs-3.9.1	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:	1396364	Environment:
Last Closed:	2017-03-08 10:19:16 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1396361, 1396364
Bug Blocks:

Description Avra Sengupta 2016-11-21 06:20:40 UTC

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

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

Description of problem:

When initializing snapshot scheduler , it gives dependency error.
  

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

glusterfs-3.8.4-5.el6rhs.x86_64

How reproducible:
100%

Steps to Reproduce:
1. Create 2*2 distribute volume
2. Enable shared storage
3. initialize snapshot scheduler 

Actual results:

[root@dhcp47-158 ~]# snap_scheduler.py init
Traceback (most recent call last):
  File "/usr/sbin/snap_scheduler.py", line 24, in <module>
    from events.gf_event import gf_event



Expected results:

Scheduler should not depend on glusterfs-events package


Additional info:

--- Additional comment from Red Hat Bugzilla Rules Engine on 2016-11-18 02:32:46 EST ---

This bug is automatically being proposed for the current release of Red Hat Gluster Storage 3 under active development, by setting the release flag 'rhgs‑3.2.0' to '?'. 

If this bug should be proposed for a different release, please manually change the proposed release flag.

--- Additional comment from Worker Ant on 2016-11-18 03:09:37 EST ---

REVIEW: http://review.gluster.org/15876 (snapshot/scheduler: Removing dependency of scheduler on eventing) posted (#1) for review on master by Avra Sengupta (asengupt)

--- Additional comment from Worker Ant on 2016-11-21 01:18:17 EST ---

COMMIT: http://review.gluster.org/15876 committed in master by Rajesh Joseph (rjoseph) 
------
commit 6b2c40d337bd30eea3ce56a71db8864e61a9a411
Author: Avra Sengupta <asengupt>
Date:   Fri Nov 18 13:13:21 2016 +0530

    snapshot/scheduler: Removing dependency of scheduler on eventing
    
    Change-Id: I7de156d8186c32092ec5e9d174d023f4782947c0
    BUG: 1396364
    Signed-off-by: Avra Sengupta <asengupt>
    Reviewed-on: http://review.gluster.org/15876
    Reviewed-by: Aravinda VK <avishwan>
    Smoke: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Rajesh Joseph <rjoseph>

Comment 1 Worker Ant 2016-11-21 06:22:31 UTC

REVIEW: http://review.gluster.org/15891 (snapshot/scheduler: Removing dependency of scheduler on eventing) posted (#1) for review on release-3.9 by Avra Sengupta (asengupt)

Comment 2 Worker Ant 2016-11-22 05:45:23 UTC

COMMIT: http://review.gluster.org/15891 committed in release-3.9 by Rajesh Joseph (rjoseph) 
------
commit 60ab6cb4e2598757279803efacb4dbcbbfce8c58
Author: Avra Sengupta <asengupt>
Date:   Fri Nov 18 13:13:21 2016 +0530

    snapshot/scheduler: Removing dependency of scheduler on eventing
    
    > Reviewed-on: http://review.gluster.org/15876
    > Reviewed-by: Aravinda VK <avishwan>
    > Smoke: Gluster Build System <jenkins.org>
    > NetBSD-regression: NetBSD Build System <jenkins.org>
    > CentOS-regression: Gluster Build System <jenkins.org>
    > Reviewed-by: Rajesh Joseph <rjoseph>
    
    (cherry picked from commit 6b2c40d337bd30eea3ce56a71db8864e61a9a411)
    
    Change-Id: I7de156d8186c32092ec5e9d174d023f4782947c0
    BUG: 1396901
    Signed-off-by: Avra Sengupta <asengupt>
    Reviewed-on: http://review.gluster.org/15891
    Smoke: Gluster Build System <jenkins.org>
    Reviewed-by: Rajesh Joseph <rjoseph>
    CentOS-regression: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>

Comment 3 Kaushal 2017-03-08 10:19:16 UTC

This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.9.1, please open a new bug report.

glusterfs-3.9.1 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://lists.gluster.org/pipermail/gluster-users/2017-January/029725.html
[2] https://www.gluster.org/pipermail/gluster-users/