Yaniv Bronhaim 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
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 
matter
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 <mper...@redhat.com>
Gerrit-Reviewer: Martin Peřina <mper...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to