HP ILO integration not working in most recent update #109581

Studmuffin1134 · 2024-02-04T02:25:29Z

The problem

The HP ILO integration stopped working the configuration has not change nor has the ilo login details it will not create the entity's any more even though it has worked up until the last upadte

What version of Home Assistant Core has the issue?

core

What was the last working version of Home Assistant Core?

up until the latest update

What type of installation are you running?

Home Assistant OS

Integration causing the issue

hp_ilo

Link to integration documentation on our website

https://www.home-assistant.io/integrations/hp_ilo/

Diagnostics information

I dont have any at this time

Example YAML snippet

- platform: hp_ilo
  host: 192.168.100.47
  username: Studmuffin
  password: no password for u
  monitored_variables:
    - name: Studmuffin XCP-NG Inlet Tempeture
      sensor_type: server_health
      unit_of_measurement: "°C"
      value_template: '{{ ilo_data.temperature["01-Inlet Ambient"].currentreading[0] }}'
    - name: Studmuffin ESXI 1 Power Consumption
      sensor_type: server_power_readings

Anything in the logs that might be useful for us?

Not currently at this time if u need data tell me how to get it and i can get it to u
the code snippet is 1 out of the 4 i have

Additional information

No response

home-assistant · 2024-02-04T02:25:32Z

hp_ilo documentation
hp_ilo source

christophermichaelshaw · 2024-02-06T03:35:12Z

Can confirm ilo integration is broken in 2024.2b8 -- likely due to the Python upgrade.

This may be applicable: eventlet/eventlet#795

`Logger: homeassistant.components.sensor
Source: helpers/entity_platform.py:581
Integration: Sensor (documentation, issues)
First occurred: 19:21:37 (13 occurrences)
Last logged: 19:21:48

hp_ilo: Error on device update!
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 581, in _async_add_entity
await entity.async_device_update(warning=False)
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1261, in async_device_update
await hass.async_add_executor_job(self.update)
File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/hp_ilo/sensor.py", line 166, in update
ilo_data = getattr(self.hp_ilo_data.data, self._ilo_function)()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/hpilo.py", line 1035, in get_embedded_health
return self._info_tag('SERVER_INFO', 'GET_EMBEDDED_HEALTH', 'GET_EMBEDDED_HEALTH_DATA',
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/hpilo.py", line 730, in _info_tag
header, message = self._request(root)
^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/hpilo.py", line 238, in _request
self._detect_protocol()
File "/usr/local/lib/python3.12/site-packages/hpilo.py", line 278, in _detect_protocol
header, data = self._communicate(b'', ILO_HTTP, save=False)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/hpilo.py", line 406, in _communicate
sock = self._get_socket()
^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/hpilo.py", line 401, in _get_socket
return ssl.wrap_socket(sock, ssl_version=ssl.PROTOCOL_TLS)
^^^^^^^^^^^^^^^
AttributeError: module 'ssl' has no attribute 'wrap_socket'`

Studmuffin1134 · 2024-02-06T03:37:00Z

so what will be done? Valera Nordstrom

…

On Mon, Feb 5, 2024 at 9:35 PM Christopher Michael Shaw < ***@***.***> wrote: Can confirm ilo integration is broken in 2024.2b8 -- likely due to the Python upgrade. This may be applicable: eventlet/eventlet#795 <eventlet/eventlet#795> `Logger: homeassistant.components.sensor Source: helpers/entity_platform.py:581 Integration: Sensor (documentation <https://rc.home-assistant.io/integrations/sensor>, issues <https://github.com/home-assistant/core/issues?q=is%3Aissue+is%3Aopen+label%3A%22integration%3A+sensor%22> ) First occurred: 19:21:37 (13 occurrences) Last logged: 19:21:48 hp_ilo: Error on device update! Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 581, in _async_add_entity await entity.async_device_update(warning=False) File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1261, in async_device_update await hass.async_add_executor_job(self.update) File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run result = self.fn(*self.args, **self.kwargs) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/src/homeassistant/homeassistant/components/hp_ilo/sensor.py", line 166, in update ilo_data = getattr(self.hp_ilo_data.data, self._ilo_function)() ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/site-packages/hpilo.py", line 1035, in get_embedded_health return self._info_tag('SERVER_INFO', 'GET_EMBEDDED_HEALTH', 'GET_EMBEDDED_HEALTH_DATA', ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/site-packages/hpilo.py", line 730, in _info_tag header, message = self._request(root) ^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/site-packages/hpilo.py", line 238, in _request self._detect_protocol() File "/usr/local/lib/python3.12/site-packages/hpilo.py", line 278, in _detect_protocol header, data = self._communicate(b'', ILO_HTTP, save=False) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/site-packages/hpilo.py", line 406, in _communicate sock = self._get_socket() ^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/site-packages/hpilo.py", line 401, in _get_socket return ssl.wrap_socket(sock, ssl_version=ssl.PROTOCOL_TLS) ^^^^^^^^^^^^^^^ AttributeError: module 'ssl' has no attribute 'wrap_socket'` — Reply to this email directly, view it on GitHub <#109581 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AWLTQPCFO2FXYT2KCBILPCLYSGQH3AVCNFSM6AAAAABCYPOP22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRYG4ZDIOJWGI> . You are receiving this because you authored the thread.Message ID: ***@***.***>

christophermichaelshaw · 2024-02-06T03:40:40Z

Give the devs some time to update the integration -- appears as if the ssl.wrap_socket() function, deprecated in Python 3.7, needs to be replaced with context = ssl.SSLContext(ssl.PROTOCOL_TLS_SERVER) or something along those lines.

phidauex · 2024-02-08T02:17:01Z

More context here from the Python documentation: https://docs.python.org/3.12/whatsnew/3.12.html#ssl

Remove the ssl.wrap_socket() function, deprecated in Python 3.7: instead, create a ssl.SSLContext object and call its ssl.SSLContext.wrap_socket method. Any package that still uses ssl.wrap_socket() is broken and insecure. The function neither sends a SNI TLS extension nor validates server hostname. Code is subject to CWE-295: Improper Certificate Validation. (Contributed by Victor Stinner in gh-94199.)

gdiciancia · 2024-02-08T08:04:01Z

I see the same error on my instance. Went from 2023.x to 2024.2.0

christophermichaelshaw · 2024-02-08T08:19:13Z

I see the same error on my instance. Went from 2023.x to 2024.2.0

Thank you for the report and confirmation!

For further reporters of similar issues -- I don't see a need for any additional "me too" reports at this point, as we've identified the issue (deprecated Python function).

I haven't yet had time to submit a fix, as I'm unfamiliar with the codebase, but will likely submit a PR with a fix by the end of this weekend if need be. The devs are awesome, so I would predict the fix will be released very soon!

Redblockmasteur · 2024-02-09T12:18:23Z

Same Here I updated to 2024.2 and all my entities related to HI ILO disappear afterwards

tranzistori · 2024-02-10T10:41:32Z

me 3

daemongloom · 2024-02-10T10:47:55Z

2024.2.1 - iLo also doesn't work.

pergolafabio · 2024-02-10T12:34:38Z

Please guys, try to read!

The issue with the deprecated and now removed ssl.wrap_socket function was fixed years ago in the library

Nirostar · 2024-02-12T11:51:43Z

I created a PR that should fix the problem. In the meantime, if you want a temporary fix until the next update, you can run

docker exec -it homeassistant bash -c "wget https://raw.githubusercontent.com/seveas/python-hpilo/5881ffab4fba6ea136176a92de58c710704e9653/hpilo.py -O /usr/local/lib/python3.12/site-packages/hpilo.py"

Be aware: This won't carry over after updates.

If you want to do that in HAOS you need the https://github.com/hassio-addons/addon-ssh addon (this is the non-restricted community version) and disable the Safe mode.

Nirostar · 2024-02-12T11:53:38Z

