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

Add /usr/lib/modules as a new volume to enable loading of kernel modules #78

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

Conversation

nixpanic
Copy link
Member

@nixpanic nixpanic commented Apr 5, 2018

It is recommended to create bricks on top of LVM (with thin-pool
provisioning). This requires certain device-mapper features in the
kernel. If these are missing, the lvm-tools will try to load the
appropriate kernel modules. Loading of kernel modules is only possible
if the .ko files are available inside the container.

Fixes: #75
Related: gluster/gluster-kubernetes#457
Signed-off-by: Niels de Vos ndevos@redhat.com

It is recommended to create bricks on top of LVM (with thin-pool
provisioning). This requires certain device-mapper features in the
kernel. If these are missing, the lvm-tools will try to load the
appropriate kernel modules. Loading of kernel modules is only possible
if the .ko files are available inside the container.

Fixes: gluster#75
Related: gluster/gluster-kubernetes#457
Signed-off-by: Niels de Vos <ndevos@redhat.com>
@nixpanic nixpanic added the bug label Apr 5, 2018
@nixpanic nixpanic requested a review from humblec April 5, 2018 08:42
@nixpanic
Copy link
Member Author

Could you review this, @jarrpa?

Copy link

@jarrpa jarrpa left a comment

Choose a reason for hiding this comment

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

LGTM. Does this mean we wouldn't need to specify the volume mount in a Pod spec?

@nixpanic nixpanic changed the title Add /usr/lib/modeles as a new volume to enable loading of kernel modules Add /usr/lib/modules as a new volume to enable loading of kernel modules Jun 28, 2018
@nixpanic
Copy link
Member Author

That needs to be specified for all I know. But the specs (or examples) are not part of this repo.

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

Successfully merging this pull request may close these issues.

2 participants