diff mbox series

generic/020: add ceph-fuse support

Message ID 20230530071552.766424-1-xiubli@redhat.com (mailing list archive)
State New, archived
Headers show
Series generic/020: add ceph-fuse support | expand

Commit Message

Xiubo Li May 30, 2023, 7:15 a.m. UTC
From: Xiubo Li <xiubli@redhat.com>

For ceph fuse client the fs type will be "ceph-fuse".

Fixes: https://tracker.ceph.com/issues/61496
Signed-off-by: Xiubo Li <xiubli@redhat.com>
---
 tests/generic/020 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Zorro Lang June 2, 2023, 10:50 a.m. UTC | #1
On Tue, May 30, 2023 at 03:15:52PM +0800, xiubli@redhat.com wrote:
> From: Xiubo Li <xiubli@redhat.com>
> 
> For ceph fuse client the fs type will be "ceph-fuse".
> 
> Fixes: https://tracker.ceph.com/issues/61496
> Signed-off-by: Xiubo Li <xiubli@redhat.com>
> ---
>  tests/generic/020 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/generic/020 b/tests/generic/020
> index e00365a9..da258aa5 100755
> --- a/tests/generic/020
> +++ b/tests/generic/020
> @@ -56,7 +56,7 @@ _attr_get_max()

Hmm... generic/020 is a very old test case, I think this _attr_get_max might
can be a common helper in common/attr.

>  {
>  	# set maximum total attr space based on fs type
>  	case "$FSTYP" in
> -	xfs|udf|pvfs2|9p|ceph|fuse|nfs)
> +	xfs|udf|pvfs2|9p|ceph|fuse|nfs|ceph-fuse)

Anyway, that's another story, for the purpose of this patch:

Reviewed-by: Zorro Lang <zlang@redhat.com>

>  		max_attrs=1000
>  		;;
>  	ext2|ext3|ext4)
> -- 
> 2.40.1
>
Xiubo Li June 5, 2023, 1:41 a.m. UTC | #2
On 6/2/23 18:50, Zorro Lang wrote:
> On Tue, May 30, 2023 at 03:15:52PM +0800, xiubli@redhat.com wrote:
>> From: Xiubo Li <xiubli@redhat.com>
>>
>> For ceph fuse client the fs type will be "ceph-fuse".
>>
>> Fixes: https://tracker.ceph.com/issues/61496
>> Signed-off-by: Xiubo Li <xiubli@redhat.com>
>> ---
>>   tests/generic/020 | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/tests/generic/020 b/tests/generic/020
>> index e00365a9..da258aa5 100755
>> --- a/tests/generic/020
>> +++ b/tests/generic/020
>> @@ -56,7 +56,7 @@ _attr_get_max()
> Hmm... generic/020 is a very old test case, I think this _attr_get_max might
> can be a common helper in common/attr.
Yeah, sounds reasonable.
>>   {
>>   	# set maximum total attr space based on fs type
>>   	case "$FSTYP" in
>> -	xfs|udf|pvfs2|9p|ceph|fuse|nfs)
>> +	xfs|udf|pvfs2|9p|ceph|fuse|nfs|ceph-fuse)
> Anyway, that's another story, for the purpose of this patch:
>
> Reviewed-by: Zorro Lang <zlang@redhat.com>
>
Thanks

- Xiubo

>>   		max_attrs=1000
>>   		;;
>>   	ext2|ext3|ext4)
>> -- 
>> 2.40.1
>>
diff mbox series

Patch

diff --git a/tests/generic/020 b/tests/generic/020
index e00365a9..da258aa5 100755
--- a/tests/generic/020
+++ b/tests/generic/020
@@ -56,7 +56,7 @@  _attr_get_max()
 {
 	# set maximum total attr space based on fs type
 	case "$FSTYP" in
-	xfs|udf|pvfs2|9p|ceph|fuse|nfs)
+	xfs|udf|pvfs2|9p|ceph|fuse|nfs|ceph-fuse)
 		max_attrs=1000
 		;;
 	ext2|ext3|ext4)