-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Statistics are reset after each reboot #2661
Comments
Hi, can you please show, what your statistics interval is set to? It's the |
@ainar-g, |
Merge in DNS/adguard-home from 2661-imp-stats-logging to master Updates #2661. Squashed commit of the following: commit 474735a Author: Ainar Garipov <[email protected]> Date: Thu Feb 11 17:44:34 2021 +0300 stats: imp err handling, logs
The current stats code seems to hide a lot of useful information, unfortunately. I've just pushed a623ac6, where I added more verbose logging to the code. Could you please update and collect the new logs? Thanks! |
@ainar-g, unfortunately, that server is gone, but i tried to reproduce it on my VM and it looks fine now🤷🏻♂️ IDK how to help here🙄 |
That's not statistics though, that's query log. How exactly do you reboot the machine? AdGuard Home keeps a buffer of recent queries in the RAM, so if it is not shut down properly, that buffer could remain unflushed to the filesystem. |
@ainar-g, via my hosting provider panel and this was not before on 0.102.0, 0.103.0 etc. version🤨 |
I don't think that that particular mechanism has changed much in the recent releases. At least I don't remember that. If you can, you should set up your machine to kill the AGH process with |
@ainar-g, i set |
Are you sure that your |
@ainar-g, yes, i'm sure. Moreover, part of the statistics disappears, and not all. |
I have no ideas besides the VPS's filesystem shenanigans. We've never seen anything similar. If you have a stable reproduction on a machine that is not on the same VPS, please let us know. |
Merge in DNS/adguard-home from 2661-imp-stats-logging to master Updates AdguardTeam#2661. Squashed commit of the following: commit 474735a Author: Ainar Garipov <[email protected]> Date: Thu Feb 11 17:44:34 2021 +0300 stats: imp err handling, logs
Prerequisites
Please answer the following questions for yourself before submitting an issue. YOU MAY DELETE THE PREREQUISITES SECTION.
Issue Details
Expected Behavior
Statistics are saved after reboot.
Actual Behavior
Statistics are reset after each reboot actually.
Additional Information
The logs is attached.
The text was updated successfully, but these errors were encountered: