Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warning in the RM-Logs every time i use "Integration neu laden" or "Homeassistant neu starten" #340

Closed
SukramJ opened this issue Mar 18, 2022 · 7 comments

Comments

@SukramJ
Copy link
Collaborator

SukramJ commented Mar 18, 2022

Since changes for this issue i got a warning in the RM-Logs every time i use "Integration neu laden" or "Homeassistant neu starten"
It's just a warning, but was not seen before and should not be there.
Don't know if it's maybe problematic.

Mar 18 15:17:37 RM-Test-Pi3Bplus-27 local0.warn ReGaHss: WARNING: cannot clear session with id '1bWXOdzQux' [ExecSystem():iseESPexec.cpp:3764]
Mar 18 15:17:49 RM-Test-Pi3Bplus-27 user.err rfd: XmlRpc fault calling system.listMethods({"RM-Test-Pi3Bplus-27-BidCos-RF"}) on http://192.168.113.29:43959/RPC2:[faultCode:1,faultString:"<class 'TypeError'>:system_listMethods() takes 1 positional argument but 2 were given"]
Mar 18 17:51:19 RM-Test-Pi3Bplus-27 local0.warn ReGaHss: WARNING: cannot clear session with id '1FObNXZ2rw' [ExecSystem():iseESPexec.cpp:3764]
Mar 18 17:51:38 RM-Test-Pi3Bplus-27 user.err rfd: XmlRpc fault calling system.listMethods({"RM-Test-Pi3Bplus-27-BidCos-RF"}) on http://192.168.113.29:36839/RPC2:[faultCode:1,faultString:"<class 'TypeError'>:system_listMethods() takes 1 positional argument but 2 were given"]
Mar 18 17:56:05 RM-Test-Pi3Bplus-27 local0.warn ReGaHss: WARNING: cannot clear session with id 'nD1aRrKtFb' [ExecSystem():iseESPexec.cpp:3764]
Mar 18 17:56:17 RM-Test-Pi3Bplus-27 user.err rfd: XmlRpc fault calling system.listMethods({"RM-Test-Pi3Bplus-27-BidCos-RF"}) on http://192.168.113.29:32829/RPC2:[faultCode:1,faultString:"<class 'TypeError'>:system_listMethods() takes 1 positional argument but 2 were given"] 

Originally posted by @Baxxy13 in #328 (comment)

@SukramJ
Copy link
Collaborator Author

SukramJ commented Mar 18, 2022

  1. XmlRpc fault calling system.listMethods is very old, and has already been a warning with the old integration.
    I think this is related to the Python XmlRPC-Server implementation. Here i have currently no idea.

  2. ReGaHss: WARNING: cannot clear session with id '1bWXOdzQux' seems to be new and I guess 1bWXOdzQux is the JSON Session ID. Here i can add a Session.logout before shutdown.

@SukramJ
Copy link
Collaborator Author

SukramJ commented Mar 18, 2022

0.37.4 should fix ReGaHss: WARNING: cannot clear session

@SukramJ
Copy link
Collaborator Author

SukramJ commented Mar 18, 2022

@Baxxy13 please retest

@SukramJ SukramJ changed the title Since changes for this issue i got a warning in the RM-Logs every time i use "Integration neu laden" or "Homeassistant neu starten" Warning in the RM-Logs every time i use "Integration neu laden" or "Homeassistant neu starten" Mar 18, 2022
@SukramJ
Copy link
Collaborator Author

SukramJ commented Mar 18, 2022

0.37.5 should fix XmlRpc fault calling system.listMethods

@Baxxy13
Copy link
Contributor

Baxxy13 commented Mar 18, 2022

zu 1.
XmlRpc fault calling system.listMethods taucht nun gar nicht mehr auf. Sehr gut.

zu 2.
ReGaHss: WARNING: cannot clear session with id '1bWXOdzQux' sehe ich noch ab und zu, aber nicht mehr bei jedem Neustart von HA oder Neu Laden der Integration.
Habe aber noch kein Muster erkannt, gucke ich mir morgen nochmal in Ruhe an.

@Baxxy13
Copy link
Contributor

Baxxy13 commented Mar 19, 2022

zu 2.
Die Warnung scheint immer dann noch zu kommen wenn ich beide Optionen (HA Neu Starten / Integration Neu laden) in einer bestimmten Reihenfolge nutze.

Also:
"Integration Neu laden" --> Warnung erscheint nicht
dann...
"HA Neu Starten" --> Warnung erscheint

Jeder weiter Neustart / Reload bringt keine Warnung, erst wenn die obige Reihenfolge eingehalten wird gibt es wieder eine.
Merkwürdig.

Da man das i.d.R. nicht so exzessiv nutzt wie ich hier beim testen und man normalerweise nur den HA nach Update der Integration neu startet kann man das ignorieren... würde ich sagen.

@SukramJ
Copy link
Collaborator Author

SukramJ commented Mar 20, 2022

Passt

@SukramJ SukramJ closed this as completed Mar 20, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Feb 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants