Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1220670

Summary: snap_scheduler script must be usable as python module.
Product: [Community] GlusterFS Reporter: Darshan <dnarayan>
Component: snapshotAssignee: Darshan <dnarayan>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: mainlineCC: bugs, gluster-bugs
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: Scheduler
Fixed In Version: glusterfs-3.8rc2 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1226213 1231078 (view as bug list) Environment:
Last Closed: 2016-06-16 13:00:40 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:    
Bug Blocks: 1226213, 1231078    

Description Darshan 2015-05-12 07:30:12 UTC
Description of problem:
snap_scheduler script must be usable as python module. It would be better for other python programs to use snap_scheduler as a module, than executing it. 

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


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Anand Avati 2015-05-12 12:58:24 UTC
REVIEW: http://review.gluster.org/10760 (snapshot/scheduler: Modified main() function to take arguments.) posted (#1) for review on master by darshan n (dnarayan)

Comment 2 Anand Avati 2015-05-19 06:58:10 UTC
REVIEW: http://review.gluster.org/10760 (snapshot/scheduler: Modified main() function to take arguments.) posted (#2) for review on master by darshan n (dnarayan)

Comment 3 Anand Avati 2015-05-20 05:46:04 UTC
REVIEW: http://review.gluster.org/10760 (snapshot/scheduler: Modified main() function to take arguments.) posted (#3) for review on master by darshan n (dnarayan)

Comment 4 Anand Avati 2015-05-29 08:20:02 UTC
REVIEW: http://review.gluster.org/10760 (snapshot/scheduler: Modified main() function to take arguments.) posted (#5) for review on master by darshan n (dnarayan)

Comment 5 Anand Avati 2015-06-02 08:47:18 UTC
REVIEW: http://review.gluster.org/10760 (snapshot/scheduler: Modified main() function to take arguments.) posted (#6) for review on master by Aravinda VK (avishwan)

Comment 6 Anand Avati 2015-06-03 06:17:13 UTC
REVIEW: http://review.gluster.org/10760 (snapshot/scheduler: Modified main() function to take arguments.) posted (#7) for review on master by Aravinda VK (avishwan)

Comment 7 Anand Avati 2015-06-04 07:00:52 UTC
COMMIT: http://review.gluster.org/10760 committed in master by Krishnan Parthasarathi (kparthas) 
------
commit b15ee49c94c07ffd3c27384b45840eafd59997d4
Author: nnDarshan <dnarayan>
Date:   Tue May 12 17:57:46 2015 +0530

    snapshot/scheduler: Modified main() function to take arguments.
    
    Modified the main function to take script arguments, so that this
    script can be used as a module by other programs .
    
    Change-Id: I902f0bc7ddfbf0d335cc087f51b1a7af4b7157fc
    BUG: 1220670
    Signed-off-by: n Darshan <dnarayan>
    Reviewed-on: http://review.gluster.org/10760
    Tested-by: Gluster Build System <jenkins.com>
    Tested-by: NetBSD Build System <jenkins.org>
    Reviewed-by: Aravinda VK <avishwan>
    Reviewed-by: Krishnan Parthasarathi <kparthas>

Comment 8 Nagaprasad Sathyanarayana 2015-10-25 14:54:42 UTC
Fix for this BZ is already present in a GlusterFS release. You can find clone of this BZ, fixed in a GlusterFS release and closed. Hence closing this mainline BZ as well.

Comment 9 Niels de Vos 2016-06-16 13:00:40 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.8.0, please open a new bug report.

glusterfs-3.8.0 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://blog.gluster.org/2016/06/glusterfs-3-8-released/
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user