Deepak C Shetty has posted comments on this change.

Change subject: [RFC] Support for GLUSTERFS_DOMAIN
......................................................................


Patch Set 3: No score

(2 inline comments)

....................................................
Commit Message
Line 3: AuthorDate: 2012-07-31 18:41:21 +0530
Line 4: Commit:     Deepak C Shetty <[email protected]>
Line 5: CommitDate: 2012-08-24 14:22:57 +0530
Line 6: 
Line 7: [RFC] Support for GLUSTERFS_DOMAIN
Done
Line 8: 
Line 9: This patch introduces a new storage domain of type
Line 10: GLUSTERFS_DOMAIN, which uses gluster as the storage backend.
Line 11: 


....................................................
File vdsm/libvirtvm.py
Line 999:             return False
Line 1000: 
Line 1001:     @property
Line 1002:     def blockDev(self):
Line 1003:         if self._blockDev is None and not self.networkDev:
My bad.
Done.
Line 1004:             try:
Line 1005:                 self._blockDev = utils.isBlockDevice(self.path)
Line 1006:             except:
Line 1007:                 self.log.debug("Unable to determine if the path '%s' 
is a "


--
To view, visit http://gerrit.ovirt.org/6856
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9ac37da88625f20d148beaf53bb6371c15b33ad7
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Deepak C Shetty <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Deepak C Shetty <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Royce Lv <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Shu Ming <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to