[LU-14370] sanity-quota test_12b: @@@@@@ FAIL: create failed, but expect success

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.12.8
• Fix Version/s: Lustre 2.15.0
• Type: Bug
• Priority: Minor
• Reporter: Alex Zhuravlev
• Assignee: Alex Zhuravlev
• Resolution: Fixed

Environment

ZFS

• Severity: 3

Description

that seem to due incorrect usage of txg_wait_callbacks() in osd_wait_quota_pending().
notice the following comment in ZFS:

 * Wait for pending commit callbacks of already-synced transactions to finish
 * processing.

i.e. it won't be waiting for transactions that in-sync holding "pending" quota.
I think we should be calling dt_sync() instead

Issue Links

• is related to: LU-14513 sanity-quota test_1g: OST hangs

Comments

[Gerrit Updater]

Alex Zhuravlev ([e-mail]) uploaded a new patch: https://review.whamcloud.com/41322
Subject: LU-14370 quota: use dt_sync() to flush pending writes
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: abcc6507fcdd9993d8ec9a228c6b447a316f23ca

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/41322/
Subject: LU-14370 quota: use dt_sync() to flush pending writes
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: da4684ff1cd918b9ddf4c22f7f97930148c5b78e

[Peter Jones]

Landed for 2.15

[Andreas Dilger]

Seem on master:
https://testing.whamcloud.com/test_sets/a6426d75-9b0a-47c2-84e2-bbeb521bb4c8

[Sergey Cheremencev]

+1 on master(ZFS): https://testing.whamcloud.com/test_sets/d8bf6e4e-70ae-4f35-be5f-cf3febf17cab

[Sergey Cheremencev]

+1 on master (review-dne-zfs-part-4) https://testing.whamcloud.com/test_sets/182923a2-3199-47f3-8de7-52a2e24c2292

Is it a time to open the ticket again?

[Alena Nikitenko]

+1 on b2_12 (full-dne-zfs-part-2) https://testing.whamcloud.com/test_sets/96c30097-9119-47df-a3ab-58781f980762

 

[Andreas Dilger]

+1 on master+zfs: https://testing.whamcloud.com/test_sets/28347c2d-f877-4ccd-8a83-93ccfadef088

Create 2048 files on mdt0...
running as uid/gid/euid/egid 60000/60000/60000/60000, groups:
 [createmany] [-m] [/mnt/lustre/d12b.sanity-quota/f12b.sanity-quota] [2048]
mknod(/mnt/lustre/d12b.sanity-quota/f12b.sanity-quota1024) error: Disk quota exceeded
total: 1024 create in 1.25 seconds: 818.75 ops/second
Disk quotas for usr quota_usr (uid 60000):
     Filesystem  kbytes   quota   limit   grace   files   quota   limit   grace
    /mnt/lustre    1536       0       0       -    1024       0    2048       -
lustre-MDT0000_UUID
                   1536       -       0       -    1024       -    2048       -
lustre-MDT0001_UUID
                      0       -       0       -       0       -       0       -
lustre-MDT0002_UUID
                      0       -       0       -       0       -       0       -
lustre-MDT0003_UUID
                      0       -       0       -       0       -       0       -

It isn't clear where the other inode qunit is?

[Chris Horn]

+1 on master+zfs: https://testing.whamcloud.com/test_sets/31838ca4-97f7-425d-a0b0-77b2a41e09e2

[Etienne Aujames]

+1 on master+zfs: https://testing.whamcloud.com/test_sets/ad37d580-a78c-4aed-8365-026feaaa38ee