glusterfs-snapshot.git/cli, branch development

cli/snapshot : cli fixes for upstream review comments.

2014-04-04T11:25:42+00:00

Change-Id: I69f32920dc48c6c52b579c792f28951bb38db344
Signed-off-by: Sachin Pandit 
Reviewed-on: http://review.gluster.org/7385
Reviewed-by: Raghavendra Bhat 
Reviewed-by: Rajesh Joseph 
Tested-by: Rajesh Joseph

glusterd/snapshot: Making snap operations crash consistent

2014-04-02T13:03:25+00:00

In the events of a volume's brick being down, or a node being
down, making snap ops like create, delete, restore, and
status crash consistent.

Marking snap status of snap bricks which were not snapshotted
because the volume brick was down as -1, and not starting those
snap bricks till the snapshot is taken.

During delete bypassing lvm snapshot remove for snap bricks
whose snap status is -1

During restore bypass replacing xattrs on the snapshot bricks
whose snap status is -1. Also bumping restored volume's version
so as to handle nodes being down. On handshake of a restored
volume, passing brick's snap_status as well.

During snapshot status of the non-snapshotted brick details
display "N/A". If a node is down, the entry itself will not
be displayed.

Change-Id: Id042efd7507829995270da0b2b2a6282a08a053d
Signed-off-by: Avra Sengupta 
Reviewed-on: http://review.gluster.org/7241
Reviewed-by: Vijaikumar Mallikarjuna 
Reviewed-by: Rajesh Joseph 
Tested-by: Rajesh Joseph

snapshot/config : snapshot config changes.

2014-03-25T13:37:12+00:00

Syntax :
        gluster snapshot config [volname] [snap-max-hard-limit ]
                                          [snap-max-soft-limit ]

        1)If volume name is not specified, then the value is set system wide.
        2)If volume name is specified, then the value is set to that particular volume.

        *NOTE : snap-max-soft-limit cannot be specified to individual volumes.

case 1) When system limit is greater than volume limit.
Sample output :

        Snapshot System Configuration:
        snap-max-hard-limit : 200
        snap-max-soft-limit : 50%

        Snapshot Volume Configuration:

        Volume : vol2
        snap-max-hard-limit : 100
        Effective snap-max-hard-limit : 100
        Effective snap-max-soft-limit : 50 (50%)

        Volume : vol1
        snap-max-hard-limit : 150
        Effective snap-max-hard-limit : 150
        Effective snap-max-soft-limit : 75 (50%)

case 2) When system limit is lesser than volume limit.
Sample output :

        Snapshot System Configuration:
        snap-max-hard-limit : 50
        snap-max-soft-limit : 50%

        Snapshot Volume Configuration:

        Volume : vol2
        snap-max-hard-limit : 100
        Effective snap-max-hard-limit : 50
        Effective snap-max-soft-limit : 25 (50%)

        Volume : vol1
        snap-max-hard-limit : 150
        Effective snap-max-hard-limit : 50
        Effective snap-max-soft-limit : 25 (50%)

Change-Id: I97b5daefec7205bb9ab7b5b51d38f504cc5ee940
BUG: 1075034
Signed-off-by: Sachin Pandit 
Reviewed-on: http://review.gluster.org/7303
Reviewed-by: Rajesh Joseph 
Tested-by: Rajesh Joseph

snapshot: cleanup unwanted CG-code

2014-03-25T10:32:21+00:00

Change-Id: Id3da2a7aac84027aeff050f6dd9a3484719362bc
BUG: 1073780
Signed-off-by: Vijaikumar M 
Reviewed-on: http://review.gluster.org/7204
Reviewed-by: Rajesh Joseph 
Tested-by: Rajesh Joseph

glusterd/snapshot: Correct the error message when the snapshot name is

2014-03-25T08:18:18+00:00

more than 255 characters

Change-Id: Iaab6889a86ab39fa73eebf30dc08af98d5a774bc
BUG: 1078166
Signed-off-by: Vijaikumar M 
Reviewed-on: http://review.gluster.org/7296
Reviewed-by: Avra Sengupta 
Reviewed-by: Rajesh Joseph 
Tested-by: Rajesh Joseph

cli/snapshot : Clear messages while creating SNAP with invalid scenarios.

2014-03-24T12:49:05+00:00

1) Create with Description key and without actual description will display proper message.
2) Create with snapname containing special character will fail (volname convention is followed).
3) Create with Multiple Description will fail with proper message.

Change-Id: I9173a0ffe2bf77c10a4e383268bd8d99906217a0
BUG: 1040978
Signed-off-by: Sachin Pandit 
Reviewed-on: http://review.gluster.org/7324
Reviewed-by: Vijaikumar Mallikarjuna 
Reviewed-by: Rajesh Joseph 
Tested-by: Rajesh Joseph

cli/snapshot : Fix for snapshot create fail.

2014-03-14T09:35:16+00:00

Change-Id: Ib3c239eadbcfc787f4f7d0734839c8425e93020a
BUG: 1076356
Signed-off-by: Sachin Pandit 
Reviewed-on: http://review.gluster.org/7275
Reviewed-by: Rajesh Joseph 
Tested-by: Rajesh Joseph

snapshot : Fix for snapshot info ambiguous output.

2014-03-13T11:45:39+00:00

Change-Id: I27e50921a2e030ae50b1fd28ee7ab0be65b313fd
BUG: 1075454
Signed-off-by: Sachin Pandit 
Reviewed-on: http://review.gluster.org/7228
Reviewed-by: Rajesh Joseph 
Tested-by: Rajesh Joseph

cli/snapshot : Fix for snapshot create with snapname l,r,i,d.

2014-03-13T11:44:49+00:00

Change-Id: Ib7fd1ca766997bce86062e5c10c5acf7e5edae60
BUG: 1075512
Signed-off-by: Sachin Pandit 
Reviewed-on: http://review.gluster.org/7236
Reviewed-by: Vijaikumar Mallikarjuna 
Reviewed-by: Rajesh Joseph 
Tested-by: Rajesh Joseph

glusterd/snapshot: Validate snapshot name length while creation

2014-03-13T11:30:41+00:00

Change-Id: Iaee04f5209e278d83c086e32c4cafd6c571370cd
Signed-off-by: Vijaikumar M 
Reviewed-on: http://review.gluster.org/7242
Reviewed-by: Rajesh Joseph 
Tested-by: Rajesh Joseph