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

bootstrap-containers: extend bootstrap-containers capabilities #2772

Merged

Conversation

arnaldo2792
Copy link
Contributor

Issue number:

Closes #2652

Description of changes:

This adds CAP_SYS_MODULE and CAP_CHROOT to bootstrap containers

Testing done:

  • I created a container image with this definition:
FROM fedora
ENTRYPOINT ["chroot", "/.bottlerocket/rootfs", "modprobe", "vfio_pci"]

I confirmed the kernel module was loaded:

bash-5.1# lsmod | grep vfio
vfio_pci               77824  0
vfio_virqfd            16384  1 vfio_pci
vfio_iommu_type1       36864  0
vfio                   45056  2 vfio_iommu_type1,vfio_pci
irqbypass              16384  1 vfio_pci

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

This adds `CAP_SYS_MODULE` and `CAP_CHROOT` to bootstrap containers

Signed-off-by: Arnaldo Garcia Rincon <[email protected]>
@arnaldo2792
Copy link
Contributor Author

(forced push to fix typo in commit message)

@arnaldo2792 arnaldo2792 changed the title bootstrap-containers: extend boootstrap-containers capabilities bootstrap-containers: extend bootstrap-containers capabilities Feb 1, 2023
@arnaldo2792 arnaldo2792 merged commit be10b16 into bottlerocket-os:develop Feb 10, 2023
@arnaldo2792 arnaldo2792 deleted the extend-bootstrap-containers branch June 19, 2023 18:36
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.

Increase bootstrap containers permissions
4 participants