Change in vdsm[master]: gluster: Warn instead of failing for unsupported gluster rep...

ahino at redhat.com ahino at redhat.com
Thu Dec 10 10:05:39 UTC 2015


Ala Hino has posted comments on this change.

Change subject: gluster: Warn instead of failing for unsupported gluster replica modes
......................................................................


Patch Set 8:

(1 comment)

https://gerrit.ovirt.org/#/c/50119/8/vdsm/storage/storageServer.py
File vdsm/storage/storageServer.py:

Line 334: 
Line 335:     def validate(self):
Line 336:         replicaCount = self.volinfo['replicaCount']
Line 337:         if replicaCount not in self.ALLOWED_REPLICA_COUNTS:
Line 338:             self.log.warning("Unsupported replica count (%s) for volume %s, "
> why?
using %r vol name is: u'rep2-vol11'

using %s: rep2-vol12

using %r msg is:
jsonrpc.Executor/5::WARNING::2015-12-09 13:28:22,322::storageServer::340::Storage.StorageServer.MountConnection:validate) Unsupported replica count (2) for volume u'rep2-vol11', please upgrade volume to replica 3

using %s:
jsonrpc.Executor/3::WARNING::2015-12-09 13:46:55,731::storageServer::340::Storage.StorageServer.MountConnection:validate) Unsupported replica count (2) for volume rep2-vol12, please upgrade volume to replica 3
Line 339:                              "please upgrade volume to replica 3",
Line 340:                              replicaCount, self._volname)
Line 341: 
Line 342:     def _get_backup_servers_option(self):


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I359e2405153f45b6ae303e00eb0e04a5ae14cb99
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list