Jump to content
olehj

[PLUGIN] Disk Location

382 posts in this topic Last Reply

Recommended Posts

3 minutes ago, radfx said:

It doesn’t show that it’s a cache drive but rather marks it as unused.
But that’ll do for now. Unraid sees them both as cache and the plugin sees them at least so I can rest easy.

Thank you for all your help and suggestions.

 

Yeah, because some things changes with the path, the plugin will have you set that back up again as the mapping has been lost (there was a discussion about this earlier in this same thread). Glad you found the problem and happy to help. Go get some sleep!

Share this post


Link to post

Should we worry about the "Warning: A non-numeric value encountered in /usr/local/emhttp/plugins/disklocation/pages/system.php on line 95" messages

Share this post


Link to post
Just now, GeekMajic said:

Should we worry about the "Warning: A non-numeric value encountered in /usr/local/emhttp/plugins/disklocation/pages/system.php on line 95" messages

 

I doubt he's getting that error anymore since he's able to retrieve good data from all of his drives again. That script should expect to see all of that data and just served as an indicator to a deeper problem that he fixed.

Share this post


Link to post
1 minute ago, GeekMajic said:

Should we worry about the "Warning: A non-numeric value encountered in /usr/local/emhttp/plugins/disklocation/pages/system.php on line 95" messages

line 95 makes way more sense than line 9. You do not need to worry about that, it is only a converter for the harddisk size to convert from pure bytes to eg. TB. It faults when it does not receive anything. I'll fix this later.

Share this post


Link to post

Joe,

 

Thanks for the rapid reply, but I just installed and am getting the error myself. As I am digging in, I'm finding some other problems like rotational drives shown as SSD.

 

Edit: The specific drives are all Toshiba X300 (Toshiba HDWE 150/160) that are incorrectly displayed as SSD

Edited by GeekMajic

Share this post


Link to post
13 minutes ago, GeekMajic said:

Joe,

 

Thanks for the rapid reply, but I just installed and am getting the error myself. As I am digging in, I'm finding some other problems like rotational drives shown as SSD.

 

Edit: The specific drives are all Toshiba X300 (Toshiba HDWE 150/160) that are incorrectly displayed as SSD

Which version of the plugin do you have? This issue should have been fixed some time ago. Yet the SSD is displayed when it find no rotation and it should also check for another input regarding solid state, it might not be 100% but then I can't do so much about it yet until smartmontools gets their info a bit better.

Share this post


Link to post

Update 2019.02.18b

  • Commit #53 - MINOR ISSUE: Found the actual line, checks for disksize value before calculating human readable size.

This should hopefully fix the warning message you got earlier

Share this post


Link to post
7 hours ago, olehj said:

Which version of the plugin do you have? This issue should have been fixed some time ago. Yet the SSD is displayed when it find no rotation and it should also check for another input regarding solid state, it might not be 100% but then I can't do so much about it yet until smartmontools gets their info a bit better.

I'm using 2019.02.18 version. Installed 9 hours, fresh.

Share this post


Link to post
2 hours ago, olehj said:

Update 2019.02.18b

  • Commit #53 - MINOR ISSUE: Found the actual line, checks for disksize value before calculating human readable size.

This should hopefully fix the warning message you got earlier

I updated to 2019.02.19b.

 

Confirmed, the line #95 error is gone.

Share this post


Link to post

@radfx & @GeekMajic

What you can try to check if the SMART data actually get read, is to set a higher delay for "SMART execution delay"

 

The plugin setting is a delay to be sure it has closed enough SMART instances before running new ones. It is mainly a nasty hack to be sure PHP has closed the run before opening too many new ones simultaneously which smartmontools does not like. A busy system might need a higher value, try 500 and 1000 just for the fun of diagnostics. It will/might take longer time to load the page with higher values.

 

You can also try to run

time smartctl -X /dev/sgX

in the terminal to check how long time it takes to execute a SMART read.

 

NB! The time it takes to read, has no influence over the setting for the plugin itself other than real time.

 

If this does not help, either SMART is reporting false data or the drive gives weird values.

Share this post


Link to post
[mention=81427]radfx[/mention] & [mention=72425]GeekMajic[/mention]
What you can try to check if the SMART data actually get read, is to set a higher delay for "SMART execution delay"

If this does not help, either SMART is reporting false data or the drive gives weird values.



Thank you for all your help, my issue appeared to be a faulty SATA connection and/or cable along that appeared to cause corruption in the plugins Database
I had to delete the sql file from the USB drive and now it’s all working

It shows the cause drives as unused, but at this stage I don’t care, it was 2am when I finished with the server. It’s running, I’m happy :)

Thanks again for everything.

Share this post


Link to post

Using Disk Location 2019.02.18b.

 

Noticed that if the plugin does not pick up drive data ("Available disk slot") then you can not add (and display) anything in the comment for this tray. I would like to be able to add (and display) the comment for empty trays.

 

TIA...

Share this post


Link to post
12 hours ago, pkn said:

Using Disk Location 2019.02.18b.

 

Noticed that if the plugin does not pick up drive data ("Available disk slot") then you can not add (and display) anything in the comment for this tray. I would like to be able to add (and display) the comment for empty trays.

 

TIA...

Empty trays does not contain any data at all, it is simply one number that defines the empty tray slot and nothing can be added to it. That's why you can't add or view the comments. 

Share this post


Link to post
4 hours ago, olehj said:

Empty trays does not contain any data at all, it is simply one number that defines the empty tray slot and nothing can be added to it. That's why you can't add or view the comments. 

