glusterfs.git/extras/snap_scheduler, branch release-3.7

snapshot/scheduler: Output correction of initialisation

2015-08-24T07:05:06+00:00

    Backport of http://review.gluster.org/#/c/11924/

Change-Id: I4a6e00805da7b254b8b08e7bb142960fb6c64923
BUG: 1245922
Signed-off-by: Avra Sengupta 
Reviewed-on: http://review.gluster.org/11925
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: mohammed rafi  kc 
Reviewed-by: Rajesh Joseph

snapshot/scheduler: Check if volume exists before adding/editing schedules

2015-08-24T06:57:42+00:00

    Backport of http://review.gluster.org/#/c/11830/

Before adding or editing a scheduler, check if the volume name provided
in the schedule, exists in the cluster or not.

Added return code VOLUME_DOES_NOT_EXIST(17) for the same.

Change-Id: Ia3fe3cc1e1568ddd10f9193bbf40a098f0fe990a
BUG: 1245923
Signed-off-by: Avra Sengupta 
Reviewed-on: http://review.gluster.org/11917
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: mohammed rafi  kc 
Reviewed-by: Rajesh Joseph

snapshot/scheduler: Use /var/run/gluster/shared_storage/snaps/tmp_file for writing data into tmp file and then making an atomic rename to the required filename

2015-07-08T09:39:31+00:00

The reason for using this location is that it adheres to
the selinux policies.

Also moving the update of the current_scheduler file,
under the lock so as to avoid multiple writes

Change-Id: I61e62b5daf6f1bce2319f64f7b1dfb8b93726077
BUG: 1239270
Signed-off-by: Avra Sengupta 
Reviewed-on: http://review.gluster.org/11536
Tested-by: NetBSD Build System 
Tested-by: Gluster Build System 
Reviewed-by: Rajesh Joseph

snapshot/scheduler: Reload /etc/cron.d/glusterfs_snap_cron_tasks when shared storage is available

2015-06-19T11:57:49+00:00

     Backport of http://review.gluster.org/#/c/11139/

If shared storage is not accessible, create a flag in /var/run/gluster/
So that when /etc/cron.d/glusterfs_snap_cron_tasks is
available again, the flag will tell us, to reload
/etc/cron.d/glusterfs_snap_cron_tasks.

Change-Id: I41b19f57ff0b8f7e0b820eaf592b0fdedb0a5d86
BUG: 1230399
Signed-off-by: Avra Sengupta 
Reviewed-on: http://review.gluster.org/11168
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Rajesh Joseph

snapshot/scheduler: Modified main() function to take arguments.

2015-06-14T04:57:28+00:00

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: 1226213
Signed-off-by: nnDarshan 
Reviewed On: http://review.gluster.org/#/c/10760/
Reviewed-on: http://review.gluster.org/10997
Tested-by: Gluster Build System 
Reviewed-by: Aravinda VK 
Tested-by: NetBSD Build System

snapshot/scheduler: Handle OSError in os. callbacks

2015-06-11T12:00:41+00:00

        Backport of http://review.gluster.org/#/c/11087/

Handle OSError and not IOError in os. callbacks.

Change-Id: I2b5bfb629bacbd2d2e410d96034b4e2c11c4931e
BUG: 1230018
Signed-off-by: Avra Sengupta 
(cherry picked from commit d835219a30327ede60e4ef28210914ab30bd0712)
Reviewed-on: http://review.gluster.org/11151
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Rajesh Joseph 
Reviewed-by: Krishnan Parthasarathi

snapshot/scheduler: Check if GCRON_TASKS exists before

2015-06-11T11:50:22+00:00

accessing it's mtime

     Backport of http://review.gluster.org/#/c/11138/

Change-Id: I873c83d21620527b20d7de428d11582c5499d1af
BUG: 1230167
Signed-off-by: Avra Sengupta 
(cherry picked from commit 5b05d3e376e984f00528f5bf2f0febd0220ca91f)
Reviewed-on: http://review.gluster.org/11159
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Rajesh Joseph 
Reviewed-by: Krishnan Parthasarathi

snapshot/scheduler: Return proper error code in case of failure

2015-06-04T09:35:45+00:00

    Backport of http://review.gluster.org/#/c/11005/

ENUM                              RETCODE     ERROR
----------------------------------------------------------
INTERNAL_ERROR                    2           Internal Error
SHARED_STORAGE_DIR_DOESNT_EXIST   3           Shared Storage Dir
                                              does not exist
SHARED_STORAGE_NOT_MOUNTED        4           Shared storage is not mounted
ANOTHER_TRANSACTION_IN_PROGRESS   5           Another transaction is in progress
INIT_FAILED                       6           Initialisation failed
SCHEDULING_ALREADY_DISABLED       7           Scheduler is already disabled
SCHEDULING_ALREADY_ENABLED        8           Scheduler is already enabled
NODE_NOT_INITIALISED              9           Node not initialised
ANOTHER_SCHEDULER_ACTIVE          10          Another scheduler is active
JOB_ALREADY_EXISTS                11          Job already exists
JOB_NOT_FOUND                     12          Job not found
INVALID_JOBNAME                   13          Jobname is invalid
INVALID_VOLNAME                   14          Volname is invalid
INVALID_SCHEDULE                  15          Schedule is invalid
INVALID_ARG                       16          Argument is invalid

Change-Id: Ia1da166659099f4c951fcdb4d755529e41167b80
BUG: 1227615
Signed-off-by: Avra Sengupta 
(cherry picked from commit 9798a24febba9bbf28e97656b81b8a01a1325f68)
Reviewed-on: http://review.gluster.org/11057
Tested-by: Gluster Build System 
Tested-by: NetBSD Build System 
Reviewed-by: Rajesh Joseph 
Reviewed-by: Krishnan Parthasarathi

snapshot/scheduler: Do not enable scheduler if another scheduler is running

2015-06-01T11:13:31+00:00

     Backport of http://review.gluster.org/#/c/10641/

Check if another snapshot scheduler is running before
enabling the scheduler

Also introducing a hidden option, disable_force

"snapshot_scheduler.py disable_force" will disable the
cli snapshot scheduler from any node, even though the node
has not been initialised for the scheduler, as long as the
shared storage is mounted

This option is hidden, because we don't want to encourage
users to use all commands from nodes that are not initialised.

Change-Id: I7ad941fbbab834225a36e740c61f8e740813e7c8
BUG: 1226120
Signed-off-by: Avra Sengupta 
Reviewed-on: http://review.gluster.org/10641
Reviewed-by: Rajesh Joseph 
Tested-by: NetBSD Build System
Reviewed-by: Kaushal M 
(cherry picked from commit d67eb34b2a5b5e3cb926ff4c86a163148743829c)
Reviewed-on: http://review.gluster.org/10993
Tested-by: Gluster Build System 
Reviewed-by: Aravinda VK 
Reviewed-by: Krishnan Parthasarathi 
Tested-by: Krishnan Parthasarathi

snapshot/scheduler: Use shutil.move instead of os.rename()

2015-05-06T11:17:24+00:00

os.rename is a wrapper on top of the rename function,
which fails with invalid cross-device link if /tmp
is a tmpfs. Hence using shutil.move

Change-Id: Ia026d2a810b725ccd398db895e612c53bc6a2f95
BUG: 1218575
Signed-off-by: Avra Sengupta 
Reviewed-on: http://review.gluster.org/10347
Reviewed-by: Rajesh Joseph 
Reviewed-by: Aravinda VK 
Tested-by: Gluster Build System 
Reviewed-by: Krishnan Parthasarathi 
Reviewed-on: http://review.gluster.org/10560