rapido-linux / rapido

Quickly test Linux kernel changes
GNU Lesser General Public License v2.1
22 stars 22 forks source link

various minor SMB and unionmount-testsuite changes #208

Closed ddiss closed 1 year ago

ddiss commented 1 year ago
The following changes since commit cf361da7f1ba028ea8e03fc4430c0d8ecd7f4c84:

  Merge remote-tracking branch 'ddiss/ntfs3' (2023-03-24 00:16:42 +0100)

are available in the Git repository at:

  https://github.com/ddiss/rapido/ smb_ovl_various

for you to fetch changes up to 09ffb601113031fd379c9163b54e92a58755a890:

  unionmount_testsuite: save test output to file (2023-04-19 16:01:10 +0200)

----------------------------------------------------------------
David Disseldorp (6):
      runtime.vars: call _fail() instead of _fatal()
      autorun/ksmbd-btrfs: handle /usr/local prefixed paths
      samba-local: configure Btrfs backed shares
      kernel: enable CONFIG_CIFS_DFS_UPCALL
      unionmount-testsuite: fix python deps and remove shutdown
      unionmount_testsuite: save test output to file

 autorun/ksmbd_btrfs.sh          |  4 +++-
 autorun/samba_local.sh          | 23 +++++++----------------
 autorun/unionmount_testsuite.sh | 15 ++-------------
 cut/cifs.sh                     |  2 +-
 cut/fstests_cifs.sh             |  2 +-
 cut/samba_local.sh              |  2 +-
 cut/unionmount_testsuite.sh     | 26 +++++++++++---------------
 kernel/sle12sp3_config          |  2 +-
 kernel/sle15_config             |  2 +-
 kernel/sle15sp1_config          |  2 +-
 kernel/sle15sp2_config          |  2 +-
 kernel/sle15sp3_config          |  2 +-
 kernel/sle15sp4_config          |  2 +-
 kernel/vanilla_config           |  2 +-
 runtime.vars                    |  2 +-
 15 files changed, 34 insertions(+), 56 deletions(-)
pevik commented 1 year ago

I consider loosing XFS in "samba-local: configure Btrfs backed shares" as not optimal (IMHO support was there quite long time, but ack (I suppose nobody will complain or just stick with older git version).

ddiss commented 1 year ago

I consider loosing XFS in "samba-local: configure Btrfs backed shares" as not optimal (IMHO support was there quite long time, but ack (I suppose nobody will complain or just stick with older git version).

Fair enough. In that case I'll add a new samba-xfs runner, as proposed in the commit message. Thanks a lot for the review Petr!

ddiss commented 1 year ago

I consider loosing XFS in "samba-local: configure Btrfs backed shares" as not optimal (IMHO support was there quite long time, but ack (I suppose nobody will complain or just stick with older git version).

Fair enough. In that case I'll add a new samba-xfs runner, as proposed in the commit message. Thanks a lot for the review Petr!

I've done this and renamed samba-local to samba-btrfs (with an alias for the old name via cut symlink). All of your other points should also be addressed in this version.