[pynfs,python3,0/7] Python3 patches for st_flex.py

Message

Thomas Haynes Dec. 19, 2020, 6:29 p.m. UTC

From: Tom Haynes <[e-mail]>

Hey Bruce,

This applies on top of the previous patch set.

This passes on my RHEL 8.2 client running python3 against a Hammerspace
server.

Thanks,
Tom

fwiw - In some of the error paths, the system would complain that the
exception StandardError was not defined.

I found this:
https://portingguide.readthedocs.io/en/latest/exceptions.html#the-removed-standarderror

Tom Haynes (7):
  CB_LAYOUTRECALL: Make string a byte array
  st_flex: Use NFS4_MAXFILELEN in layout calls
  st_flex: Provide an empty ff_layoutreturn4 by default for LAYOUTRETURN
  st_flex: Use range instead of xrange for python3
  st_flex: Test is now redundant
  st_flex: Return the layout before closing the file
  st_flex: testFlexLayoutStatsSmall needed loving to pass python3

 nfs4.1/nfs4client.py            |   2 +-
 nfs4.1/server41tests/st_flex.py | 124 +++++++++++++++++---------------
 2 files changed, 68 insertions(+), 58 deletions(-)

Comments

J. Bruce Fields Dec. 21, 2020, 8 p.m. UTC | #1

On Sat, Dec 19, 2020 at 10:29:41AM -0800, Tom Haynes wrote:
> From: Tom Haynes <[e-mail]>
> 
> Hey Bruce,
> 
> This applies on top of the previous patch set.
> 
> This passes on my RHEL 8.2 client running python3 against a Hammerspace
> server.

Thanks, and thanks to Tigran for testing; applied.--b.

> 
> Thanks,
> Tom
> 
> fwiw - In some of the error paths, the system would complain that the
> exception StandardError was not defined.
> 
> I found this:
> https://portingguide.readthedocs.io/en/latest/exceptions.html#the-removed-standarderror
> 
> Tom Haynes (7):
>   CB_LAYOUTRECALL: Make string a byte array
>   st_flex: Use NFS4_MAXFILELEN in layout calls
>   st_flex: Provide an empty ff_layoutreturn4 by default for LAYOUTRETURN
>   st_flex: Use range instead of xrange for python3
>   st_flex: Test is now redundant
>   st_flex: Return the layout before closing the file
>   st_flex: testFlexLayoutStatsSmall needed loving to pass python3
> 
>  nfs4.1/nfs4client.py            |   2 +-
>  nfs4.1/server41tests/st_flex.py | 124 +++++++++++++++++---------------
>  2 files changed, 68 insertions(+), 58 deletions(-)
> 
> -- 
> 2.26.2