-
Notifications
You must be signed in to change notification settings - Fork 547
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
build: make sure nfs-utils
is installed
#4243
Conversation
I think its better we run the entire ci just to be sure nothing breaks with this change. |
@Mergifyio queue |
✅ The pull request has been merged automaticallyThe pull request has been merged automatically at d753eaa |
Ceph is minimizing their container-images, which can cause the `nfs-utils` package to be dropped. As Ceph-CSI supports mounting NFS, it needs the `/sbin/mount.nfs` executable, so install the package (or a no-op if it is installed already). See-also: https://rook-io.slack.com/archives/C46Q5UC05/p1699188662893109 Signed-off-by: Niels de Vos <[email protected]>
6b44f8f
to
5cf0348
Compare
/test ci/centos/k8s-e2e-external-storage/1.28 |
/test ci/centos/upgrade-tests-cephfs |
/test ci/centos/mini-e2e-helm/k8s-1.28 |
/test ci/centos/k8s-e2e-external-storage/1.26 |
/test ci/centos/mini-e2e/k8s-1.28 |
/test ci/centos/upgrade-tests-rbd |
/test ci/centos/mini-e2e-helm/k8s-1.26 |
/test ci/centos/k8s-e2e-external-storage/1.27 |
/test ci/centos/mini-e2e/k8s-1.26 |
/test ci/centos/mini-e2e-helm/k8s-1.27 |
/test ci/centos/mini-e2e/k8s-1.27 |
Ceph is minimizing their container-images, which can cause the
nfs-utils
package to be dropped. As Ceph-CSI supports mounting NFS, itneeds the
/sbin/mount.nfs
executable, so install the package (or ano-op if it is installed already).
See-also: https://rook-io.slack.com/archives/C46Q5UC05/p1699188662893109
Signed-off-by: Niels de Vos [email protected]