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

why cgroup? #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

why cgroup? #29

wants to merge 1 commit into from

Conversation

NL-TCH
Copy link
Collaborator

@NL-TCH NL-TCH commented Feb 19, 2024

if we remove this line, and run the container using the following command:
docker run --name raspap -it -d --privileged --network=host --cap-add SYS_ADMIN <imagename>
there is no command or code relating to cgroups and the container starts + hostapd is running.

in this way the container runs with the same command on ARM64 and X86

or do we need cgroup?

@NL-TCH
Copy link
Collaborator Author

NL-TCH commented Feb 19, 2024

maybe @jrcichra is the right one to ask here

@NL-TCH
Copy link
Collaborator Author

NL-TCH commented Feb 19, 2024

con: why fix it if it ain't broken
pro: for a homeassistant addon we cannot have the cgroup tag, as the addons in HA do not support it

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

Successfully merging this pull request may close these issues.

None yet

1 participant