Skip to content
This repository has been archived by the owner on Apr 19, 2019. It is now read-only.

nfs mount not permitted in lxc-openstack aa profile #487

Closed
BjoernT opened this issue Nov 6, 2014 · 10 comments
Closed

nfs mount not permitted in lxc-openstack aa profile #487

BjoernT opened this issue Nov 6, 2014 · 10 comments

Comments

@BjoernT
Copy link
Contributor

BjoernT commented Nov 6, 2014

The lxc-openstack profile does not allow nfs mounts, that prevents cinder-volume from mounting as NFS storage backend

@cloudnull
Copy link
Member

To use NFS you will need to change the profile to "unconfined". This is not a default setup though is totally configurable. This can be modified using the RPC user config in the cinder section under the container vars area.

@BjoernT
Copy link
Contributor Author

BjoernT commented Nov 6, 2014

I have already a commit to add

mount fstype=nfs* -> /var/lib/cinder/mnt/**,

in the profile. I make a pull request

@cloudnull
Copy link
Member

We need to revisit this PR and modify the allowable NFS mounts to anything from within the container. IE: potentially glance would be using NFS. From a supportability standpoint we need

mount fstype=nfs* -> /var/lib/cinder/mnt/**,

To be

mount fstype=nfs* -> /**,

@BjoernT
Copy link
Contributor Author

BjoernT commented Nov 20, 2014

@cloudnull
Glance most likely would be a bind mount from a host mounted NFS ? What do you think

mancdaz pushed a commit to mancdaz/ansible-lxc-rpc that referenced this issue Nov 21, 2014
@cloudnull
Copy link
Member

@BjoernT - I think the user should be able to have the option to either bind mount the path into the container and then modifying the mounts in lxc config to make it persistent, or have the option to be able to simply perform an NFS mount in the container. At present I know of at least 1 deployment where the container profile was changed to "unconfined" to allow for NFS to be used within the glance / cinder containers and it would be better to have a little more liberal NFS policy in containers than to simply set them as unconfined.

@cloudnull
Copy link
Member

Related review from QE when backporting: https://github.com/rcbops/ansible-lxc-rpc/pull/557/files

@BjoernT
Copy link
Contributor Author

BjoernT commented Nov 24, 2014

Yes we can add /var/lib/glance/images in the profile. I personally would keep the profiles as precise as possible.

@mancdaz
Copy link
Contributor

mancdaz commented Nov 25, 2014

@cloudnull @BjoernT so we need a new PR that adds /var/lib/glance/images/** to the profile, rather than /**, since we want to be as controlled as possible?

@cloudnull
Copy link
Member

That sounds like a plan to me.

@b3rn4rd0s b3rn4rd0s assigned BjoernT and unassigned cloudnull Nov 25, 2014
@cloudnull cloudnull assigned BjoernT and unassigned cloudnull Nov 25, 2014
@BjoernT
Copy link
Contributor Author

BjoernT commented Nov 25, 2014

I just added this directory with a pull request

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

No branches or pull requests

5 participants