x/pyghmi
Code Issues Proposed changes

Change from 'Power' to 'Energy'

Browse Source 
While it was 'Power' for the sake of being convenient to people
about power metrics, it is not strictly accurate.  Change to the
accurate description and people will just have to know to look for
Energy rather than Power.

Change-Id: Iaa303b417740015bd35ef0a4d41928a4c40bf933
This commit is contained in:
Jarrod Johnson 2017-10-17 10:58:03 -04:00
parent 243cf797de
commit 0b14ffdbca
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
pyghmi/ipmi/oem/lenovo/imm.py
View File

@ -300,7 +300,7 @@ class IMMClient(object):
return self._energymanager.supportedmeters
def get_oem_sensor_descriptions(self, ipmicmd):
return [{'name': x, 'type': 'Power'
return [{'name': x, 'type': 'Energy'
} for x in self.get_oem_sensor_names(ipmicmd)]
def get_oem_sensor_reading(self, name, ipmicmd):
@ -316,7 +316,7 @@ class IMMClient(object):
'value': kwh, 'states': [],
'state_ids': [],
'health': pygconst.Health.Ok,
'type': 'Power'}, 'kWh')
'type': 'Energy'}, 'kWh')
def weblogout(self):
if self._wc: