diff mbox series

tests: check quota file space usage does not get accounted

Message ID 20210804121652.25833-1-jack@suse.cz
State Accepted
Headers show
Series tests: check quota file space usage does not get accounted | expand

Commit Message

Jan Kara Aug. 4, 2021, 12:16 p.m. UTC 
Check that space used by quota files themselves does not get accounted
into the space tracked by quota subsystem.

Signed-off-by: Jan Kara <jack@suse.cz>
---
 tests/t_quota_add/name   |  1 +
 tests/t_quota_add/script | 46 ++++++++++++++++++++++++++++++++++++++++
 2 files changed, 47 insertions(+)
 create mode 100644 tests/t_quota_add/name
 create mode 100644 tests/t_quota_add/script

Comments

Theodore Ts'o Aug. 13, 2021, 3:02 p.m. UTC | #1 
On Wed, 4 Aug 2021 14:16:52 +0200, Jan Kara wrote:
> Check that space used by quota files themselves does not get accounted
> into the space tracked by quota subsystem.

Applied, thanks!

[1/1] tests: check quota file space usage does not get accounted
      commit: 958b95eb0b37de5e3a52cb7abbfdfeed79635727

Best regards,
diff mbox series

Patch

diff --git a/tests/t_quota_add/name b/tests/t_quota_add/name
new file mode 100644
index 000000000000..43a4bfd084db
--- /dev/null
+++ b/tests/t_quota_add/name
@@ -0,0 +1 @@ 
+add several quota types using tune2fs and check computed usage
diff --git a/tests/t_quota_add/script b/tests/t_quota_add/script
new file mode 100644
index 000000000000..c26c37a7b814
--- /dev/null
+++ b/tests/t_quota_add/script
@@ -0,0 +1,46 @@ 
+FSCK_OPT=-yf
+
+if [ "$QUOTA" != "y" ]; then
+	echo "$test_name: $test_description: skipped"
+	return 0
+fi
+
+$MKE2FS -q -F -o Linux -b 4096 -O quota -E quotatype=prjquota $TMPFILE 10000 > $test_name.log 2>&1
+status=$?
+if [ "$status" != 0 ] ; then
+	echo "mke2fs failed" > $test_name.failed
+	echo "$test_name: $test_description: failed"
+	return $status
+fi
+
+for type in usrquota grpquota; do
+	$TUNE2FS -Q $type $TMPFILE >> $test_name.log 2>&1
+	status=$?
+	if [ "$status" != 0 ] ; then
+		echo "tune2fs -O quota failed with $status" > $test_name.failed
+		echo "$test_name: $test_description: failed"
+		return $status
+	fi
+done
+
+UUSAGE=$($DEBUGFS 2>/dev/null -R "lq user" $TMPFILE | grep "^ *0 ")
+for type in group project; do
+	TUSAGE=$($DEBUGFS 2>/dev/null -R "lq $type" $TMPFILE | grep "^ *0 ")
+	if [ "$TUSAGE" != "$UUSAGE" ]; then
+		echo "user and $type quota entries are different" >$test_name.failed
+		echo "$test_name: $test_description: failed"
+		return 1
+	fi
+done
+
+$FSCK $FSCK_OPT $TMPFILE >> $test_name.log 2>&1
+status=$?
+if [ "$status" = 0 ] ; then
+	echo "$test_name: $test_description: ok"
+	touch $test_name.ok
+else
+	echo "e2fsck with quota enabled failed with $status" > $test_name.failed
+	echo "$test_name: $test_description: failed"
+	return $status
+fi
+rm -f $TMPFILE