-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Refactor package updater #2630
Comments
Merge in DNS/adguard-home from 2630-updates to master Updates #2630. Squashed commit of the following: commit 6dab69d Author: Ainar Garipov <A.Garipov@AdGuard.COM> Date: Tue Feb 2 20:00:59 2021 +0300 all: changelog commit 63720bb Author: Ainar Garipov <A.Garipov@AdGuard.COM> Date: Tue Feb 2 19:54:28 2021 +0300 updater: fix pkg unpacking
Okay, so we've pushed a patch for this in 9cd9054. But we still have a lot of users with bad updaters. As such the plan @ameshkov has proposed is the following:
@ameshkov, can you please make sure that I've described the plan correctly? |
I think we can drop it once we release the first patch to v0.105 |
We've removed the crutch, but we still need to refactor the updater. |
We'll probably rewrite the package completely in the next cycle. Moving the issue there. |
As part of the re-write may I suggest a few things? I wrote this thread on how to install and use AGH's edge builds on OpenWrt. : https://forum.openwrt.org/t/how-to-updated-2021-installing-adguardhome-on-openwrt/113904 It details a fair chunk of issues others have had installing and using AGH. For OpenWrt router installs you are usually limited on space and memory. As such disk space checks before you update should look to provide feedback to users updating from the web interface. At present updates that fail due to low diskspace do generate a warning in syslog but do not return a useful error message to the web interface. That was one of the reasons I filed the #3801 issue. Some of this can be mitigated by using exroot or even mounting a usb drive as a folder via the router to avoid disk space issues. However not all routers can do this so improving disk usage and making users understand logging choices and space required is much preferred. Its also why I generated this script to update instead. It downloads the AGH zip to /tmp (memory) and then extracts directly over the existing binary after stopping AGH. It then restarts AGH. Its not ideal because a backup is preferable but with limited space that is the compromise I've had to settle with. It is also additionally an issue as from 106 to 107 the binary has grown considerably. #4025 With 16days of usage right now i'm at 4mb for statistics, and 80mb and 50mb for 2 sets of query data. That's with 473,525 A proper logrotate with a prerequisite listed of some ideas of disk space needed (eg 30mb for binary, 10mb per day for queries?) will help less technical users to understand what is going on better and also enable better judgement of how to manage the setup of logging and filters. Hopefully also avoid "hey i have 2million rules why have i run out of space" posts :) Hopefully my experiences help with planning out your upgrader flow and improve feedback via the webinterface for users. updateAGH.sh
|
Merge in DNS/adguard-home from 2630-updates to master Updates AdguardTeam#2630. Squashed commit of the following: commit 6dab69d Author: Ainar Garipov <A.Garipov@AdGuard.COM> Date: Tue Feb 2 20:00:59 2021 +0300 all: changelog commit 63720bb Author: Ainar Garipov <A.Garipov@AdGuard.COM> Date: Tue Feb 2 19:54:28 2021 +0300 updater: fix pkg unpacking
Our testers reported that they cannot autoupdate from
v0.105.0-beta.2
tov0.105.0-beta.3
. Probably packageupdater
issues. In the future, the package requires a thorough rewrite.The text was updated successfully, but these errors were encountered: