Core: SMA stops reporting after 4 days

[johansmitsnl]

Home Assistant release with the issue: 0.99.2

Operating environment (Hass.io/Docker/Windows/etc.): Docker

Integration: SMA

Description of problem: stops reporting after +/- 4 days

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

  - platform: sma
    host: 10.10.10.10
    ssl: true
    group: user
    verify_ssl: false
    password: !secret sma_password
    sensors:
      - grid_power
      - pv_power
      - total_yield
      - pv_gen_meter
      - voltage_l1
      - voltage_l2
      - voltage_l3
      - frequency

Additional information:

After +/- 4 days the information stops. I need to restart homeassistant to get it working again.

[probot-home-assistant[bot]]

Hey there @kellerza, mind taking a look at this issue as its been labeled with a integration (sma) you are listed as a codeowner for? Thanks!

[johansmitsnl]

Probable the same issue kellerza/pysma#25

[johansmitsnl]

It can be reproduced when the connection to the SMA is lost and then recovers. Is the session torn down correctly or does it needs to wait a little longer?

If you can suggest a change I can test it now.

[johansmitsnl]

A fix is available in a open PR: kellerza/pysma#34 waiting it to be accepted and released.

[roybosch]

I'm experiencing the same behavior. I'm also thinking that it could be related to a connection loss. If the SMA integration could try to establish a new connection every few minutes, a full hassio restart won't be necessary.

[Hooch76]

@roybosch A fix is on it's way, see https://github.com/home-assistant/home-assistant/issues/27154#issuecomment-568895001.
(I've already added the change manually just yesterday and it seems to work. So it's just a matter of time before the PR gets accepted)

[roybosch]

@roybosch A fix is on it's way, see #27154 (comment).
(I've already added the change manually just yesterday and it seems to work. So it's just a matter of time before the PR gets accepted)

Great, I'm looking forward to it! :D

[Human]

FYI, my PR was accepted to the pysma project.

[roybosch]

Does that mean it will be part of the next hass.io release?

[johansmitsnl]

I just asked @kellerza when we can expect a release of the patched version and PR in HA.