Project

General

Profile

Actions

Bug #59346

open

qa/workunits/fs/test_python.sh failed with "AssertionError: DiskQuotaExceeded not raised by write"

Added by Xiubo Li about 1 year ago. Updated 12 days ago.

Status:
Fix Under Review
Priority:
Normal
Assignee:
Category:
-
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
quincy reef squid
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
Labels (FS):
qa
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

https://pulpito.ceph.com/yuriw-2023-04-05_14:51:06-fs-wip-yuri5-testing-2023-04-04-0814-distro-default-smithi/7232889/

2023-04-05T21:50:42.347 INFO:tasks.workunit.client.0.smithi101.stderr:======================================================================
2023-04-05T21:50:42.347 INFO:tasks.workunit.client.0.smithi101.stderr:FAIL: test_cephfs.test_disk_quota_exceeeded_error
2023-04-05T21:50:42.347 INFO:tasks.workunit.client.0.smithi101.stderr:----------------------------------------------------------------------
2023-04-05T21:50:42.348 INFO:tasks.workunit.client.0.smithi101.stderr:Traceback (most recent call last):
2023-04-05T21:50:42.348 INFO:tasks.workunit.client.0.smithi101.stderr:  File "/usr/lib/python3/dist-packages/nose/case.py", line 197, in runTest
2023-04-05T21:50:42.348 INFO:tasks.workunit.client.0.smithi101.stderr:    self.test(*self.arg)
2023-04-05T21:50:42.348 INFO:tasks.workunit.client.0.smithi101.stderr:  File "/home/ubuntu/cephtest/clone.client.0/src/test/pybind/test_cephfs.py", line 859, in test_disk_quota_exceeeded_error
2023-04-05T21:50:42.348 INFO:tasks.workunit.client.0.smithi101.stderr:    assert_raises(libcephfs.DiskQuotaExceeded, cephfs.write, fd, b"abcdeghiklmnopqrstuvwxyz1234567890qwertyuioddd", -1)
2023-04-05T21:50:42.348 INFO:tasks.workunit.client.0.smithi101.stderr:AssertionError: DiskQuotaExceeded not raised by write
2023-04-05T21:50:42.348 INFO:tasks.workunit.client.0.smithi101.stderr:-------------------- >> begin captured stdout << ---------------------
2023-04-05T21:50:42.348 INFO:tasks.workunit.client.0.smithi101.stderr:ret_buf=b''
2023-04-05T21:50:42.349 INFO:tasks.workunit.client.0.smithi101.stderr:
2023-04-05T21:50:42.349 INFO:tasks.workunit.client.0.smithi101.stderr:--------------------- >> end captured stdout << ----------------------
2023-04-05T21:50:42.349 INFO:tasks.workunit.client.0.smithi101.stderr:
2023-04-05T21:50:42.349 INFO:tasks.workunit.client.0.smithi101.stderr:----------------------------------------------------------------------
2023-04-05T21:50:42.349 INFO:tasks.workunit.client.0.smithi101.stderr:Ran 54 tests in 11.970s
2023-04-05T21:50:42.349 INFO:tasks.workunit.client.0.smithi101.stderr:
2023-04-05T21:50:42.349 INFO:tasks.workunit.client.0.smithi101.stderr:FAILED (errors=5, failures=1)
2023-04-05T21:50:42.374 DEBUG:teuthology.orchestra.run:got remote process result: 1
2023-04-05T21:50:42.375 INFO:tasks.workunit:Stopping ['fs/test_python.sh'] on client.0...


Related issues 1 (1 open0 closed)

Related to CephFS - Bug #56397: client: `df` will show incorrect disk size if the quota size is not aligned to 4MBIn ProgressXiubo Li

Actions
Actions #1

Updated by Xiubo Li about 1 year ago

The quota was successfully set:

