Federico Simoncelli has posted comments on this change.

Change subject: fileSD: enable mailbox on file domains
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.ovirt.org/#/c/26414/4/vdsm/storage/fileSD.py
File vdsm/storage/fileSD.py:

Line 45: FILE_SD_MD_FIELDS[REMOTE_PATH] = (str, str)
Line 46: 
Line 47: METADATA_PERMISSIONS = 0o660
Line 48: 
Line 49: # IDS and LEASES volumes are not used on file domains
This comment is not entirely correct. Revise.
Line 50: FILE_SPECIAL_VOLUME_SIZES_MIB = sd.SPECIAL_VOLUME_SIZES_MIB.copy()
Line 51: FILE_SPECIAL_VOLUME_SIZES_MIB.update({
Line 52:     sd.IDS: 0,
Line 53:     sd.LEASES: 0,


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iba8036e33ed8718d05e9d83cad1d63949794f067
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to