[Bug 2059197] Re: mount.nfs: Fix minor version parsing when '-t nfs4' and '-o vers=4.x' are used together

2024-04-25 Thread Launchpad Bug Tracker
This bug was fixed in the package nfs-utils - 1:1.3.4-2.5ubuntu3.7 --- nfs-utils (1:1.3.4-2.5ubuntu3.7) focal; urgency=medium * Fix minor version parsing when '-t nfs4' and '-o vers=4.x' or '-o vers=4,minorversion=x' are all used together. This fixes a regression caused by

[Bug 2059197] Re: mount.nfs: Fix minor version parsing when '-t nfs4' and '-o vers=4.x' are used together

2024-04-25 Thread Andreas Hasenack
Your verification showed "permission denied" when NFSv3 was attempted, which I expect when -t nfs4 is used, but not in the -t nfs case, or when "-t" is not used at all. Since this is a quick check, I ran these tests: root@j-nfs-server:~# cat /proc/fs/nfsd/versions -2 +3 -4 -4.0 -4.1 -4.2

[Bug 2059197] Re: mount.nfs: Fix minor version parsing when '-t nfs4' and '-o vers=4.x' are used together

2024-04-25 Thread Ryan S
This fixes my issue! https://bugs.launchpad.net/ubuntu/+source/util- linux/+bug/2063381 nfs-common/focal-proposed,now 1:1.3.4-2.5ubuntu3.7 amd64 [installed] Installed from proposed repo. Mounted via pam_mount upon ssh login using krb5p; NFSv4 successful mount with vers=4.0 which I could not do

[Bug 2059197] Re: mount.nfs: Fix minor version parsing when '-t nfs4' and '-o vers=4.x' are used together

2024-04-18 Thread Matthew Ruffell
Performing verification for Focal. I started two VMs, one Jammy, which is the nfs server, and one Focal, which is the client. I used the instructions in the testcase sections of this bug, and also bug 2049262. The Focal VM has nfs-common 1:1.3.4-2.5ubuntu3.6 from -updates. Let's try the

[Bug 2059197] Re: mount.nfs: Fix minor version parsing when '-t nfs4' and '-o vers=4.x' are used together

2024-04-18 Thread Andreas Hasenack
Thanks for the detailed -o vers=x.y tests. I think we are just missing the default case, with nothing fancy specified in the command-line. Please also include in the test a normal without-options mount attempt, i.e., the default. Something like a series of these, verifying which version it ended

[Bug 2059197] Re: mount.nfs: Fix minor version parsing when '-t nfs4' and '-o vers=4.x' are used together

2024-04-11 Thread Heitor Alves de Siqueira
Thanks for the revised debdiff, Matthew! And nice work on the extensive sanity check for the version laddering. The new debdiff looks good, I've sponsored it for Focal. Thanks! -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu.

[Bug 2059197] Re: mount.nfs: Fix minor version parsing when '-t nfs4' and '-o vers=4.x' are used together

2024-04-10 Thread Matthew Ruffell
Thanks for pointing out the missing patch Heitor. I was indeed missing it. Please review and sponsor the V2 debdiff. Thanks, Matthew -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/2059197 Title:

[Bug 2059197] Re: mount.nfs: Fix minor version parsing when '-t nfs4' and '-o vers=4.x' are used together

2024-04-10 Thread Matthew Ruffell
Sanity testing the test package: https://launchpad.net/~mruffell/+archive/ubuntu/lp2059197-test nfs-common 1:1.3.4-2.5ubuntu3.6+lp2059197v20240411b1 Testcase for this bug, i.e. the regression. $ sudo mount -t nfs4 -o vers=4.2 -vvv jammy-nfs-server:/ /mnt mount.nfs4: timeout set for Thu Apr 11

[Bug 2059197] Re: mount.nfs: Fix minor version parsing when '-t nfs4' and '-o vers=4.x' are used together

2024-04-10 Thread Matthew Ruffell
Attached is a new debdiff which includes the missing patch. I did another look upstream, but I think we are okay now. ** Patch added: "Debdiff for nfs-utils on Focal V2" https://bugs.launchpad.net/ubuntu/+source/nfs-utils/+bug/2059197/+attachment/5763564/+files/lp2059197_focal_V2.debdiff **

[Bug 2059197] Re: mount.nfs: Fix minor version parsing when '-t nfs4' and '-o vers=4.x' are used together

2024-04-10 Thread Matthew Ruffell
** Patch removed: "Debdiff for nfs-utils on Focal" https://bugs.launchpad.net/ubuntu/+source/nfs-utils/+bug/2059197/+attachment/5761398/+files/lp2059197_focal.debdiff -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu.

[Bug 2059197] Re: mount.nfs: Fix minor version parsing when '-t nfs4' and '-o vers=4.x' are used together

2024-04-10 Thread Heitor Alves de Siqueira
Hi Matthew, thanks for the quick follow-up on the regression! Being the sponsor for the original patch in bug 2049262, I wanted to give this one some deeper attention. Version parsing seems to have been a difficult area upstream, with several follow-up fixes indeed, so thank you for detailing the

[Bug 2059197] Re: mount.nfs: Fix minor version parsing when '-t nfs4' and '-o vers=4.x' are used together

2024-04-03 Thread Jeff
Hi Matthew, Its no problem, I can see how changing the version negotiation would be tricky, thanks for looking at it so quickly. Using the workaround and everything working correctly, no dramas Thanks again, Jeffrey -- You received this bug notification because you are a member of Ubuntu

[Bug 2059197] Re: mount.nfs: Fix minor version parsing when '-t nfs4' and '-o vers=4.x' are used together

2024-04-02 Thread Matthew Ruffell
Hi Jeff, Again, I apologise for introducing this regression, but we are on our way to getting this fixed. The fixes are: commit 1e3ca7fe199531a372bb8e1c1033d8973097be50 Author: NeilBrown Date: Wed Jul 26 13:47:42 2017 -0400 Subject: mount: move handling of "-t nfs4" into nfs_nfs_version()