Change in vdsm[master]: sampling: Collect kdump status

mperina at redhat.com mperina at redhat.com
Thu Mar 20 12:39:27 UTC 2014


Martin Peřina has posted comments on this change.

Change subject: sampling: Collect kdump status
......................................................................


Patch Set 2:

(2 comments)

http://gerrit.ovirt.org/#/c/25926/2//COMMIT_MSG
Commit Message:

Line 7: sampling: Collect kdump status
Line 8: 
Line 9: Adds kdump configuration status to sampling module. The status will be
Line 10: reported to engine in xml response using boolean key 'kdumpStatus'.
Line 11: 
> will be reported as part of vdsGetStats response
Done
Line 12: Change-Id: I68d7a2a24fdaad74255004af0f327197eaee65f2
Line 13: Bug-Url: https://bugzilla.redhat.com/970259


http://gerrit.ovirt.org/#/c/25926/2/vdsm/sampling.py
File vdsm/sampling.py:

Line 196:             self.thpState = 'never'
Line 197:         try:
Line 198:             self.kdumpStatus = int(
Line 199:                 file('/sys/kernel/kexec_crash_loaded').read().strip("\n")
Line 200:             )
> you can add the path near _THP_STATE_PATH as constant , but doesn't really 
Done
Line 201:         except:
Line 202:             self.kdumpStatus = -1
Line 203: 
Line 204: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I68d7a2a24fdaad74255004af0f327197eaee65f2
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list