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

sys-kernel/coreos-modules: disable CONFIG_NFS_V4_2_READ_PLUS #2390

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

ader1990
Copy link
Contributor

@ader1990 ader1990 commented Oct 22, 2024

Disable CONFIG_NFS_V4_2_READ_PLUS kernel config, as Linux kernel >= 6.6 enabled the CONFIG_NFS_V4_2_READ_PLUS config option by default, and nfs-ganesha version <= 6.1 is broken due to mishandling of the read_plus operation.

See:
nfs-ganesha/nfs-ganesha@24da5c3

See: flatcar/Flatcar#1565

See: nfs-ganesha/nfs-ganesha#1188

Testing done

Manually checked on a two node K8S cluster that the issue is no longer present when CONFIG_NFS_V4_2_READ_PLUS=n.

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
  • Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

Disable CONFIG_NFS_V4_2_READ_PLUS kernel config, as
Linux kernel >= 6.6 enabled the CONFIG_NFS_V4_2_READ_PLUS config option
by default, and nfs-ganesha version <= 6.1 is broken due to mishandling
of the read_plus operation.

See:
nfs-ganesha/nfs-ganesha@24da5c3

See: flatcar/Flatcar#1565

See: nfs-ganesha/nfs-ganesha#1188
Copy link

github-actions bot commented Oct 22, 2024

@krnowak
Copy link
Member

krnowak commented Oct 22, 2024

Do we want to reenable this CONFIG at some point?

@ader1990
Copy link
Contributor Author

Do we want to reenable this CONFIG at some point?

Maybe, I have not tried/found yet a nfs-ganesha version that fixes the issue, as it is not released yet, waiting for kubernetes-sigs/nfs-ganesha-server-and-external-provisioner#152

@ader1990
Copy link
Contributor Author

But even if the nfs-ganesha is fixed, as the possible fix landed a few days ago, it is safe to assume that we need to backport this change into the /stable/beta/alpha Flatcar versions.

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

Successfully merging this pull request may close these issues.

5 participants