Whamcloud - gitweb
git://git.whamcloud.com / fs / lustre-release.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (from parent 1: 1b5abf6)
LU-13840 utils: --pool instead of -o with lfs setquota 15/39715/5
authorSergey Cheremencev <sergey.cheremencev@hpe.com>
Mon, 24 Aug 2020 18:28:04 +0000 (21:28 +0300)
committerOleg Drokin <green@whamcloud.com>
Fri, 25 Sep 2020 03:12:46 +0000 (03:12 +0000)
The use of "lfs setquota -o" as the short option for
specifying the quota pool is confusing, because "lfs quota -o"
is the short option for getting quota on an OST. Use only
long option --pool for lfs setquota.

Patch also changes -o with --pool for setquota in
sanity-quota.sh

Change-Id: I18a695dcbbcd668611b1301c48f4e10d354b2686
Signed-off-by: Sergey Cheremencev <sergey.cheremencev@hpe.com>
Reviewed-on: https://review.whamcloud.com/39715
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Tested-by: jenkins <devops@whamcloud.com>
Reviewed-by: Jian Yu <yujian@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lustre/tests/sanity-quota.sh patch | blob | history
lustre/utils/lfs.c patch | blob | history

diff --git a/lustre/tests/sanity-quota.sh b/lustre/tests/sanity-quota.sh
index 9884f51..9a360a0 100755 (executable)
--- a/lustre/tests/sanity-quota.sh
+++ b/lustre/tests/sanity-quota.sh
@@ -671,7 +671,7 @@ test_1b() {
        pool_add_targets $qpool 0 $(($OSTCOUNT - 1)) ||
                error "pool_add_targets failed"
 
-       $LFS setquota -u $TSTUSR -B ${limit}M -o $qpool $DIR ||
+       $LFS setquota -u $TSTUSR -B ${limit}M --pool $qpool $DIR ||
                error "set user quota failed"
 
        # make sure the system is clean
@@ -700,7 +700,7 @@ test_1b() {
        $LFS setquota -g $TSTUSR -b 0 -B ${global_limit}M -i 0 -I 0 $DIR ||
                error "set group quota failed"
 
-       $LFS setquota -g $TSTUSR -b 0 -B ${limit}M -o $qpool $DIR ||
+       $LFS setquota -g $TSTUSR -b 0 -B ${limit}M --pool $qpool $DIR ||
                error "set group quota failed"
 
        testfile="$DIR/$tdir/$tfile-1"
@@ -739,7 +739,7 @@ test_1b() {
        $LFS setquota -p $TSTPRJID -b 0 -B ${global_limit}M -i 0 -I 0 $DIR ||
                error "set project quota failed"
 
-       $LFS setquota -p $TSTPRJID -b 0 -B ${limit}M -o $qpool $DIR ||
+       $LFS setquota -p $TSTPRJID -b 0 -B ${limit}M --pool $qpool $DIR ||
                error "set project quota failed"
 
 
@@ -786,10 +786,10 @@ test_1c() {
 
        # create pools without hard limit
        # initially such case raised several bugs
-       $LFS setquota -u $TSTUSR -B 0M -o $qpool1 $DIR ||
+       $LFS setquota -u $TSTUSR -B 0M --pool $qpool1 $DIR ||
                error "set user quota failed"
 
-       $LFS setquota -u $TSTUSR -B 0M -o $qpool2 $DIR ||
+       $LFS setquota -u $TSTUSR -B 0M --pool $qpool2 $DIR ||
                error "set user quota failed"
 
        # make sure the system is clean
@@ -848,10 +848,10 @@ test_1d() {
        pool_add_targets $qpool2 0 $(($OSTCOUNT - 1)) ||
                error "pool_add_targets failed"
 
-       $LFS setquota -u $TSTUSR -B ${limit1}M -o $qpool1 $DIR ||
+       $LFS setquota -u $TSTUSR -B ${limit1}M --pool $qpool1 $DIR ||
                error "set user quota failed"
 
-       $LFS setquota -u $TSTUSR -B ${limit2}M -o $qpool2 $DIR ||
+       $LFS setquota -u $TSTUSR -B ${limit2}M --pool $qpool2 $DIR ||
        error "set user quota failed"
 
        # make sure the system is clean
@@ -908,7 +908,7 @@ test_1e() {
        pool_add_targets $qpool1 1 1 ||
                error "pool_add_targets failed"
 
-       $LFS setquota -u $TSTUSR -B ${limit1}M -o $qpool1 $DIR ||
+       $LFS setquota -u $TSTUSR -B ${limit1}M --pool $qpool1 $DIR ||
                error "set user quota failed"
 
        # make sure the system is clean
@@ -966,7 +966,7 @@ test_1f() {
        pool_add_targets $qpool1 0 0 ||
                error "pool_add_targets failed"
 
-       $LFS setquota -u $TSTUSR -B ${limit1}M -o $qpool1 $DIR ||
+       $LFS setquota -u $TSTUSR -B ${limit1}M --pool $qpool1 $DIR ||
                error "set user quota failed"
 
        # make sure the system is clean
@@ -1025,7 +1025,7 @@ test_1g() {
        pool_add_targets $qpool 0 $(($OSTCOUNT - 1)) ||
                error "pool_add_targets failed"
 
-       $LFS setquota -u $TSTUSR -B ${limit}M -o $qpool $DIR ||
+       $LFS setquota -u $TSTUSR -B ${limit}M --pool $qpool $DIR ||
                error "set user quota failed"
 
        # make sure the system is clean
@@ -1365,11 +1365,11 @@ test_3b() {
        $LFS setquota -t -u --block-grace $glbl_grace --inode-grace \
                $MAX_IQ_TIME $DIR || error "set user grace time failed"
        $LFS setquota -t -u --block-grace $grace \
-               -o $qpool $DIR || error "set user grace time failed"
+               --pool $qpool $DIR || error "set user grace time failed"
 
        $LFS setquota -u $TSTUSR -b ${glbl_limit}M -B 0 -i 0 -I 0 $DIR ||
                error "set user quota failed"
-       $LFS setquota -u $TSTUSR -b ${limit}M -B 0 -o $qpool $DIR ||
+       $LFS setquota -u $TSTUSR -b ${limit}M -B 0 --pool $qpool $DIR ||
                error "set user quota failed"
 
        test_block_soft $testfile $grace $limit "u" $qpool
@@ -1383,11 +1383,11 @@ test_3b() {
        $LFS setquota -t -g --block-grace $glbl_grace --inode-grace \
                $MAX_IQ_TIME $DIR || error "set group grace time failed"
        $LFS setquota -t -g --block-grace $grace \
-               -o $qpool $DIR || error "set group grace time failed"
+               --pool $qpool $DIR || error "set group grace time failed"
 
        $LFS setquota -g $TSTUSR -b ${glbl_limit}M -B 0 -i 0 -I 0 $DIR ||
                error "set group quota failed"
-       $LFS setquota -g $TSTUSR -b ${limit}M -B 0 -o $qpool $DIR ||
+       $LFS setquota -g $TSTUSR -b ${limit}M -B 0 --pool $qpool $DIR ||
                error "set group quota failed"
 
        test_block_soft $testfile $grace $limit "g" $qpool
@@ -1404,31 +1404,32 @@ test_3b() {
                        $MAX_IQ_TIME $DIR ||
                                error "set project grace time failed"
                $LFS setquota -t -p --block-grace $grace \
-                       -o $qpool $DIR || error "set project grace time failed"
+                       --pool $qpool $DIR ||
+                               error "set project grace time failed"
 
                $LFS setquota -p $TSTPRJID -b ${glbl_limit}M -B 0 -i 0 -I 0 \
                        $DIR || error "set project quota failed"
-               $LFS setquota -p $TSTPRJID -b ${limit}M -B 0 -o $qpool $DIR ||
-                       error "set project quota failed"
+               $LFS setquota -p $TSTPRJID -b ${limit}M -B 0 \
+                       --pool $qpool $DIR || error "set project quota failed"
 
                test_block_soft $testfile $grace $limit "p" $qpool
                resetquota -p $TSTPRJID
                $LFS setquota -t -p --block-grace $MAX_DQ_TIME --inode-grace \
                        $MAX_IQ_TIME $DIR ||
                                error "restore project grace time failed"
-               $LFS setquota -t -p --block-grace $MAX_DQ_TIME -o $qpool $DIR ||
-                       error "set project grace time failed"
+               $LFS setquota -t -p --block-grace $MAX_DQ_TIME --pool $qpool \
+                       $DIR || error "set project grace time failed"
        fi
 
        # cleanup
        $LFS setquota -t -u --block-grace $MAX_DQ_TIME --inode-grace \
                $MAX_IQ_TIME $DIR || error "restore user grace time failed"
        $LFS setquota -t -u --block-grace $MAX_DQ_TIME \
-               -o $qpool $DIR || error "restore user grace time failed"
+               --pool $qpool $DIR || error "restore user grace time failed"
        $LFS setquota -t -g --block-grace $MAX_DQ_TIME --inode-grace \
                $MAX_IQ_TIME $DIR || error "restore group grace time failed"
        $LFS setquota -t -g --block-grace $MAX_DQ_TIME \
-               -o $qpool $DIR || error "restore group grace time failed"
+               --pool $qpool $DIR || error "restore group grace time failed"
 }
 run_test 3b "Quota pools: Block soft limit (start timer, expires, stop timer)"
 
@@ -1472,19 +1473,19 @@ test_3c() {
        $LFS setquota -t -u --block-grace $glbl_grace --inode-grace \
                $MAX_IQ_TIME $DIR || error "set user grace time failed"
        $LFS setquota -t -u --block-grace $grace1 \
-               -o $qpool $DIR || error "set user grace time failed"
+               --pool $qpool $DIR || error "set user grace time failed"
        $LFS setquota -t -u --block-grace $grace2 \
-               -o $qpool2 $DIR || error "set user grace time failed"
+               --pool $qpool2 $DIR || error "set user grace time failed"
 
        $LFS setquota -u $TSTUSR -b ${glbl_limit}M -B 0 -i 0 -I 0 $DIR ||
                error "set user quota failed"
-       $LFS setquota -u $TSTUSR -b ${limit}M -B 0 -o $qpool $DIR ||
+       $LFS setquota -u $TSTUSR -b ${limit}M -B 0 --pool $qpool $DIR ||
                error "set user quota failed"
        # qpool has minimum soft limit, but it's grace is grater than
        # grace period of qpool2. Thus write shouldn't fail when
        # hit qpool soft limit - only when reaches up qpool2 limit
        # after grace2 seconds.
-       $LFS setquota -u $TSTUSR -b ${limit2}M -B 0 -o $qpool2 $DIR ||
+       $LFS setquota -u $TSTUSR -b ${limit2}M -B 0 --pool $qpool2 $DIR ||
                error "set user quota failed"
 
        test_block_soft $testfile $grace2 $limit2 "u" $qpool2
@@ -1493,9 +1494,9 @@ test_3c() {
        $LFS setquota -t -u --block-grace $MAX_DQ_TIME --inode-grace \
                $MAX_IQ_TIME $DIR || error "restore user grace time failed"
        $LFS setquota -t -u --block-grace $MAX_DQ_TIME \
-               -o $qpool $DIR || error "restore user grace time failed"
+               --pool $qpool $DIR || error "restore user grace time failed"
        $LFS setquota -t -u --block-grace $MAX_DQ_TIME \
-               -o $qpool2 $DIR || error "restore user grace time failed"
+               --pool $qpool2 $DIR || error "restore user grace time failed"
 }
 run_test 3c "Quota pools: check block soft limit on different pools"
 
@@ -4400,7 +4401,7 @@ test_69()
                error "set user quota failed"
 
        log "User quota (block hardlimit:$limit MB)"
-       $LFS setquota -u $TSTUSR -B ${limit}M -o $qpool $DIR ||
+       $LFS setquota -u $TSTUSR -B ${limit}M --pool $qpool $DIR ||
                error "set user quota failed"
 
        $RUNAS dd if=/dev/zero of="$dom0/f1" bs=1K count=512 oflag=sync ||
@@ -4452,7 +4453,7 @@ test_70()
        pool_add $qpool || error "pool_add failed"
        pool_add_targets $qpool 0 0 || error "pool_add_targets failed"
 
-       $LFS setquota -u $TSTUSR -B ${limit}M -o $qpool $DIR
+       $LFS setquota -u $TSTUSR -B ${limit}M --pool $qpool $DIR
        rc=$?
        [ $rc -eq $err ] || error "setquota res $rc != $err"
 
@@ -4499,14 +4500,14 @@ test_71a()
        pool_add_targets $qpool 0 1 ||
                error "pool_add_targets failed"
 
-       $LFS setquota -u $TSTUSR -B ${limit}M -o $qpool $DIR ||
+       $LFS setquota -u $TSTUSR -B ${limit}M --pool $qpool $DIR ||
                error "set user quota failed"
 
        pool_add $qpool2 || error "pool_add failed"
        pool_add_targets $qpool2 1 1 ||
                error "pool_add_targets failed"
 
-       $LFS setquota -u $TSTUSR -B ${limit}M -o $qpool2 $DIR ||
+       $LFS setquota -u $TSTUSR -B ${limit}M --pool $qpool2 $DIR ||
                error "set user quota failed"
 
        # make sure the system is clean
@@ -4584,14 +4585,14 @@ test_71b()
        pool_add_targets $qpool 0 1 ||
                error "pool_add_targets failed"
 
-       $LFS setquota -u $TSTUSR -B ${limit1}M -o $qpool $DIR ||
+       $LFS setquota -u $TSTUSR -B ${limit1}M --pool $qpool $DIR ||
                error "set user quota failed"
 
        pool_add $qpool2 || error "pool_add failed"
        pool_add_targets $qpool2 1 1 ||
                error "pool_add_targets failed"
 
-       $LFS setquota -u $TSTUSR -B ${limit2}M -o $qpool2 $DIR ||
+       $LFS setquota -u $TSTUSR -B ${limit2}M --pool $qpool2 $DIR ||
                error "set user quota failed"
 
        # make sure the system is clean
@@ -4645,7 +4646,7 @@ test_72()
        pool_add $qpool || error "pool_add failed"
        pool_add_targets $qpool 1 1 || error "pool_add_targets failed"
 
-       $LFS setquota -u $TSTUSR -B ${limit}M -o $qpool $DIR ||
+       $LFS setquota -u $TSTUSR -B ${limit}M --pool $qpool $DIR ||
                error "set user quota failed"
 
        # make sure the system is clean
diff --git a/lustre/utils/lfs.c b/lustre/utils/lfs.c
index 5f3555f..99539b2 100644 (file)
--- a/lustre/utils/lfs.c
+++ b/lustre/utils/lfs.c
@@ -6738,14 +6738,15 @@ int lfs_setquota_times(int argc, char **argv, struct if_quotactl *qctl)
        { .val = 'p',   .name = "projid",       .has_arg = no_argument },
        { .val = 't',   .name = "times",        .has_arg = no_argument },
        { .val = 'u',   .name = "user",         .has_arg = no_argument },
-       { .val = 'o',   .name = "pool",         .has_arg = required_argument },
+       { .val = LFS_POOL_OPT,
+                       .name = "pool",         .has_arg = required_argument },
        { .name = NULL } };
        int qtype;
 
        qctl->qc_cmd  = LUSTRE_Q_SETINFO;
        qctl->qc_type = ALLQUOTA;
 
-       while ((c = getopt_long(argc, argv, "b:gi:ptuo:",
+       while ((c = getopt_long(argc, argv, "b:gi:ptu",
                                long_opts, NULL)) != -1) {
                switch (c) {
                case 'u':
@@ -6796,7 +6797,7 @@ quota_type:
                        break;
                case 't': /* Yes, of course! */
                        break;
-               case 'o':
+               case LFS_POOL_OPT:
                        if (lfs_verify_poolarg(optarg))
                                return -1;
                        fprintf(stdout,
@@ -6860,7 +6861,8 @@ int lfs_setquota(int argc, char **argv)
        { .val = 'P',   .name = "default-prj",  .has_arg = no_argument },
        { .val = 'u',   .name = "user",         .has_arg = required_argument },
        { .val = 'U',   .name = "default-usr",  .has_arg = no_argument },
-       { .val = 'o',   .name = "pool",         .has_arg = required_argument },
+       { .val = LFS_POOL_OPT,
+                       .name = "pool",         .has_arg = required_argument },
        { .name = NULL } };
        unsigned int limit_mask = 0;
        bool use_default = false;
@@ -6886,7 +6888,7 @@ int lfs_setquota(int argc, char **argv)
                                   * isn't reinitialized from command line
                                   */
 
-       while ((c = getopt_long(argc, argv, "b:B:dg:Gi:I:p:Pu:Uo:",
+       while ((c = getopt_long(argc, argv, "b:B:dg:Gi:I:p:Pu:U",
                long_opts, NULL)) != -1) {
                switch (c) {
                case 'U':
@@ -6995,7 +6997,7 @@ quota_type_def:
                                        (unsigned long long)dqb->dqb_ihardlimit,
                                        progname);
                        break;
-               case 'o':
+               case LFS_POOL_OPT:
                        if (lfs_verify_poolarg(optarg)) {
                                rc = -1;
                                goto out;
@@ -7712,7 +7714,7 @@ static int lfs_quota(int argc, char **argv)
        bool show_default = false;
        int qtype;
        struct option long_opts[] = {
-       { .val = 1,     .name = "pool", .has_arg = required_argument },
+       { .val = LFS_POOL_OPT, .name = "pool", .has_arg = required_argument },
        { .name = NULL } };
 
        qctl = calloc(1, sizeof(*qctl) + LOV_MAXPOOLNAME + 1);
@@ -7788,7 +7790,7 @@ quota_type:
                case 'h':
                        human_readable = true;
                        break;
-               case 1:
+               case LFS_POOL_OPT:
                        if (lfs_verify_poolarg(optarg)) {
                                rc = -1;
                                goto out;
Lustre primary development and releases