Re: [PATCH] test-appliance: add ext4/050 to encrypt.exclude

["Theodore Ts'o" <tytso@mit.edu>]

On Sat, Dec 18, 2021 at 08:21:06PM -0600, Eric Biggers wrote:
> On Fri, Dec 17, 2021 at 11:08:14PM -0500, Theodore Ts'o wrote:
> > The ext4/050 test can't handle encrypted directories, so skip it for
> > now.
> > 
> > Signed-off-by: Theodore Ts'o <tytso@mit.edu>
> > ---
> >  .../test-appliance/files/root/fs/ext4/cfg/encrypt.exclude    | 5 +++++
> >  1 file changed, 5 insertions(+)
> > 
> > diff --git a/kvm-xfstests/test-appliance/files/root/fs/ext4/cfg/encrypt.exclude b/kvm-xfstests/test-appliance/files/root/fs/ext4/cfg/encrypt.exclude
> > index f3c7a959..21a8b45f 100644
> > --- a/kvm-xfstests/test-appliance/files/root/fs/ext4/cfg/encrypt.exclude
> > +++ b/kvm-xfstests/test-appliance/files/root/fs/ext4/cfg/encrypt.exclude
> > @@ -12,6 +12,11 @@ ext4/028
> >  # file systems with encryption enabled can't be mounted with ext3
> >  ext4/044
> >  
> > +# This test to make sure ext4 directory entries are appropriately
> > +# wiped after a file is deleted, or after htree operations is
> > +# incompatible with an encrypted directory.
> > +ext4/048
> 
> The commit message says ext4/050, but the test added to the list is ext4/048.
> Is ext4/048 the one intended?

Yes, it's the commit message that was incorrect.  Thanks for pointing
that out.  I'll fix the commit description in my tree.

     	   	    	       		   - Ted