-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
Deduplication info not visible for volumes in storage manager #67
Comments
I can't test it until tomorrow because my DS1821+ is doing a data scrub, and the drives from my DS720+ are currently connected to a PC for developing my Synology_Recover_Data script. What we need to change may be in storage_panel.js or |
I don't know which image from #25 you are referring to because I can't see your screenshot. All I get is a webpage saying "This private-user-images.githubusercontent.com page can’t be found". |
Out the 43,000 lines in storage_panel.js only 371 line contain the word dedupe. If we search for dedupe related functions there are only 23.
My javascript knowledge is very basic so I use ChatGPT to tell me what the function is doing 😄
ChatGPT said: It checks if this.dedup_info exists and is truthy (!!this.dedup_info). The double negation !! is a common way to coerce any value to a Boolean. So I would try:
Or this:
Or even this:
showDedupRate may also play a part in what is shown or when:
|
I'm sorry, the link included my session data I assume. Here's the screenshot I was referring to: |
In the meantime I looked deeper into storage_panel.js and I doubt that the deduplication info can fully be enabled just by changing the JavaScript. It's like with "&&e.dedup_info.show_config_btn" that you removed to enable hdd support.
Then
So all these values indicate that data deduplication is not active for this volume but it is! This might be because it's an hdd volume. Only for my ssd volume With that being said, in my opinion it doesn't help to remove or change checks for To fix all this it must be fixed in the web service implementation on the DS. Is it known if this is something that can be achieved? |
What are we looking at in that screenshot? Is something missing? |
Nothing is missing in the screenshot, which I took from issue #25 just to show what is missing in my Storage Manager. In the screenshot there's the expanded volume, and its "Info" includes "Deduplication status:" and "Deduplication savings:". These two are missing for my dedupe-enabled hdd volumes. I only see "File system:" and "Total capacity:". |
Ah, that's what the screenshots in issue #25 were showing. I only noticed the choices in the ... menu. I didn't notice that the "Deduplication status:" and "Deduplication savings:" were missing under the "Info" section. I don't think I've seen them before. I never realised something was missing. I can't find SYNO.Storage.CGI.Storage in DSM 7.2.1 with Update 1 or in Storage Manager 1.0.0-0017 Storage Manager 1.0.0-0017 does have /usr/local/packages/@appstore/StorageManager/webapi/SYNO.Storage.CGI.lib And for DSM 7.2 where Storage Manager was part of DSM it's /usr/syno/synoman/webman/modules/StorageManager/SYNO.Storage.CGI.lib |
My understanding so far is: libStorage.so also contains This is another way of calling
There is another API called when you click "Configure Data Deduplication" in Storage Manager. Using
It is strange that deduplication works in general but Last update here: I found |
I just played around with deduplication on an NVMe volume. For manual deduplication "Deduplication status" shows the last run time and there is no "Deduplication savings". For automatic deduplication "Deduplication status" shows Automatic and "Deduplication savings" shows either:
For the NVMe volume with automatic deduplication enabled but not run yet
For the same NVMe volume with automatic deduplication disabled, and after manually running deduplication
It sounds like I need to edit /lib/libsynostoragemgmt.so so that SYNOBtrfsDedupeAllowedSSDModelsGet returns true. |
When I run the following command with just HDD volumes in the NAS:
the first line is always:
But when I run that command with an SSD volume in the NAS there was no [Line 295] error. SYNO.Storage.CGI.lib contains a single line, so I'm not sure if it's a synowebapi error or not. |
Maybe it's not indicating an error but just echoing what it's doing? I have two HDD volumes and one SSD volume in my DS920+ (and DX517) and get |
The results from googling for |
I also tried creating two volumes with deduplication, one on an HDD and one on an NVMe. However, deduplication info is not displayed for the HDDs. The system is a 920+ running DSM 7.2.1-69057 Update 5, with Storage Manager 1.0.0-0017. While not essential for functionality, having this information would be very convenient. |
I was wondering if it's possible to view deduplication statistics via SSH since they aren't directly visible in the graphical interface? |
Hi @007revad, |
I haven't figured out how to show "Deduplication status" and "Deduplication savings" in the volume's info panel for HDDs. |
I successfully enabled btrfs data deduplication for my two hdd volumes with the script on my DS920+ running DSM 7.2.1-69057 Update 3. I use it for my two volumes that are on hdd-only storage pools and it seems that it is working fine.
But I don't see any deduplication info. I just see file system and total capacity. Deduplication status and savings are missing.
From what I found out so far in file storage_panel.js, it seems that this.dedup_info.show is always false. Even if it would be true it wouldn't display the correct status because this.dedup_info.status is "disable" (but it should be "auto" for two volumes). Unfortunately I have not found out where these values come from.
Is something missing on my DS920+? It would be great if the deduplication info can be displayed in storage manager.
The text was updated successfully, but these errors were encountered: