Attachment 10932 Details for Bug 54537

[patch] bug54537.patch

bug54537.patch (text/plain), 4.53 KB, created by Arvid Requate on 2022-03-28 13:58 CEST

(hide)

Description:

Filename:

MIME Type:

Creator: Arvid Requate

Created: 2022-03-28 13:58 CEST

Size: 4.53 KB

patch

obsolete

>diff --git a/base/univention-quota/files/univention-group-quota b/base/univention-quota/files/univention-group-quota
>index b48e03e0bb..c834046239 100644
>--- a/base/univention-quota/files/univention-group-quota
>+++ b/base/univention-quota/files/univention-group-quota
>@@ -89,13 +89,13 @@ setQuotaForUserOnFilesystem () { # USER FILESYSTEM BSL BHL
> 	local ISL=$(quota -wulv "$USER"|tr -s '\t' ' '|grep "^ *$FILESYSTEM "|cut -d' ' -f6)
> 	local IHL=$(quota -wulv "$USER"|tr -s '\t' ' '|grep "^ *$FILESYSTEM "|cut -d' ' -f7)
> 	verbose "Setting quota $BSL/$BHL ($ISL/$IHL) for $USER on $FILESYSTEM"
>-	setquota -u "$USER" "$BSL" "$BHL" "$ISL" "$IHL" "$FILESYSTEM"
>+	setquota --always-resolve -u "$USER" "$BSL" "$BHL" "$ISL" "$IHL" "$FILESYSTEM"
> }
> 
> userHasQuotaOnFilesystem () { # USER FILESYSTEM
> 	local USER="$1"
> 	local FILESYSTEM="$2"
>-	quota -wul "$USER"|tr -s '\t' ' '|grep -qs "^ *$FILESYSTEM "
>+	quota --always-resolve -wul "$USER"|tr -s '\t' ' '|grep -qs "^ *$FILESYSTEM "
> }
> 
> usage () {
>diff --git a/base/univention-quota/files/univention-setquota b/base/univention-quota/files/univention-setquota
>index 5b393ee40d..8ea51c4d32 100644
>--- a/base/univention-quota/files/univention-setquota
>+++ b/base/univention-quota/files/univention-setquota
>@@ -76,8 +76,8 @@ len=0
> #search for mountpoint
> for mpoint in `cat /etc/mtab | awk '{print $2}'`; do if [ "`echo "$share" | sed -e "s|^$mpoint||"`" != "$share" -a $len -lt `echo $mpoint | wc -m` ]; then len=`echo $mpoint | wc -m`; filesystem=$mpoint; fi; done
> 
>-echo "/usr/sbin/setquota $type $sid $blocksoft $blockhard $inodesoft $inodehard $filesystem" >> $LOGFILE
>-if /usr/sbin/setquota $type $sid $blocksoft $blockhard $inodesoft $inodehard $filesystem >>  /dev/null 2>&1
>+echo "/usr/sbin/setquota --always-resolve $type $sid $blocksoft $blockhard $inodesoft $inodehard $filesystem" >> $LOGFILE
>+if /usr/sbin/setquota --always-resolve $type $sid $blocksoft $blockhard $inodesoft $inodehard $filesystem >>  /dev/null 2>&1
> 	then
> 	echo "univention-setquota: done"
> else
>diff --git a/base/univention-quota/files/univention-user-quota b/base/univention-quota/files/univention-user-quota
>index fe4b85739c..97a11e1ae0 100755
>--- a/base/univention-quota/files/univention-user-quota
>+++ b/base/univention-quota/files/univention-user-quota
>@@ -222,8 +222,8 @@ if __name__ == '__main__':
> 		log.write('Found %i shares and %i mountpoints\n' % (len(shares), len(mountpoints)))
> 		for mountpoint_path, attrs in mountpoints.items():
> 			if attrs['reapplyQuota']:
>-				log.write('Set quota on mountpoint "%s": /usr/sbin/setquota -u %s %s %s %s %s %s\n' % (mountpoint_path, user, attrs['spaceSoftLimit'], attrs['spaceHardLimit'], attrs['inodeSoftLimit'], attrs['inodeHardLimit'], mountpoint_path))
>-				command = ['/usr/sbin/setquota', '-u', user, str(attrs['spaceSoftLimit']), str(attrs['spaceHardLimit']), str(attrs['inodeSoftLimit']), str(attrs['inodeHardLimit']), mountpoint_path]
>+				log.write('Set quota on mountpoint "%s": /usr/sbin/setquota --always-resolve -u %s %s %s %s %s %s\n' % (mountpoint_path, user, attrs['spaceSoftLimit'], attrs['spaceHardLimit'], attrs['inodeSoftLimit'], attrs['inodeHardLimit'], mountpoint_path))
>+				command = ['/usr/sbin/setquota', '--always-resolve', '-u', user, str(attrs['spaceSoftLimit']), str(attrs['spaceHardLimit']), str(attrs['inodeSoftLimit']), str(attrs['inodeHardLimit']), mountpoint_path]
> 				set_quota = subprocess.Popen(command, stderr=log)
> 				set_quota.wait()
> 			else:
>diff --git a/base/univention-quota/test/quotatest b/base/univention-quota/test/quotatest
>index e907116adf..8dbad5e918 100755
>--- a/base/univention-quota/test/quotatest
>+++ b/base/univention-quota/test/quotatest
>@@ -208,7 +208,7 @@ _quota() {
> 	case "$mtype" in
> 		local)
> 			if [[ "$qtype" == "local" ]]; then
>-				setquota -u "$user" "$soft" "$limit" 0 0 "$fs"
>+				setquota --always-resolve -u "$user" "$soft" "$limit" 0 0 "$fs"
> 			fi
> 			__quota "$user" "$fs/$file" "$limit"
> 			rc=$?
>diff --git a/base/univention-quota/umc/python/quota/tools.py b/base/univention-quota/umc/python/quota/tools.py
>index c9fedcfe62..52dd6b6907 100644
>--- a/base/univention-quota/umc/python/quota/tools.py
>+++ b/base/univention-quota/umc/python/quota/tools.py
>@@ -113,7 +113,7 @@ def repquota_parse(partition, output):
> 
> 
> def setquota(partition, user, bsoft, bhard, fsoft, fhard):
>-	return subprocess.call(['/usr/sbin/setquota', '-u', user, str(bsoft), str(bhard), str(fsoft), str(fhard), partition])
>+	return subprocess.call(['/usr/sbin/setquota', '--always-resolve', '-u', user, str(bsoft), str(bhard), str(fsoft), str(fhard), partition])
> 
> 
> class QuotaActivationError(Exception):

Actions: View | Diff

Attachments on bug 54537: 10932 | 10933