I see the same error on my instance. Went from 2023.x to 2024.2.0

Thank you for the report and confirmation!

For further reporters of similar issues -- I don't see a need for any additional "me too" reports at this point, as we've identified the issue (deprecated Python function).

I haven't yet had time to submit a fix, as I'm unfamiliar with the codebase, but will likely submit a PR with a fix by the end of this weekend if need be. The devs are awesome, so I would predict the fix will be released very soon!

I already created a PR. Would you maybe review it?

Studmuffin1134 · 2024-02-12T16:27:59Z

i will try it later

globus243 · 2024-02-12T20:58:13Z

I see, I have chosen a bad day to start using the hp_ilo integration :D

I created a PR that should fix the problem. In the meantime, if you want a temporary fix until the next update, you can run
docker exec -it homeassistant bash -c "wget https://raw.githubusercontent.com/seveas/python-hpilo/5881ffab4fba6ea136176a92de58c710704e9653/hpilo.py -O /usr/local/lib/python3.12/site-packages/hpilo.py"
Be aware: This won't carry over after updates.

If you want to do that in HAOS you need the https://github.com/hassio-addons/addon-ssh addon (this is the non-restricted community version) and disable the Safe mode.

I downloaded your version as you described and tested it in advanced terminal where it worked, but HA still does not create the entities no matter how often and thoroughly I restart. could there maybe more broken?

also, I don't know if it was something with my python installation but I needed to change the \ to \\ on line 204, 336 and 432, otherwise it refused to run as well.

docker exec -it homeassistant bash
cd /usr/local/lib/python3.12/site-packages/
python

and then

hpilo.Ilo("xx.xx.0.200", login="a_username", password="a_password", timeout=60, port=443, protocol=None, delayed=False, ssl_verify=False, ssl_context=None)
ilo.get_power_readings

works! but:

sensor:
  - platform: hp_ilo
    host: xx.xx.0.200
    username: a_username
    password: a_password
    monitored_variables:
      - name: Server Power Wattage
        sensor_type: server_power_readings
        unit_of_measurement: Watt
        value_template: '{{ ilo_data.present_power_reading }}'

not

Nirostar · 2024-02-13T08:28:42Z

but:

sensor:
  - platform: hp_ilo
    host: xx.xx.0.200
    username: a_username
    password: a_password
    monitored_variables:
      - name: Server Power Wattage
        sensor_type: server_power_readings
        unit_of_measurement: Watt
        value_template: '{{ ilo_data.present_power_reading }}'

not

I think you forgot a [0].

      - name: Server Power Wattage
        sensor_type: server_power_readings
        value_template: "{{ ilo_data.present_power_reading[0] }}"
        unit_of_measurement: "W"

Nirostar · 2024-02-13T11:03:13Z

also, I don't know if it was something with my python installation but I needed to change the \ to \\ on line 204, 336 and 432, otherwise it refused to run as well.

This is only a syntax warning afaik. It's not nice, but in the library for ages it seems. It should work fine with the warnings.

RobertWojtowicz · 2024-02-16T19:16:31Z

Hi,
Does not work on Core version 2024.2.2

I tried the commands (HAOS 11.5), what am I doing wrong ?

[core-ssh ~]$ docker exec -it homeassistant bash -c "wget https://raw.githubusercontent.com/seveas/python-hpilo/5881ffab4fba6ea136176a92de58c710704e9653/hpilo.py -O /usr/local/lib/python3.12/site-packages/hpilo.py"
-bash: docker: command not found
[core-ssh ~]$ ha docker exec -it homeassistant bash -c "wget https://raw.githubusercontent.com/seveas/python-hpilo/5881ffab4fba6ea136176a92de58c710704e9653/hpilo.py -O /usr/local/lib/python3.12/site-packages/hpilo.py"
Error: unknown shorthand flag: 'i' in -it

christophermichaelshaw · 2024-02-16T19:31:52Z

