ceph-devel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] generic/020: add ceph-fuse support
@ 2023-05-30  7:15 xiubli
  2023-06-02 10:50 ` Zorro Lang
  0 siblings, 1 reply; 3+ messages in thread
From: xiubli @ 2023-05-30  7:15 UTC (permalink / raw)
  To: fstests; +Cc: zlang, david, djwong, ceph-devel, Xiubo Li

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()
 {
 	# 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)
-- 
2.40.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] generic/020: add ceph-fuse support
  2023-05-30  7:15 [PATCH] generic/020: add ceph-fuse support xiubli
@ 2023-06-02 10:50 ` Zorro Lang
  2023-06-05  1:41   ` Xiubo Li
  0 siblings, 1 reply; 3+ messages in thread
From: Zorro Lang @ 2023-06-02 10:50 UTC (permalink / raw)
  To: xiubli; +Cc: fstests, ceph-devel

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
> 


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] generic/020: add ceph-fuse support
  2023-06-02 10:50 ` Zorro Lang
@ 2023-06-05  1:41   ` Xiubo Li
  0 siblings, 0 replies; 3+ messages in thread
From: Xiubo Li @ 2023-06-05  1:41 UTC (permalink / raw)
  To: Zorro Lang; +Cc: fstests, ceph-devel


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
>>


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-06-05  1:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-05-30  7:15 [PATCH] generic/020: add ceph-fuse support xiubli
2023-06-02 10:50 ` Zorro Lang
2023-06-05  1:41   ` Xiubo Li

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).