Bug 1220670 - snap_scheduler script must be usable as python module.
Summary: snap_scheduler script must be usable as python module.
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: snapshot
Version: mainline
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
Assignee: Darshan
QA Contact:
URL:
Whiteboard: Scheduler
Depends On:
Blocks: 1226213 1231078
TreeView+ depends on / blocked
 
Reported: 2015-05-12 07:30 UTC by Darshan
Modified: 2016-06-16 13:00 UTC (History)
2 users (show)

Fixed In Version: glusterfs-3.8rc2
Clone Of:
: 1226213 1231078 (view as bug list)
Environment:
Last Closed: 2016-06-16 13:00:40 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

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


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