Change in vdsm[master]: gluster: volume snapshot config list and set verbs.

danken at redhat.com danken at redhat.com
Sun Mar 8 13:38:36 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: gluster: volume snapshot config list and set verbs.
......................................................................


gluster: volume snapshot config list and set verbs.

This patch adds new verbs to list and set the
configuration of gluster volume snapshots.

*glusterSnapshotConfigList: get the snapshot configuration.
This verb gives the snapshot configuration for the cluster
and for all the volumes present in it.
Returns:
{'system':{'snap-max-hard-limit': 'hardlimit',
           'snap-max-soft-limit': 'softLimit',
           'auto-delete': 'enable/disable'},
 'volume':{'name' :
            {'snap-max-hard-limit: 'hardlimit',
             'effective-hard-limit: 'EffectiveHardlimit',
             'snap-max-soft-limit': 'softLimit'}
          }...
}

*glusterVolumeSnapshotConfigList: Get the snapshot
configuration for a volume. Volume name should be provided
as an argument, configuration for that volume along with
cluster snapshot configuration is listed.
Returns:
{'system':{'snap-max-hard-limit': 'hardlimit',
           'snap-max-soft-limit': 'softLimit',
           'auto-delete': 'enable/disable'},
 'volume':{'name' :
            {'snap-max-hard-limit: 'hardlimit',
             'effective-hard-limit: 'EffectiveHardlimit',
             'snap-max-soft-limit': 'softLimit'}
          }
}

*glusterSnapshotConfigSet: Set snapshot configuration
at cluster leval. The possible configurations that can
be set are:
snap-max-hard-limit, snap-max-soft-limit and auto-delete

*glusterVolumeSnapshotConfigSet: Set snapshot configuration
at volume leval. Volume name has to be provided as argument.
And the configurable parameter is snap-max-hard-limit.

Change-Id: Iee32a95cf028cda33f25776c08820c5ed80f1033
Signed-off-by: Darshan N <dnarayan at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/34741
Reviewed-by: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M client/vdsClientGluster.py
M tests/Makefile.am
A tests/glusterSnapshotConfig.xml
M tests/glusterTestData.py
M tests/gluster_cli_tests.py
M vdsm.spec.in
M vdsm/gluster/api.py
M vdsm/gluster/apiwrapper.py
M vdsm/gluster/cli.py
M vdsm/gluster/exception.py
M vdsm/rpc/vdsmapi-gluster-schema.json
11 files changed, 354 insertions(+), 0 deletions(-)

Approvals:
  Piotr Kliczewski: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved
  Darshan N: Verified



-- 
To view, visit https://gerrit.ovirt.org/34741
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iee32a95cf028cda33f25776c08820c5ed80f1033
Gerrit-PatchSet: 14
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list