Help with Luup Reloads
-
One thing to add here, a possibility: I notice that the plugin is using the 'ip' attribute to store the IP address of the controller. I have found that this causes a lot of instability. Luup has certain automatic things that it does in the background to try to open connections when the IP is stored on that attribute. It's all part of their built-in automatic I/O for serial and IP connections, which this plugin doesn't use. But Luup will try, try and retry forever to start and maintain a connection to that IP. I believe that this is at least one source of a memory leak -- I suspect their error handling on connection failure is not completely cleaning up after itself, so the system leaks memory, and/or possible even sockets. I discovered this some years ago and switched all of my plugins to just use state variables for the IP. My systems do not restart due to memory leaks, and memory usage is generally flat.
-
@toggledbits said in Help with Luup Reloads:
But Luup will try, try and retry forever to start and maintain a connection to that IP.
Yes, this is, indeed, an issue, although NOT one, I would say, for openLuup. It may well explain why Vera behaves (sort of) when you remove some plugins.
For some time, now, I had a Vera Edge running without any plugins or Z-wave devices, and, I have to say, it just runs and runs!
-
@toggledbits yep, I will change it in next build. I'm onto it as we speak. On my system I have few plug-ins, no one is using an IP and I'm still experiencing memory leaks. But I will probably just move everything to openLuup and use the Vera as a radio system.
-
@akbooer said in Help with Luup Reloads:
Yes, this is, indeed, an issue, although NOT one, I would say, for openLuup.
Sorry, did I miss that this was openLuup? Or are you just confirming that openLuup is better than real Luup (which everyone already knows)?
-
toggledbitsreplied to therealdb on Jun 22, 2021, 3:22 PM last edited by toggledbits Jun 22, 2021, 11:26 AM
@therealdb said in Help with Luup Reloads:
yep, I will change it in next build. I'm onto it as we speak.
One more thing... some versions of Luup have trouble returning the status from
os.execute()
. They said they have fixed it in the newer beta builds (5363/4/5), but it's concerning that it could be broken at all (should be straightforward, not unstable), and Edward's response to me when queried was a bit uncertain. So I've started moving away from using the return value under Luup and started doing error detection other ways (e.g. does the file I expect now exist?). -
@toggledbits said in Help with Luup Reloads:
Sorry, did I miss that this was openLuup?
Oooh, hope not... NO, just checked...
@pabla said in Help with Luup Reloads:
Recently I have been getting constant reloads on my Vera Plus
@toggledbits said in Help with Luup Reloads:
are you just confirming that openLuup is better than real Luup
Er, yes, just that, really. Let's say that this is one feature I chose not to emulate.
-
-
@therealdb No rush, just reporting back
-
@pabla 1.51-hotfix is on GitHub.
- programs duplication is now fixed, so a reconfiguration is OK
- IP is now a variable (and it's automatically migrated - don't worry)
- temp files are correctly deleted
- I've tracked the problem with infinite stations - and I'm working on a permanent fix. this seems to be related to reloads and I'll investigate this further in the next days.
Give it a try!
-
therealdbreplied to toggledbits on Jun 22, 2021, 6:12 PM last edited by therealdb Jun 22, 2021, 5:32 PM
@toggledbits I found that AltHue by @amg0 is still using IP attr. I'll probably send him a pull request, since it's very popular.
-
@therealdb So I updated to the latest hot fix, and the files still aren't getting deleted. I have a fill from 11:06am local time and its 11:27am. Do they auto delete after a longer time?
-
-
@therealdb Manually deleted the files and they aren't auto deleting still. This is really weird
-
Okay good news, so I uploaded the file via WinSCP since I noticed I had doubles of each file type one of the doubles was a .lzo file type. Now the files are deleting properly too. Question for everyone: Does the .lzo compression make a difference? Every time I get Github files I decompress and then drag them into the Luup Files section under Develop Apps. But I have began to notice that I have a lot of files with the .lzo extension
-
toggledbitswrote on Jun 22, 2021, 11:46 PM last edited by toggledbits Jun 22, 2021, 7:48 PM
When you drag them to the Upload button in Apps > Develop apps > Luup files, the uploader will LZO compress them, always. If you use WinSCP and copy uncompressed files, they stay uncompressed. When Luup looks for a file, it will select the uncompressed file before the compressed if both are present, and this is how things get goofed up -- if you have old uncompressed files and try to update via the uploader, Luup still sees/uses the uncompressed files and ignores the newer compressed files (it doesn't check timestamps -- that would be nice!). Personally, I always use the uploader, and I don't keep uncompressed plugin files on my Veras, even during development (I never edit on the Vera so it doesn't matter).
-
Pablareplied to toggledbits on Jun 22, 2021, 11:48 PM last edited by Pabla Jun 22, 2021, 7:52 PM
@toggledbits That makes sense why the plugin wasnt working as expected. That explanation solves so many issues that I have had with plugins lol. Since I would alternate between WinSCP and Uploading via the web UI
-
@toggledbits blimey that's poor. Don't know why I should be surprised, though!
C
16/33