-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add support for the i386 architecture #818
Conversation
To support this, wouldn't the FTL need a corresponding i386 build? Just because the docker image support i386 doesn't mean Pihole itself would run. Right? |
Hey Daniel! I would think so, and looks like the support is already there https://github.com/pi-hole/docker-base-images/blob/edf2e156c54959cb6c9fb1a679755945892a114e/docker-compose.yaml#L60. Here is what gets downloaded during the build https://github.com/pi-hole/docker-pi-hole/pull/818/checks?check_run_id=2257113716#step:3:2063. Lastly, I can absolutely confirm it is working because this image is currently running on a CPU that has no AMD64 support whatsoever (https://ark.intel.com/content/www/us/en/ark/products/41411/intel-atom-processor-n280-512k-cache-1-66-ghz-667-mhz-fsb.html). |
I clearly don't understand all the nuances between x86, i386, etc. Luckily, me understanding architectures isn't a requirement for PRs. You've done a good job following the same pattern from the other architectures. Unfortunately, you have a conflict, so that will need to be fixed. Also, I think you might be missing a change in here as well? docker-pi-hole/gh-actions-deploy.sh Lines 51 to 56 in 731cd5e
Otherwise, I think it will be up to someone else to decide if this is something the project wants to support or not. |
Signed-off-by: static-void-main <[email protected]>
Signed-off-by: static-void-main <[email protected]>
Good catch regarding gh-actions-deploy.sh, updated that one as well. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. I'm unable to test it, and so I would also be unable to support this feature- but all the changes look correct
Yes, there is a dedicated Concerning Check out Intel 80386 for more information. To be independent of vendor-specific names we prefer |
That being said I should also mention that the end of the 32-bit era is coming closer. Ubuntu/RedHat/SuSe, they all stopped offering 32 bit versions of their operating systems already years ago. IIRC Ubuntu even announced that they will stop supporting 32bit applications on 64bit operating systems. Debian is like the last garrison for 32-bit operating systems. But even here Wheezy required I think we will support 32 bit as long as there is at least one Debian release that officially supports it, though. Unless there are external limitation that force us away from being 32 bit compatible but I don't see this happening (would also be an issue for ARM). |
This PR adds support for the i386 architecture.
Description
Adds a new image that is able to run on 32-bit hosts. The image is based on 32-bit versions of debian-debootstrap and S6.
Motivation and Context
Being able to run Pi-hole through Docker on 32-bit machines. The i386 architecture is officially supported on Debian according to Supported Operating Systems.
How Has This Been Tested?
Built and pushed the image to a private Docker Hub repo with Actions. Deployed it to an Atom N280 powered thin client with Docker Compose. Runs great.
Types of changes
Checklist: