Skip to content

replace k8s.io/utils/mount with k8s.io/mount-utils#21

Closed
hensur wants to merge 1 commit into
openebs:developfrom
hensur:use-mount-utils
Closed

replace k8s.io/utils/mount with k8s.io/mount-utils#21
hensur wants to merge 1 commit into
openebs:developfrom
hensur:use-mount-utils

Conversation

@hensur

@hensur hensur commented Aug 1, 2024

Copy link
Copy Markdown
Contributor

What this PR does:
replaces the k8s.io/utils/mount dependency with k8s.io/mount-utils

the upstream package has been renamed: https://github.com/kubernetes/utils/tree/master/mount

The new package also includes a change to drop consistentread calls on newer kernels which don't require that workaround anymore: kubernetes/mount-utils@4ae857e

On some of our Nodes with high mount activity the zfs provisioner would often fail to read /proc/mounts because of an inconsistentread error.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Checklist

  • PR messages has document related information
  • Labelled this PR & related issue with documentation tag
  • PR messages has breaking changes related information
  • PR messages has upgrade related information
  • Labelled this PR & related issue with requires-upgrade tag
  • Tests updated

package has been renamed: https://github.com/kubernetes/utils/tree/master/mount

The new package also includes a change to drop inconsistentread calls on
newer kernels which don't require that workaround anymore:
kubernetes/mount-utils@4ae857e

Signed-off-by: Henning Surmeier <h.surmeier@mittwald.de>
@Abhinandan-Purkait

Abhinandan-Purkait commented Nov 1, 2025

Copy link
Copy Markdown
Member

@hensur Thanks for the PR, the changes have been incorporated as a part of #29 We have included your commit in it. Closing this!

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.

2 participants