Conversation
556f034 to
1468e5f
Compare
1468e5f to
06881ea
Compare
|
@ezhil56x I'm traveling today and will check your PR in the next 24hrs, thanks |
|
@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 |
There was a problem hiding this comment.
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.
|
@billz Any update? |
|
I'll be free to test the containers tomorrow. |
|
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 |
|
I tested the bookworm version by building it with this: 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. |
|
not to sh!t on other claimants but why the whole change in the workflow and the two seperate containers? |
The base image differs for each container to work correctly. |
|
@jrcichra Does this PR require any change or is this ready to merge? |
|
@ezhil56x I was traveling the last 48hrs. Will do more testing with both PRs and revert back with any change requests. thanks! |
|
@billz Any update? |
|
hey @jrcichra,
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? |
|
@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 😞 |
|
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. |
Issue
Fixes #18
/claim #18
Description
build_docker_image.ymljrcichra/raspap-docker:bookwormandjrcichra/raspap-docker:bullseyedocker images are built and pushed to the repositoryREADME.mdTested on Raspberry Pi 3 Model B+ with
Raspberry Pi OS bookwormandRaspberry Pi OS bullseye