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

[iRobot] Make thing ID persistent #10116

Closed
krocans opened this issue Feb 10, 2021 · 3 comments · Fixed by #10860
Closed

[iRobot] Make thing ID persistent #10116

krocans opened this issue Feb 10, 2021 · 3 comments · Fixed by #10860
Labels
enhancement An enhancement or new feature for an existing add-on

Comments

@krocans
Copy link

krocans commented Feb 10, 2021

Currently IP address of iRobot is used for thing ID like irobot:roomba:192_168_57_17.
If robot gets new IP that makes configured thing to stop working and new thing with new IP is being discovered.

Please make Roomba's ID persistent. I suggest to use Roomba's blid as thing ID - it never changes.

@krocans krocans added the enhancement An enhancement or new feature for an existing add-on label Feb 10, 2021
@J-N-K
Copy link
Member

J-N-K commented Feb 12, 2021

Your issue is two-fold.

I agree that using the IP as representation property is a bad idea and should be fixed.

However, it is always necessary to know the IP address. So you should either make the IP static or use a DNS name that is properly updated if the IP changes.

@krocans
Copy link
Author

krocans commented Feb 13, 2021

@J-N-K thanks for fixing it!

I am using statically configured things in .things file and DNS updates so for me problem was only bunch of bogus things in "inbox".
The situation is following:
If statically configured thing id exactly matches discovered thing id - nothing new appears in inbox and everything works.
If the same discovered item changes ID all the time (on IP change) then my statically configured Roomba works fine of course but new bogus thing is being added to inbox again and again on every IP change.

If using GUI config only then it's different story, then changing id of the same thing is really bad.

@openhab-bot
Copy link
Collaborator

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

https://community.openhab.org/t/irobot-9xx-on-openhab/17774/258

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
3 participants