You could likely just strip the docker exec wrapper and use the curl command, as it appears your pwd is already in the correct container. But to use the command as is (works for me), you’d need to use the Advanced SSH & Web Terminal Add-On with Protection mode disabled. Then just paste it into the terminal.

…

-- Christopher Shaw ***@***.*** 425.435.8440

On Fri, Feb 16, 2024 at 11:16 Robert Wójtowicz ***@***.***> wrote: Hi, Does not work on Core version 2024.2.2 I tried the commands (HAOS), what am I doing wrong ? [core-ssh ~]$ docker exec -it homeassistant bash -c "wget https://raw.githubusercontent.com/seveas/python-hpilo/5881ffab4fba6ea136176a92de58c710704e9653/hpilo.py -O /usr/local/lib/python3.12/site-packages/hpilo.py" -bash: docker: command not found [core-ssh ~]$ ha docker exec -it homeassistant bash -c "wget https://raw.githubusercontent.com/seveas/python-hpilo/5881ffab4fba6ea136176a92de58c710704e9653/hpilo.py -O /usr/local/lib/python3.12/site-packages/hpilo.py" Error: unknown shorthand flag: 'i' in -it — Reply to this email directly, view it on GitHub <#109581 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/APSZUKH4CCJI5C5ES45PWJDYT6WB5AVCNFSM6AAAAABCYPOP22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBZGE3DMMBVGM> . You are receiving this because you commented.Message ID: ***@***.***>

Nornode · 2024-02-16T19:39:01Z

@emontnemery - looks like you are owning this integration here: https://github.com/home-assistant/core/tree/dev/homeassistant/components/hp_ilo
Could you please help us have a look if this can be merged?

RobertWojtowicz · 2024-02-16T19:56:26Z

@christophermichaelshaw it works!, I used wrong ssh plugin.

Thx

tjmjansen · 2024-02-23T09:59:36Z

@emontnemery - looks like you are owning this integration here: https://github.com/home-assistant/core/tree/dev/homeassistant/components/hp_ilo Could you please help us have a look if this can be merged?

or @epenet ?

pug306d · 2024-02-23T16:52:47Z

docker exec -it homeassistant bash -c "wget https://raw.githubusercontent.com/seveas/python-hpilo/5881ffab4fba6ea136176a92de58c710704e9653/hpilo.py -O /usr/local/lib/python3.12/site-packages/hpilo.py"

This fixed it for me on 2024.2.2

