arevindh / pihole-speedtest

Pihole Speedtest Mod
https://arevindh.github.io/pihole-speedtest/
MIT License
218 stars 22 forks source link

(Re)install and Uninstall Failsafe Script #130

Closed ipitio closed 1 year ago

ipitio commented 1 year ago

This is part of https://github.com/arevindh/AdminLTE/pull/70

Features:

Fixes:

ipitio commented 1 year ago

Everything works now!

town3r commented 1 year ago

I got this error when running your install/upgrde script

fatal: HEAD does not point to a branch
Error: Remote revision could not be obtained, please contact Pi-hole Support Additional debugging output:
HEAD detached at v5.18.1

image

ipitio commented 1 year ago

@town3r oops forgot to sync test branch, should work now

town3r commented 1 year ago

Still getting some errors:

image

ipitio commented 1 year ago

Hmm now I'm only cloning the last commit, could you try now

town3r commented 1 year ago

No dice.

image

image

ipitio commented 1 year ago

what about now

town3r commented 1 year ago

Still getting the HEAD detached error. :(

No more crazy long list of version tags.

image

ipitio commented 1 year ago

What about now

town3r commented 1 year ago

This is what I'm getting currently:

image

ipitio commented 1 year ago

Woo! It works! I got rid of those errors but they didn't matter.

town3r commented 1 year ago

image

ipitio commented 1 year ago

Lol no way they released an update rn. Perfect timing to test how my script would handle a pihole update before the mod is updated for it. And looks like the test passed so that's good

ipitio commented 1 year ago

How have you been running speedtests without speedtest?

If you were using speedtest-cli, previous versions of my script would've removed it.

town3r commented 1 year ago

I have speedtest, so not sure why it said it couldn't find it.

image

ipitio commented 1 year ago

How did you install it? What instructions did you use to install it or who's script did you use?

town3r commented 1 year ago

Used the install steps from arevindh

https://github.com/arevindh/pihole-speedtest/wiki/Installing-Speedtest-Mod#install-speedtest-cli

Grabbed link to latest version here

https://www.speedtest.net/apps/cli

ie: https://install.speedtest.net/app/cli/ookla-speedtest-1.2.0-linux-armhf.tgz

ipitio commented 1 year ago

Ah, will now account for that

ipitio commented 1 year ago

Can u take a sc of the output now

town3r commented 1 year ago

image

town3r commented 1 year ago

Ahh your new script added a bad repo

image

ipitio commented 1 year ago

Yeah ookla is dumb and don't detect os properly, I'm fixing their script. They account for raspbian and get so so close to using Debian.

town3r commented 1 year ago

Is your script trying to reinstall the speedtest app itself and the speedtest pihole mod?

ipitio commented 1 year ago

It's trying to add the speedtest source so that apt knows what speedtest is and stops showing that error. Otherwise the script does exactly what those time stamps say it's doing.

ipitio commented 1 year ago

Try now

town3r commented 1 year ago

image

town3r commented 1 year ago

Doesn't look like it's auto running the Speedtest hourly like within the settings tho. :(

image

ipitio commented 1 year ago

Could you click confirm in speedtest settings? Just confirm will do it but you could update too. Should work after, we'll know in 30 min. Simply updating doesn't update/fix the timer cuz sid wanted that to only happen when user selects confirm.

ipitio commented 1 year ago

And then you could send a sc of that cat command

town3r commented 1 year ago

Cat command?

ipitio commented 1 year ago

cat /var/log/pimod.log it's at the bottom of speedtest settings

town3r commented 1 year ago

image

ipitio commented 1 year ago

Did the Speedtest run, if it didn't I could still maybe fix before next hour.

town3r commented 1 year ago

Looks like it! 🥳 image

town3r commented 1 year ago

Hmm I'm now seeing this when manually running pihole -up

image

ipitio commented 1 year ago

I think the local revision thing is because my latest changes aren't under a release, if pihole -up worked before.

Are the DNS issues related to the Mod?

Did running pihole -up work before, even with the mod still installed? If so it might be unnecessary to uninstall the mod before updating pihole.

town3r commented 1 year ago

I think the local revision thing is because my latest changes aren't under a release, if pihole -up worked before. Are the DNS issues related to the Mod?

I think so? Only thing that I've changed has been from your script.

Did running pihole -up work before, even with the mod still installed? If so it might be unnecessary to uninstall the mod before updating pihole.

It did work before. I'll try uninstalling the mod via the settings option, running pihole -up followed by running your script again

town3r commented 1 year ago

Didn't even have to uninstall, actually just ran arevindh's upgrade script and once ran I was able to run pihole -up successfully.

Running your script, brought back the errors from above

ipitio commented 1 year ago

Just added a new tag, if pihole -up works now then it means it'll work when sid merges. It just won't work when updating to my test branch. If it still doesn't work, then ig just don't use it when you have the mod installed lol. Is it vital that it works?

curl mod.sh | sudo bash -s up un uninstalls the mod and updates pihole so you can think of it as an equivalent command. This is the same as ticking the reinstall latest and uninstall boxes.

town3r commented 1 year ago

Any idea why the hourly tests would stop randomly at 11 yesterday?

image

image

arevindh commented 1 year ago

That's weird

town3r commented 1 year ago

Looks like it started back up at 08:00 this morning due to me selecting the (Re)install Latest option under Speedtest Settings at like 07:30.

image

So something must have been off w/ the previous version ¯_(ツ)_/¯

town3r commented 1 year ago

That's weird

Right! Not sure what happened/changed around that time.

¯_(ツ)_/¯

Glad it seems to be happy again since running your fancy (Re)install Latest feature.