2
0
mirror of https://opendev.org/x/pyghmi synced 2025-01-28 11:57:34 +00:00

Fix redfish issues

Fix the remote media generic path to actually try to work.  This
required adding a stub attach function to the generic oemhandler class.

Fix adapter health checks being skipped if no memory errors are seen.

Change-Id: I3c6bc2c32fa63545a89cd005b4fbb2a271dacfbe
This commit is contained in:
Jarrod Johnson 2019-08-02 08:25:25 -04:00
parent bfbe3f41c2
commit 196e70b447
2 changed files with 11 additions and 8 deletions

View File

@ -776,14 +776,14 @@ class Command(object):
meminfo = self.sysinfo['MemorySummary']
meminfo['Name'] = 'Memory'
summary['badreadings'].append(SensorReading(meminfo))
for adapter in self.sysinfo['PCIeDevices']:
adpinfo = self._do_web_request(adapter['@odata.id'])
if adpinfo['Status']['Health'] not in ('OK', None):
summary['badreadings'].append(SensorReading(adpinfo))
for fun in self.sysinfo['PCIeFunctions']:
funinfo = self._do_web_request(fun['@odata.id'])
if funinfo['Status']['Health'] not in ('OK', None):
summary['badreadings'].append(SensorReading(funinfo))
for adapter in self.sysinfo['PCIeDevices']:
adpinfo = self._do_web_request(adapter['@odata.id'])
if adpinfo['Status']['Health'] not in ('OK', None):
summary['badreadings'].append(SensorReading(adpinfo))
for fun in self.sysinfo['PCIeFunctions']:
funinfo = self._do_web_request(fun['@odata.id'])
if funinfo['Status']['Health'] not in ('OK', None):
summary['badreadings'].append(SensorReading(funinfo))
return summary
def _get_biosreg(self, url):

View File

@ -31,6 +31,9 @@ class OEMHandler(object):
return cachent['contents']
return None
def attach_remote_media(self, url, username, password, vmurls):
return None
def get_description(self):
return {}