2023-04-05T21:50:42.257+0000 7fc342ffd700 12 client.4716 add_update_inode had 0x10000000008.head(faked_ino=0 nref=12 ll_ref=0 cap_refs={} open={} mode=40755 size=0/0 nlink=1 btime=2023-04-05T21:50:42.238630+0000 mtime=2023-04-05T21:50:42.248329+0000 ctime=2023-04-05T21:50:42.248329+0000 change_attr=2 caps=pAsLsXsFsx(0=pAsLsXsFsx) COMPLETE parents=0x1.head["dir-1"] quota(max_bytes = 4096 max_files = 0) 0x7fc32800bab0) caps -
2023-04-05T21:50:42.257+0000 7fc342ffd700 10 client.4716 update_inode_file_time 0x10000000008.head(faked_ino=0 nref=12 ll_ref=0 cap_refs={} open={} mode=40755 size=0/0 nlink=1 btime=2023-04-05T21:50:42.238630+0000 mtime=2023-04-05T21:50:42.248329+0000 ctime=2023-04-05T21:50:42.248329+0000 change_attr=2 caps=pAsLsXsFsx(0=pAsLsXsFsx) COMPLETE parents=0x1.head["dir-1"] quota(max_bytes = 4096 max_files = 0) 0x7fc32800bab0) pAsLsXsFsx ctime 2023-04-05T21:50:42.248329+0000 mtime 2023-04-05T21:50:42.248329+0000

But it still successfully wrote the extra bytes:

2023-04-05T21:50:42.245+0000 7fc3506a7740 10 client.4716 get_quota_root realm 0x10000000008
2023-04-05T21:50:42.245+0000 7fc3506a7740 10 client.4716 get_quota_root 0x10000000009.head -> 0x10000000008.head
2023-04-05T21:50:42.245+0000 7fc3506a7740 10 client.4716 get_quota_root realm 0x10000000008
2023-04-05T21:50:42.245+0000 7fc3506a7740 10 client.4716 get_quota_root realm 0x1
2023-04-05T21:50:42.245+0000 7fc3506a7740 10 client.4716 get_quota_root 0x10000000008.head -> 0x1.head
2023-04-05T21:50:42.245+0000 7fc3506a7740  7 client.4716 wrote to 4136, extending file size
2023-04-05T21:50:42.245+0000 7fc3506a7740 10 mark_caps_dirty 0x10000000009.head(faked_ino=0 nref=11 ll_ref=0 cap_refs={4=0,1024=1,4096=1,8192=1} open={2=1} mode=100755 size=4136/4194304 nlink=1 btime=2023-04-05T21:50:42.248329+0000 mtime=2023-04-05T21:50:42.250592+0000 ctime=2023-04-05T21:50:42.250592+0000 change_attr=2 caps=pAsxLsXsxFsxcrwb(0=pAsxLsXsxFsxcrwb) dirty_caps=AxFxw objectset[0x10000000009 ts 0/0 objects 1 dirty_or_tx 46] parents=0x10000000008.head["file-1"] 0x7fc32800c250) AxFxw -> AxFxw
2023-04-05T21:50:42.245+0000 7fc3506a7740 10 client.4716 cap_delay_requeue on 0x10000000009.head(faked_ino=0 nref=11 ll_ref=0 cap_refs={4=0,1024=1,4096=1,8192=1} open={2=1} mode=100755 size=4136/4194304 nlink=1 btime=2023-04-05T21:50:42.248329+0000 mtime=2023-04-05T21:50:42.250592+0000 ctime=2023-04-05T21:50:42.250592+0000 change_attr=2 caps=pAsxLsXsxFsxcrwb(0=pAsxLsXsxFsxcrwb) dirty_caps=AxFxw objectset[0x10000000009 ts 0/0 objects 1 dirty_or_tx 46] parents=0x10000000008.head["file-1"] 0x7fc32800c250)
2023-04-05T21:50:42.245+0000 7fc3506a7740  3 client.4716 write(10, "...", 46, -1) = 46

It seems the seek didn't correctly set the offset.

Actions #2

Updated by Xiubo Li about 1 year ago

  • Status changed from New to Fix Under Review
  • Pull request ID set to 50910
Actions #3

Updated by Xiubo Li about 1 year ago

  • Backport set to pacific, quincy, reef
Actions #6

Updated by Venky Shankar 9 months ago

  • Related to Bug #56397: client: `df` will show incorrect disk size if the quota size is not aligned to 4MB added
Actions #7

Updated by Konstantin Shalygin 12 days ago

  • Backport changed from pacific, quincy, reef to quincy reef squid
Actions

Also available in: Atom PDF