Skip to content

fix: rpi-compatible Docker images#19

Open
ezhil56x wants to merge 9 commits into
RaspAP:masterfrom
ezhil56x:docker-images
Open

fix: rpi-compatible Docker images#19
ezhil56x wants to merge 9 commits into
RaspAP:masterfrom
ezhil56x:docker-images

Conversation

@ezhil56x

@ezhil56x ezhil56x commented Dec 28, 2023

Copy link
Copy Markdown

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

billz commented Dec 28, 2023

Copy link
Copy Markdown
Member

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

@billz

billz commented Dec 31, 2023

Copy link
Copy Markdown
Member

@jrcichra mind checking this PR? thanks :)

Comment thread 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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Author

@billz @jrcichra

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

@ezhil56x

ezhil56x commented Jan 1, 2024

Copy link
Copy Markdown
Author

@billz Any update?

@jrcichra

jrcichra commented Jan 1, 2024

Copy link
Copy Markdown
Collaborator

I'll be free to test the containers tomorrow.

@ezhil56x

ezhil56x commented Jan 1, 2024

Copy link
Copy Markdown
Author

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

jrcichra commented Jan 2, 2024

Copy link
Copy Markdown
Collaborator

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

NL-TCH commented Jan 2, 2024

Copy link
Copy Markdown
Collaborator

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

ezhil56x commented Jan 3, 2024

Copy link
Copy Markdown
Author

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

ezhil56x commented Jan 3, 2024

Copy link
Copy Markdown
Author

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

@ezhil56x

ezhil56x commented Jan 4, 2024

Copy link
Copy Markdown
Author

@billz @jrcichra Any update or changes required?

@billz

billz commented Jan 4, 2024

Copy link
Copy Markdown
Member

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

@ezhil56x

ezhil56x commented Jan 6, 2024

Copy link
Copy Markdown
Author

@billz Any update?

@NL-TCH

NL-TCH commented Jan 6, 2024

Copy link
Copy Markdown
Collaborator

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

ezhil56x commented Jan 7, 2024

Copy link
Copy Markdown
Author

@billz @jrcichra Any update?

@ezhil56x

ezhil56x commented Jan 7, 2024

Copy link
Copy Markdown
Author

@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

billz commented Jan 8, 2024

Copy link
Copy Markdown
Member

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