[PLUGIN] Disk Location


Recommended Posts

44 minutes ago, aerobrain said:

Is anyone else having issues with the new version causing issues with their dashboard tab?

 

Just updated the plugin on both my servers and then my Dashboard is totally blank and clicking on that tab sometimes make the GUI hang. Removed the plug in on my second server and it returned to normal operation.

 

If there's any specific logs you may need to check just let me know (ideally with how to get those logs as it's not something I've done before).

 

Ta


Yes. Dashboard appeared frozen but just took a very long time to load.

Edited by Niklas
Link to comment

Like many my dashboard is not loading with the newest update. Other tabs will load (after a long time). If I try to go into settings for this plugin. My UI goes back to snail pace. 

 

I am new to unraid. Can anyone tell me how I can roll back to the previous version? 

Edited by BartyB
Link to comment
18 minutes ago, cobalt027 said:

I'm having the same behavior. I didn't uninstall yet. Hoping an update fixes things.

I uninstalled since it made the dashboard unusable. When I did it said the settings were not deleted so I am hoping once it's fixed and I reinstall everything goes back to the way it was. 

Link to comment

Same issue here. Either empty dashboard (blank page, or only header and top menu) or other pages taking like 2 minutes to load and become unresponsive. Tried deleting `/tmp/disklocation/db.lock` but it didn't help.

Removing the plugin resolved it.

Reinstalling brought the same issue again.

Will have to wait for the update.

 

P.S. I installed it on again on test server with nothing in it and got the same behaviour. Diagnostics attached.

discovery-diagnostics-20240306-1036.zip

Edited by brankko
Diagnostics added
  • Thanks 1
Link to comment

Ouch, that was painful. Got to the point I could not load anything from the UI unless I knew what the exact URL was. I saw in the release notes that the multi minute delay was possible. IMHO, if there would be that much of an impact, it should not have been released that way. This was very disruptive.

 

I've removed the plugin for now until it's stable again.

Edited by snowboardjoe
Link to comment

I'm here for the same reason. I did find an error that it's throwing in PHP.

 

[06-Mar-2024 06:27:07 America/New_York] PHP Fatal error:  Uncaught TypeError: array_column(): Argument #1 ($array) must be of type array, null given in /usr/local/emhttp/plugins/disklocation/pages/functions.php:600
Stack trace:
#0 /usr/local/emhttp/plugins/disklocation/pages/functions.php(600): array_column(NULL, 'serial')
#1 /usr/local/emhttp/plugins/disklocation/pages/functions.php(613): zfs_node('000000000000010...', NULL)
#2 /usr/local/emhttp/plugins/disklocation/pages/devices.php(536): zfs_disk('000000000000010...', NULL, NULL, 1)
#3 /usr/local/emhttp/plugins/dynamix/include/DefaultPageLayout.php(715) : eval()'d code(25): require_once('/usr/local/emht...')
#4 /usr/local/emhttp/plugins/dynamix/include/DefaultPageLayout.php(715): eval()
#5 /usr/local/emhttp/plugins/dynamix/template.php(82): require_once('/usr/local/emht...')
#6 {main}
  thrown in /usr/local/emhttp/plugins/disklocation/pages/functions.php on line 600

 

  • Thanks 1
Link to comment

Yeap, bye bye to this plugin, not deaing with this.

 

Sometimes old school wins, just gonna write down the locations of the disk using acient tools know as paper and pen.

Edited by thedinz
  • Haha 1
Link to comment

Update 2024.03.06c <- make sure it says "c" and NOT "b" before updating.

  • Commit #294 - BUG: Check for ZFS before executing functions based on it. (reported by @malaki86)
  • Commit #293 - BUG: Deactivated the database lock as it seem to have caused some serious issues. #290 will be incomplete again.

Please check if this helps the issue, I'm not home at the moment and can't test it myself. Sorry for the inconvenience etc. and so on :P  

 

Thanks for people who contribute and provide with proper information, wonderful to see people are using the plugin :) 

To some others, I would just recommend to read the license, especially §15-§17 😘 https://github.com/olehj/disklocation/blob/master/COPYING

  • Like 3
Link to comment
5 hours ago, Zonediver said:

Where is the developer?

Gone with the wind? 🤣

Sorry, but I do have a life besides writing Unraid plugins ;) I'm now correcting bugs on my laptop in a hotel directly on GitHub on work travel after work.

  • Haha 1
  • Upvote 1
Link to comment
2 hours ago, FlamongOle said:

Sorry, but I do have a life besides writing Unraid plugins ;) I'm now correcting bugs on my laptop in a hotel directly on GitHub on work travel after work.

...this was just a "little" joke 😉

I Know, you take care of bugs 👍👍👍

Link to comment
2 hours ago, FlamongOle said:

Update 2024.03.06c <- make sure it says "c" and NOT "b" before updating.

Im using CA Auto Update, not sure how one can avoid these things, unless using a delay :D

Thank you for the plug-in, and thanks for fixing it :D

Link to comment
6 hours ago, FlamongOle said:

Update 2024.03.06c <- make sure it says "c" and NOT "b" before updating.

  • Commit #294 - BUG: Check for ZFS before executing functions based on it. (reported by @malaki86)
  • Commit #293 - BUG: Deactivated the database lock as it seem to have caused some serious issues. #290 will be incomplete again.

Please check if this helps the issue, I'm not home at the moment and can't test it myself. Sorry for the inconvenience etc. and so on :P  

 

Thanks for people who contribute and provide with proper information, wonderful to see people are using the plugin :) 

To some others, I would just recommend to read the license, especially §15-§17 😘 https://github.com/olehj/disklocation/blob/master/COPYING

 

All fixed now, Dashboard is loading just fine! Loading from Tools just fine as well. Thank you again for all your efforts!

Link to comment

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.