Skip to content
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

fix: rpi-compatible Docker images #19

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

ezhil56x
Copy link

@ezhil56x ezhil56x commented Dec 28, 2023

Issue

Fixes #18
/claim #18

Description

  • Added support for debian based systems
  • Works on bookworm and github.repo:bullseye
  • Made required changes to build_docker_image.yml
  • When there is a push on master branch jrcichra/raspap-docker:bookworm and jrcichra/raspap-docker:bullseye docker images are built and pushed to the repository
  • Made required changes on README.md

Tested on Raspberry Pi 3 Model B+ with Raspberry Pi OS bookworm and Raspberry Pi OS bullseye

@billz
Copy link
Member

billz commented Dec 28, 2023

@ezhil56x I'm traveling today and will check your PR in the next 24hrs, thanks

@billz
Copy link
Member

billz commented Dec 31, 2023

@jrcichra mind checking this PR? thanks :)

README.md Outdated
1. Start the container with `jrcichra/raspap-docker:bookworm` image

```
docker run --name raspap -it -d --privileged --network=host -v /sys/fs/cgroup:/sys/fs/cgroup:rw --cap-add SYS_ADMIN jrcichra/raspap-docker:bookworm
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this image would be out of date. The build pipeline is probably building an image on ghcr.io/raspap/raspap-docker, which would be preferable to my docker.io account.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jrcichra Changed the image path

@ezhil56x
Copy link
Author

@billz @jrcichra

I have completed everything. Can you review once and if some changes are required, happy to do!

@ezhil56x
Copy link
Author

ezhil56x commented Jan 1, 2024

@billz Any update?

@jrcichra
Copy link
Collaborator

jrcichra commented Jan 1, 2024

I'll be free to test the containers tomorrow.

@ezhil56x
Copy link
Author

ezhil56x commented Jan 1, 2024

Earlier due to ro there was an issue, so I have changed it to rw and when the setup file runs as root it is creating some issue while installing and configuring raspap. So added user to the dockerfile

@jrcichra
Copy link
Collaborator

jrcichra commented Jan 2, 2024

I tested the bookworm version by building it with this:
docker build -t ghcr.io/raspap/raspap-docker:bookworm -f Dockerfile.deb12 .
and running with this:
docker run --name raspap -it -d --privileged --network=host --cgroupns host -v /sys/fs/cgroup:/sys/fs/cgroup:rw --cap-add SYS_ADMIN ghcr.io/raspap/raspap-docker:bookworm

Then following the steps of making a user and installing raspap, then restarting the container. That all worked.

I was able to connect to the AP but couldn't get internet until I did the step @billz mentioned here: #20 (comment)

Then I connected and it worked. I could do a speedtest.net on my phone.

I tried restarting the AP without WPA. That required me to restart the docker container + reapply the iptables rules for it to work. So it's functional but has some rough edges we can iron out over time.

@NL-TCH
Copy link
Collaborator

NL-TCH commented Jan 2, 2024

not to sh!t on other claimants but why the whole change in the workflow and the two seperate containers?
as stated in #11 (comment) this is not the problem of raspap, but of docker, so i honestly don't get all the changes...

@ezhil56x
Copy link
Author

ezhil56x commented Jan 3, 2024

not to sh!t on other claimants but why the whole change in the workflow and the two seperate containers?
as stated in #11 (comment) this is not the problem of raspap, but of docker, so i honestly don't get all the changes...

The base image differs for each container to work correctly.

@ezhil56x
Copy link
Author

ezhil56x commented Jan 3, 2024

@jrcichra Does this PR require any change or is this ready to merge?

@ezhil56x
Copy link
Author

ezhil56x commented Jan 4, 2024

@billz @jrcichra Any update or changes required?

@billz
Copy link
Member

billz commented Jan 4, 2024

@ezhil56x I was traveling the last 48hrs. Will do more testing with both PRs and revert back with any change requests. thanks!

@ezhil56x
Copy link
Author

ezhil56x commented Jan 6, 2024

@billz Any update?

@NL-TCH
Copy link
Collaborator

NL-TCH commented Jan 6, 2024

hey @jrcichra,

I tried restarting the AP without WPA. That required me to restart the docker container + reapply the iptables rules for it to work. So it's functional but has some rough edges we can iron out over time.

I don't have this issue while using the changes in #20 and #21 . When i reboot and stop-start the container, the firewall rules are still there. can you verify if using these PR's fixes the issue?

@ezhil56x
Copy link
Author

ezhil56x commented Jan 7, 2024

@billz @jrcichra Any update?

@ezhil56x
Copy link
Author

ezhil56x commented Jan 7, 2024

@billz I have bumped the version that fixes the issue and added good practices 7 days back. Not sure why this PR wasn't selected 😞

@billz
Copy link
Member

billz commented Jan 8, 2024

I'm approaching this as an end-user, not a Docker pro, and generally finding it more difficult to use this PR. The majority of RaspAP users, I believe, want as little friction as possible getting a container running. In the interest of simplifying things, we've also decided to drop Debian 11 support.

There are still issues with the docker image build and ghcr.io, so we may create a separate bounty for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a RPi-compatible Docker image for RaspAP
4 participants