From: Venky Shankar <vshankar@redhat.com>
Date: Mon, 13 Mar 2023 06:25:18 +0000 (+0530)
Subject: qa, cleanup: no need to specify human readable quota values in comment
X-Git-Tag: v18.2.4~417^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=0f4bad63f76d5b4645e5f190f17e4f9c4c573671;p=ceph.git

qa, cleanup: no need to specify human readable quota values in comment

... since the values themselves are self-explanatory.

Signed-off-by: Venky Shankar <vshankar@redhat.com>
(cherry picked from commit 3286d478e7733120b07b9ba5147e8dde2da4514f)
---

diff --git a/qa/workunits/fs/quota/quota.sh b/qa/workunits/fs/quota/quota.sh
index 5320c85a34e..ba611d5e52d 100755
--- a/qa/workunits/fs/quota/quota.sh
+++ b/qa/workunits/fs/quota/quota.sh
@@ -29,7 +29,7 @@ mkdir quota-test
 cd quota-test
 
 # bytes
-setfattr . -n ceph.quota.max_bytes -v 100M  # 100MiB
+setfattr . -n ceph.quota.max_bytes -v 100M
 expect_false write_file big 1000     # 1g
 expect_false write_file second 10
 setfattr . -n ceph.quota.max_bytes -v 0
@@ -57,7 +57,7 @@ rm -rf *
 # mix
 mkdir bytes bytes/files
 
-setfattr bytes -n ceph.quota.max_bytes -v 10M   #10MiB
+setfattr bytes -n ceph.quota.max_bytes -v 10M
 setfattr bytes/files -n ceph.quota.max_files -v 5
 dd if=/dev/zero of=bytes/files/1 bs=1M count=4
 dd if=/dev/zero of=bytes/files/2 bs=1M count=4
@@ -78,7 +78,7 @@ rm -rf *
 #mv
 mkdir files limit
 truncate files/file -s 10G
-setfattr limit -n ceph.quota.max_bytes -v 1M #1MiB
+setfattr limit -n ceph.quota.max_bytes -v 1M
 expect_false mv files limit/
 
 
@@ -88,8 +88,8 @@ rm -rf *
 #limit by ancestor
 
 mkdir -p ancestor/p1/p2/parent/p3
-setfattr ancestor -n ceph.quota.max_bytes -v 1M #1MiB
-setfattr ancestor/p1/p2/parent -n ceph.quota.max_bytes -v 1G #1GiB
+setfattr ancestor -n ceph.quota.max_bytes -v 1M
+setfattr ancestor/p1/p2/parent -n ceph.quota.max_bytes -v 1G
 expect_false write_file ancestor/p1/p2/parent/p3/file1 900 #900m
 stat --printf="%n %s\n" ancestor/p1/p2/parent/p3/file1