search

gluster / gluster-containers

Dockerfiles (CentOS, Fedora, Red Hat) for GlusterFS
https://github.com/gluster/gluster-containers/pkgs/container/gluster-containers
223 stars 135 forks source link

remove sudo from the containers #160

Closed raghavendra-talur closed 4 years ago

raghavendra-talur commented 4 years ago

We don't need the sudo util when Gluster is containerized. The requirements are met by running the container as privileged.

Signed-off-by: Raghavendra Talur rtalur@redhat.com

raghavendra-talur commented 4 years ago

@nixpanic @phlogistonjohn Please have a look.

phlogistonjohn commented 4 years ago

Change looks good to me at first glance but the travis job is failing and its not obvious why.

raghavendra-talur commented 4 years ago

It was failing because a command later in the Dockerfile was updating the sudoers file which won't exist now. I have pushed an update.