search

dappnode / DAppNode

General repository of the project dappnode
GNU General Public License v3.0
581 stars 99 forks source link

Update debian ISO to v11.7.0 #562

Closed alexpeterson91 closed 1 year ago

alexpeterson91 commented 1 year ago

Update debian ISO to v11.7.0 to test with NUC12's to see if it resolves the firmware issue currently resolved only by workaround in the experiemental ISO release

pablomendezroyo commented 1 year ago

@alexpeterson91 nice catch!

cc: @3alpha @dsimog01

dsimog01 commented 1 year ago

I already talked to @alexpeterson91 and he told me the drivers issue is still not fixed by this Debian version. However, I agree we should test and merge this anyway

alexpeterson91 commented 1 year ago

I already talked to @alexpeterson91 and he told me the drivers issue is still not fixed by this Debian version. However, I agree we should test and merge this anyway

yes totally agreed, If 11.7 doesn't have any breaking changes to dependencies of core DNPs etc we should certainly upgrade to it ASAP Details for everyone's reference regarding Debian 11.7: As I relayed via Diego already, unfortunately it does not solve all the driver issues that came up with NUC12 and 13s,+, but it does solve a LOT of them, including an edge case that only a few current users (obviously I was the first to notice it by being the first Nodler to use a NUC with x16 PCI-e Slot like end of 2021) even would need to worry about, the issue being that the NUCs with the slots randomly throw errors on startup and boot and then just on the console over days about the PCI-e drivers not working among other things i collected a bunch of these errors, including HD audio, wifi, some storage drivers and the 2 Ethernet cards as issues that needed to be fixed for the NUC12 experimental firmware.

Diego's branch resolved all the issues we found except for the PCI-e issues, but has backports and a bunch of firmware files manually added.

Testing Debian 11.6 resolved none of the issues at all.

Testing Debian 11.7 here Resolved:

  1. Ethernet Card Firmware for Intel 2.5Gbe Port
  2. Ethernet Card Firmware for Marvell 10Gbe Port
  3. PCI-E Firmware (have not tested with a expansion card yet but errors are all gone and don't recur. Until now we hadn't even found a workaround for it, though Diego and I put little to no effort into this particular component)

Debian 11.7 did NOT Resolve:

  1. Intel Wifi 6E AX211 NVMe Card drivers, not detected by debian without manually installing driver, This is the only remaining blocker to resolve NUC12+ issues though since wired internet works with this new Debian 11.7 distro, adding a script installing just the Wifi driver for this particular card is trivial, I think...
  2. Intel/Realtek HD-Audio Sound Card Driver/firmware/Codec (not an issue to worry about really, nothing currently uses audio in any way on Dappnode AFAIK) issue is just the error which pops up on the console of NUC models with the HD sound card, but even that can be suppressed by disabling the sound card in the BIOS. Also we do have the missing wifi and sound card drivers already in Diego's Branch And the biggest issue for NUC12's was no way to get internet, with 11.7 we get wired internet on NUC12s so we can much more easily get NUC12's working with Wi-Fi using 11.7

Finally, @3alpha thanks for setting the link to the permalink not just the current folder as it was up till 11.5. I wasn't thinking further than just testing the firmware personally and expected it not to work, so I just quickly reverted the URL to the current from the previous permalink for 11.5 i added when 11.6 was released and broke all the CI

So separately, if no one has already done it; as @pablomendezroyo we need to keep upgrading Debian versions, we should add a bump Debian iso upstream version workflow to check stable releases, on a cron schedule like weekly or something to check the current URL or hash signature and see if it resolves or matches to what we have in our master repo currently, and if not open a PR with the new filename, path, and signature. and of course run all the build checks on that PR as well.

alexpeterson91 commented 1 year ago

Debian 12 supercedes this in PR #575

pablomendezroyo commented 1 year ago

If this PR is not needed, then please, remove it in favor of the Debian 12 one