-
Notifications
You must be signed in to change notification settings - Fork 101
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
Update Pi-hole to v6 development branch #261
Conversation
6b26344
to
1e59db2
Compare
@eiddor I've got this branch mostly working if you want to give it a try! (I was super wrong about the required changes being minimal) |
36a4832
to
ec736e3
Compare
DO NOT MERGE Signed-off-by: Kyle Harding <[email protected]>
ec736e3
to
bd693a4
Compare
I can, though I've got IPv6 disabled on my LAN. Do you just need it tested with v4 or do you want me to enable v6 and try testing with that?
Oh goodness, I see! |
It's not at all related to IPv6 or IPv4, the v6 is referring to the next big Pi-hole core release following v5. All Pi-hole core and related projects have a |
Oh man - This is what happens when you live and breathe networking. This entire time I thought we were talking about IPv6. I'll test it out this weekend. |
A bunch of things:
|
|
I'm not sure what happened to your FTL database. Was this a fresh install or a migration? What env vars do you have set so I can try to reproduce? |
Not right now, in case you hit an issue that needs fixing. Migration is the expected path after all, just surprised I didn't hit it. If you have another device to test with a clean slate that would narrow down whether it is your environment or the migration that is to blame. |
I have a zillion devices without displays - I'll load up on a new one here in a little bit. Should I start with a new fleet as well? |
up to you, I don't think a new fleet brings any value as the releases are independent and devices can be pinned to whatever release |
The new device seems to be working properly and hasn't looped like earlier. There are still some errors on the console, but nothing like before:
|
That's still more errors than I saw on my test device migration. And you're sure that the errors go away when rolling back, and reappear when going to v6 again? I should spin up another device and try to reproduce. |
Definitely no SQL errors when I roll back, though I do see these messages:
However, Gravity is doing its thing on the non v6 version: Went back to v6 one more time and I don't get the SQL errors anymore, however I still see the above messages. |
It seems the migration has corrupted your DB in both cases, even with a fresh install. I'm not sure our wrapper has anything to do with that, but it's concerning nonetheless. We will keep an eye on it and maybe it gets fixed upstream? |
Ok, I lied because I had the scroll paused - The SQL errors indeed return (and gravity stops working) when I go back to the v6 version:
It does look like there have been some upstream updates according to the UI: |
New fleet, new devices, fresh push of this branch, similar messages - This one is constantly scrolling:
Pushed the main branch and all is good. |
Are these all on the same network? I haven't tried to reproduce but I know you have a history of "special" networks ;) |
Hahaha I was waiting for you to start. There's nothing "special" about my network just because I don't run plebe gear! I assure you that it's a regular network, just running Enterprise-grade hardware. And yes, it's the same network, but I can confidently say that my network does not cause SQL errors. |
Maybe we should pause development until the upstream is baked. I'm seeing other issues inside of pihole unrelated to what we're doing (I think.) |
Superseded by #297 |
DO NOT MERGE
Signed-off-by: Kyle Harding [email protected]