* Update hpilo version in requirements_all.txt (#109581) The issue with the deprecated and now removed ssl.wrap_socket function was fixed years ago in the library * Update hpilo version in manifest.json (#109581)

adamllryan · 2024-02-27T14:53:52Z

I am using HAOS Core and don't have a python 3.12 folder, how would I go about fixing this? I'm reading addons run within docker instances but I can't seem to find it.

Nirostar · 2024-02-28T14:28:54Z

I am using HAOS Core and don't have a python 3.12 folder, how would I go about fixing this? I'm reading addons run within docker instances but I can't seem to find it.

I created a PR that should fix the problem. In the meantime, if you want a temporary fix until the next update, you can run
docker exec -it homeassistant bash -c "wget https://raw.githubusercontent.com/seveas/python-hpilo/5881ffab4fba6ea136176a92de58c710704e9653/hpilo.py -O /usr/local/lib/python3.12/site-packages/hpilo.py"
Be aware: This won't carry over after updates.

If you want to do that in HAOS you need the https://github.com/hassio-addons/addon-ssh addon (this is the non-restricted community version) and disable the Safe mode.

Please read the comment to the end. You need this addon (and really the non-restricted one) and execute it in that then.

home-assistant bot added the integration: hp_ilo label Feb 4, 2024

Nirostar added a commit to Nirostar/homeassistant-core that referenced this issue Feb 12, 2024

Update hpilo version in requirements_all.txt (home-assistant#109581)

34fc2a5

The issue with the deprecated and now removed ssl.wrap_socket function was fixed years ago in the library

Nirostar added a commit to Nirostar/homeassistant-core that referenced this issue Feb 12, 2024

Update hpilo version in manifest.json (home-assistant#109581)

a0d9c66

Nirostar mentioned this issue Feb 12, 2024

Bump python-hpilo to 4.4.3 #110325

Merged

20 tasks

gjohansson-ST closed this as completed in #110325 Feb 23, 2024

pinkywafer mentioned this issue Feb 28, 2024

HP iLo inegration fails to update #111668

Closed

github-actions bot locked and limited conversation to collaborators Mar 29, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

HP ILO integration not working in most recent update #109581

HP ILO integration not working in most recent update #109581

Studmuffin1134 commented Feb 4, 2024

home-assistant bot commented Feb 4, 2024

christophermichaelshaw commented Feb 6, 2024

Studmuffin1134 commented Feb 6, 2024 via email

christophermichaelshaw commented Feb 6, 2024

phidauex commented Feb 8, 2024

gdiciancia commented Feb 8, 2024

christophermichaelshaw commented Feb 8, 2024

Redblockmasteur commented Feb 9, 2024

tranzistori commented Feb 10, 2024

daemongloom commented Feb 10, 2024

pergolafabio commented Feb 10, 2024

Nirostar commented Feb 12, 2024 •

edited

Loading

Nirostar commented Feb 12, 2024

Studmuffin1134 commented Feb 12, 2024

globus243 commented Feb 12, 2024 •

edited

Loading

Nirostar commented Feb 13, 2024 •

edited

Loading

Nirostar commented Feb 13, 2024

RobertWojtowicz commented Feb 16, 2024 •

edited

Loading

christophermichaelshaw commented Feb 16, 2024 via email

Nornode commented Feb 16, 2024

RobertWojtowicz commented Feb 16, 2024

tjmjansen commented Feb 23, 2024

pug306d commented Feb 23, 2024

adamllryan commented Feb 27, 2024 •

edited

Loading

Nirostar commented Feb 28, 2024

HP ILO integration not working in most recent update #109581

HP ILO integration not working in most recent update #109581

Comments

Studmuffin1134 commented Feb 4, 2024

The problem

What version of Home Assistant Core has the issue?

What was the last working version of Home Assistant Core?

What type of installation are you running?

Integration causing the issue

Link to integration documentation on our website

Diagnostics information

Example YAML snippet

Anything in the logs that might be useful for us?

Additional information

home-assistant bot commented Feb 4, 2024

christophermichaelshaw commented Feb 6, 2024

Studmuffin1134 commented Feb 6, 2024 via email

christophermichaelshaw commented Feb 6, 2024

phidauex commented Feb 8, 2024

gdiciancia commented Feb 8, 2024

christophermichaelshaw commented Feb 8, 2024

Redblockmasteur commented Feb 9, 2024

tranzistori commented Feb 10, 2024

daemongloom commented Feb 10, 2024

pergolafabio commented Feb 10, 2024

Nirostar commented Feb 12, 2024 • edited Loading

Nirostar commented Feb 12, 2024

Studmuffin1134 commented Feb 12, 2024

globus243 commented Feb 12, 2024 • edited Loading

Nirostar commented Feb 13, 2024 • edited Loading

Nirostar commented Feb 13, 2024

RobertWojtowicz commented Feb 16, 2024 • edited Loading

christophermichaelshaw commented Feb 16, 2024 via email

Nornode commented Feb 16, 2024

RobertWojtowicz commented Feb 16, 2024

tjmjansen commented Feb 23, 2024

pug306d commented Feb 23, 2024

adamllryan commented Feb 27, 2024 • edited Loading

Nirostar commented Feb 28, 2024

Nirostar commented Feb 12, 2024 •

edited

Loading

globus243 commented Feb 12, 2024 •

edited

Loading

Nirostar commented Feb 13, 2024 •

edited

Loading

RobertWojtowicz commented Feb 16, 2024 •

edited

Loading

adamllryan commented Feb 27, 2024 •

edited

Loading