That's too bad... so my page looks like this (see below) where trays 12 and 16 are actually RAID pair configured as cache drive, and only tray 14 is really empty... it has broken SATA connector and there is no way to comment both trays 16 and 14...

image.png

Share this post


Link to post
6 hours ago, pkn said:

That's too bad... so my page looks like this (see below) where trays 12 and 16 are actually RAID pair configured as cache drive, and only tray 14 is really empty... it has broken SATA connector and there is no way to comment both trays 16 and 14...

 

I would highly suggest and recommend to get a new connector instead (or fix it if possible, maybe just slightly bad connection with the pins - swapping the cable might help as well).

I do not intend to fix something which is really not broken, and adding comments for empty trays might mess up the database and it will require a stupid amount of coding to just get that. Sorry I can't help you with this, at least not with this design.

Share this post


Link to post

"Warning: SQLite3::exec(): near "ON": syntax error in /usr/local/emhttp/plugins/disklocation/pages/system.php on line 651
near "ON": syntax error"

 

First use, version Disk Location 2019.02.18b, unraid Version 6.5.3 2018-06-12

Share this post


Link to post
10 hours ago, pkn said:

"Warning: SQLite3::exec(): near "ON": syntax error in /usr/local/emhttp/plugins/disklocation/pages/system.php on line 651
near "ON": syntax error"

 

First use, version Disk Location 2019.02.18b, unraid Version 6.5.3 2018-06-12

I looked into the unraid changelogs; The version which supports "UPSERT" came in SQLite version 3.24.0. This version got included in Unraid 6.6.0.

I guess then I have to conclude that this plugin is compatible with Unraid 6.6+, I weren't aware of this as I normally don't deal with SQLite but MySQL/MariaDB.

Share this post


Link to post

The installer script is now updated. No other additional changes has been made, so an update won't bring anything new. 

New requirements are Unraid 6.6.0 and above, front page is updated.

If you have earlier versions, the development package needs to be installed instead. This will -not- fix the SQLite syntax error issues, but it will be possible to install the script. Check the first page for more info on howto swap between stable and development branch.

Share this post


Link to post

2 quick questions...

 

I have an unassigned device that is a USB attached drive. Any way to get that to show up?

Also, is it possible to show the unRaid assigned 3 digit code (don't know that that is technically called) in the configuration tab? I can more easily identify the drives that way.

 

... and a wish item.

Ability to change orientation of each drive individually. I have 6 vertical bays and 4 horizontal in my case. Would be great to allow them to be displayed properly.


Thanks! Great plugin!! 

Share this post


Link to post
3 minutes ago, jpotrz said:

2 quick questions...

 

I have an unassigned device that is a USB attached drive. Any way to get that to show up?

Also, is it possible to show the unRaid assigned 3 digit code (don't know that that is technically called) in the configuration tab? I can more easily identify the drives that way.

 

... and a wish item.

Ability to change orientation of each drive individually. I have 6 vertical bays and 4 horizontal in my case. Would be great to allow them to be displayed properly.


Thanks! Great plugin!! 

USB drives are not supported, it has to be accessible via SAS/SATA controller.

It is a very bad practice to follow the device node in general as it might shift node, it might be added in the future for the configuration - but I see no real use for it when it is accessible at the front page which is the main page for the overview. I also want to keep the configuration page/list as small as possible.

 

The wish item is a task for design v2.0, this will not happen anytime soon though. Meanwhile you have to play with tray overrides, like 1*6 tray, and 10 for total trays for override. At least you can then separate them a bit.

Share this post


Link to post
10 minutes ago, olehj said:

USB drives are not supported, it has to be accessible via SAS/SATA controller.

It is a very bad practice to follow the device node in general as it might shift node, it might be added in the future for the configuration - but I see no real use for it when it is accessible at the front page which is the main page for the overview. I also want to keep the configuration page/list as small as possible.

 

The wish item is a task for design v2.0, this will not happen anytime soon though. Meanwhile you have to play with tray overrides, like 1*6 tray, and 10 for total trays for override. At least you can then separate them a bit.

Makes sense all around. Thanks for the quick and detailed response!

 

Share this post


Link to post
On 3/5/2019 at 4:01 AM, pkn said:

"Warning: SQLite3::exec(): near "ON": syntax error in /usr/local/emhttp/plugins/disklocation/pages/system.php on line 651
near "ON": syntax error"

 

First use, version Disk Location 2019.02.18b, unraid Version 6.5.3 2018-06-12

To make it more precise about this syntax error; this means the updated harddisk data like SMART status, hours, load cycle count etc. is not updated when refreshing/loading the page. It will destroy some of the benefits of this plugin, so I highly recommend to update Unraid to 6.6+ But I'll leave that up to you of course, but the plugin will not work as intended.

Share this post


Link to post

Is this supposed to automatically assign the devices?? I can see all of my hard drives but none are assigned. It is not necessarily a server box but rather a home build with 10 drives.

 

i am showing this error however

 

Warning: Invalid argument supplied for foreach() in /usr/local/emhttp/plugins/disklocation/pages/system.php on line 317

Share this post


Link to post
6 hours ago, Nightowl805 said:

Is this supposed to automatically assign the devices?? I can see all of my hard drives but none are assigned. It is not necessarily a server box but rather a home build with 10 drives.

 

i am showing this error however

 

Warning: Invalid argument supplied for foreach() in /usr/local/emhttp/plugins/disklocation/pages/system.php on line 317

It's impossible for anyone to know where your drives are located as the configurations out there varies a lot, so you must assign them manually the first time. So no automagic is happening there.

For the error it looks like it might detect a false blank or something odd in the SMART rotational data. I'll fix this later.

Share this post


Link to post

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.