mbox series

[pynfs,v2,0/7] pynfs: add CB_GETATTR tests and tests for delegated timestamps

Message ID 20241014-cb_getattr-v2-0-3782e0d7c598@kernel.org (mailing list archive)
Headers show
Series pynfs: add CB_GETATTR tests and tests for delegated timestamps | expand

Message

Jeff Layton Oct. 14, 2024, 8:50 p.m. UTC
I sent these a month or so ago, but Calum was on PTO. Sending again,
with some additions.

This patchset adds a couple of CB_GETATTR tests, and then updates them
to also test delegated mtime support. There is also a patch to make
the nfsv4.1 tests default to minorversion 2

Signed-off-by: Jeff Layton <jlayton@kernel.org>
---
Changes in v2:
- check timestamps in WRT18, and pass_warn if they don't change
- have v4.1 tests default to minorversion 2
- have DELEG2 open the file r/w
- add support for the "delstid" draft symbols
- test delegated timestamps in new CB_GETATTR tests
- Link to v1: https://lore.kernel.org/r/20240905-cb_getattr-v1-0-0af05c68234f@kernel.org

---
Jeff Layton (7):
      WRT18: have it also check the ctime between writes
      DELEG2: fix write delegation test to open the file RW
      pynfs: update maintainer info
      nfs4.1: add two CB_GETATTR tests
      nfs4.1: default to minorversion 2
      nfs4.1: add support for the "delstid" draft
      st_deleg: test delegated timestamps in CB_GETATTR

 CONTRIBUTING                          |   6 +-
 nfs4.0/servertests/st_write.py        |  28 ++++++---
 nfs4.1/nfs4client.py                  |   8 ++-
 nfs4.1/nfs4lib.py                     |   3 +
 nfs4.1/server41tests/environment.py   |   3 +
 nfs4.1/server41tests/st_delegation.py | 102 ++++++++++++++++++++++++++++++-
 nfs4.1/testserver.py                  |   2 +-
 nfs4.1/xdrdef/nfs4.x                  | 111 ++++++++++++++++++++++++++++++++--
 8 files changed, 242 insertions(+), 21 deletions(-)
---
base-commit: c75f65983498a3254e3970da86eb6954415cac01
change-id: 20240905-cb_getattr-8db184a5b4bf

Best regards,

Comments

Calum Mackay Oct. 14, 2024, 9:24 p.m. UTC | #1
thanks very much Jeff, and for the reminder. I'll look at these this week.

cheers,
c.

On 14/10/2024 9:50 pm, Jeff Layton wrote:
> I sent these a month or so ago, but Calum was on PTO. Sending again,
> with some additions.
> 
> This patchset adds a couple of CB_GETATTR tests, and then updates them
> to also test delegated mtime support. There is also a patch to make
> the nfsv4.1 tests default to minorversion 2
> 
> Signed-off-by: Jeff Layton <jlayton@kernel.org>
> ---
> Changes in v2:
> - check timestamps in WRT18, and pass_warn if they don't change
> - have v4.1 tests default to minorversion 2
> - have DELEG2 open the file r/w
> - add support for the "delstid" draft symbols
> - test delegated timestamps in new CB_GETATTR tests
> - Link to v1: https://lore.kernel.org/r/20240905-cb_getattr-v1-0-0af05c68234f@kernel.org
> 
> ---
> Jeff Layton (7):
>        WRT18: have it also check the ctime between writes
>        DELEG2: fix write delegation test to open the file RW
>        pynfs: update maintainer info
>        nfs4.1: add two CB_GETATTR tests
>        nfs4.1: default to minorversion 2
>        nfs4.1: add support for the "delstid" draft
>        st_deleg: test delegated timestamps in CB_GETATTR
> 
>   CONTRIBUTING                          |   6 +-
>   nfs4.0/servertests/st_write.py        |  28 ++++++---
>   nfs4.1/nfs4client.py                  |   8 ++-
>   nfs4.1/nfs4lib.py                     |   3 +
>   nfs4.1/server41tests/environment.py   |   3 +
>   nfs4.1/server41tests/st_delegation.py | 102 ++++++++++++++++++++++++++++++-
>   nfs4.1/testserver.py                  |   2 +-
>   nfs4.1/xdrdef/nfs4.x                  | 111 ++++++++++++++++++++++++++++++++--
>   8 files changed, 242 insertions(+), 21 deletions(-)
> ---
> base-commit: c75f65983498a3254e3970da86eb6954415cac01
> change-id: 20240905-cb_getattr-8db184a5b4bf
> 
> Best regards,
Calum Mackay Oct. 22, 2024, 9:56 p.m. UTC | #2
On 14/10/2024 9:50 pm, Jeff Layton wrote:
> I sent these a month or so ago, but Calum was on PTO. Sending again,
> with some additions.
> 
> This patchset adds a couple of CB_GETATTR tests, and then updates them
> to also test delegated mtime support. There is also a patch to make
> the nfsv4.1 tests default to minorversion 2

Applied; apologies for the delay.

Thanks very much, Jeff.

cheers,
calum.

> 
> Signed-off-by: Jeff Layton <jlayton@kernel.org>
> ---
> Changes in v2:
> - check timestamps in WRT18, and pass_warn if they don't change
> - have v4.1 tests default to minorversion 2
> - have DELEG2 open the file r/w
> - add support for the "delstid" draft symbols
> - test delegated timestamps in new CB_GETATTR tests
> - Link to v1: https://lore.kernel.org/r/20240905-cb_getattr-v1-0-0af05c68234f@kernel.org
> 
> ---
> Jeff Layton (7):
>        WRT18: have it also check the ctime between writes
>        DELEG2: fix write delegation test to open the file RW
>        pynfs: update maintainer info
>        nfs4.1: add two CB_GETATTR tests
>        nfs4.1: default to minorversion 2
>        nfs4.1: add support for the "delstid" draft
>        st_deleg: test delegated timestamps in CB_GETATTR
> 
>   CONTRIBUTING                          |   6 +-
>   nfs4.0/servertests/st_write.py        |  28 ++++++---
>   nfs4.1/nfs4client.py                  |   8 ++-
>   nfs4.1/nfs4lib.py                     |   3 +
>   nfs4.1/server41tests/environment.py   |   3 +
>   nfs4.1/server41tests/st_delegation.py | 102 ++++++++++++++++++++++++++++++-
>   nfs4.1/testserver.py                  |   2 +-
>   nfs4.1/xdrdef/nfs4.x                  | 111 ++++++++++++++++++++++++++++++++--
>   8 files changed, 242 insertions(+), 21 deletions(-)
> ---
> base-commit: c75f65983498a3254e3970da86eb6954415cac01
> change-id: 20240905-cb_getattr-8db184a5b4bf
> 
> Best regards,