Zhou Zheng Sheng has posted comments on this change.

Change subject: Don't monitor the usage of '/var/run/vdsm' in diskStats
......................................................................


Patch Set 2: (1 inline comment)

....................................................
File lib/vdsm/utils.py
Line 230:     A sample of host-related statistics.
Line 231: 
Line 232:     Contains the sate of the host in the time of initialization.
Line 233:     """
Line 234:     MONITORED_PATHS = ['/tmp', '/var/log', '/var/log/core']
I think we can keep /tmp, /var/tmp and other directories, then filter out the 
directory which is on tmpfs.
Line 235: 
Line 236:     def _getDiskStats(self):
Line 237:         d = {}
Line 238:         for p in self.MONITORED_PATHS:


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idb0a4ae2cf7ceb6297e348d9e90c166373461ca1
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Mark Wu <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Deepak C Shetty <[email protected]>
Gerrit-Reviewer: Mark Wu <[email protected]>
Gerrit-Reviewer: Zhou Zheng Sheng <[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