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

[vdr] getting error logs with active things #14747

Closed
udo1toni opened this issue Apr 4, 2023 · 13 comments
Closed

[vdr] getting error logs with active things #14747

udo1toni opened this issue Apr 4, 2023 · 13 comments
Labels
bug An unexpected problem or unintended behavior of an add-on

Comments

@udo1toni
Copy link
Contributor

udo1toni commented Apr 4, 2023

Current Behavior

When a vdr Thing is active, when loading the Items list results in an error message (one line for each active Thing)

[ERROR] [al.ChannelCommandDescriptionProvider] - Dynamic command description matches original command description. DynamicCommandDescriptionProvider implementations must never return the original command description. class org.openhab.core.internal.types.CommandDescriptionImpl has to be fixed.

For reference:
https://community.openhab.org/t/vdr-binding/10839/30

Your Environment

  • Version used: openHAB 3.4.2, addon from official store
  • Java 11-64 openjdk 11.0.18
  • debian bullseye 11.6 amd-64
@udo1toni udo1toni added the bug An unexpected problem or unintended behavior of an add-on label Apr 4, 2023
@openhab-bot
Copy link
Collaborator

This issue has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/channel-command-description-provider-gives-lot-of-errors/145806/4

@openhab-bot
Copy link
Collaborator

This issue has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/vdr-binding/10839/30

@udo1toni
Copy link
Contributor Author

Same error in

  • openHAB 4.0.1,
  • Java 17-64 openjdk 17.0.8 2023-07-18
  • debian bookworm 12.1 amd-64

@lolodomo
Copy link
Contributor

@MatthiasKlocke : could you please investigate ?

@MatthiasKlocke
Copy link
Contributor

Sure, I will have a look beginning of next week @udo1toni, @lolodomo.

@MatthiasKlocke
Copy link
Contributor

Hello @udo1toni ,
so far I wasn't lucky to reproduce the issue in the VDR binding. Would you be so kind enough to give me some more details?
You mentioned that this happens when you reload the items list. How do you do that? Putting text file configuration into the conf directory or does this happen when you open the item list in the openhab user interface? Thanks!

@udo1toni
Copy link
Contributor Author

udo1toni commented Aug 10, 2023

It's sufficient to click the update button in VS Code, even selecting Items in the Main UI will trigger the error.

@MatthiasKlocke
Copy link
Contributor

Hello @udo1toni ,

thanks for your reply. I'm sorry, but I still can't reproduce it...

I have defined VDR things in text files and also in MainUI. I added items to them with an assigned semantic class. I've gone through all pages but the error is not shown in log file.
I checked in openHAB github repo to see where your log message is being generated. It comes from a class that is not used by the VDR binding.

Can you set the VDR's binding log level to TRACE and also the following log level to debug?
-> KARAF console
log:set TRACE org.openhab.binding.vdr
log:set DEBUG org.openhab.core.thing.internal.ChannelCommandDescriptionProvider

Thanks!

@udo1toni
Copy link
Contributor Author

The log did not give any hints.
But: I deleted my things (created via text file) and recreated one of them via UI -> no Error message.
I then recreated the Things via text file but set all parameters (I did not set the port before because this was the default port anyway. Seems that did the trick, the error is gone since.

@udo1toni
Copy link
Contributor Author

udo1toni commented Aug 18, 2023

Sorry, have to reopen the Issue, I had a small typo in my things file, so the things were never populated...
After correction, the error is back... :(

For reference: my things file:

Thing vdr:vdr:vdrServer "vdr Server" [
    host="vdr-server.udo.lokal",
    port=6419,
    refresh=30
]
Thing vdr:vdr:vdrWozi "vdr Wohnzimmer" [
    host="vdr-wozi.udo.lokal",
    port=6419,
    refresh=30
]
Thing vdr:vdr:vdrSchlaf "vdr Schlafzimmer" [
    host="vdr-schlaf.udo.lokal",
    port=6419,
    refresh=30
]

@udo1toni udo1toni reopened this Aug 18, 2023
@udo1toni udo1toni reopened this Aug 18, 2023
@MatthiasKlocke
Copy link
Contributor

Hello @udo1toni , sorry to hear that it's still not working.

I can't reproduce... I now took your things file (which looks unspectacular) and tested it in a fresh 4.0.1 on debian bullseye (raspberry) with DNS spoofing for your local hostnames. Everything looked fine.

Can you do a similar test with a fresh openHAB 4.0.1 and share the openhab log file with me?

@udo1toni
Copy link
Contributor Author

yes, sure, will report

@udo1toni
Copy link
Contributor Author

Seems that redoing the complete configuration has done the trick. Lucky me I'm using text configuration wherever possible, so the only stuff which went do be done after copying all text files was to copy the yaml for widgets and main ui :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An unexpected problem or unintended behavior of an add-on
Projects
None yet
Development

No branches or pull requests

4 participants