Bug #68639: quincy: Test failure: test_client_evict_with_invalid_id (tasks.cephfs.test_misc.TestSessionClientEvict) - CephFS - Ceph

Actions

Copy link

Bug #68639

open

quincy: Test failure: test_client_evict_with_invalid_id (tasks.cephfs.test_misc.TestSessionClientEvict)

Added by Venky Shankar over 1 year ago. Updated 6 months ago.

Status:

Fix Under Review

Priority:

Normal

Assignee:

Neeraj Pratap Singh

Category:

Testing

Target version:

% Done:

Source:

Q/A

Backport:

Regression:

Severity:

3 - minor

Reviewed:

Affected Versions:

ceph-qa-suite:

Component(FS):

Labels (FS):

Pull request ID:

60417

Tags (freeform):

Merge Commit:

Fixed In:

Released In:

Upkeep Timestamp:

Description

/a/yuriw-2024-10-17_13:52:02-fs-wip-yuri12-testing-2024-10-16-1152-quincy-distro-default-smithi/7953797

The MDS returns an empty response since evicting a client with an invalid session-id is a no-operation, but the qa code tries to decode the JSON. This is not seen in main branch since the qa code handles that.

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Neeraj Pratap Singh over 1 year ago

Status changed from New to Fix Under Review
Pull request ID set to 60417

https://github.com/ceph/ceph/commit/f5433792ecb0d08cc4a845336b50651bc6909f4a
https://github.com/ceph/ceph/commit/464ef51525d673d38187bb9a841c4802c0681e0f

Above mentioned commits were only backported to squid not to quincy and reef.
We might be seeing this issue due to missing these backports in quincy

Actions

Copy link

Updated by Venky Shankar over 1 year ago

Neeraj Pratap Singh wrote in #note-1:

https://github.com/ceph/ceph/commit/f5433792ecb0d08cc4a845336b50651bc6909f4a
https://github.com/ceph/ceph/commit/464ef51525d673d38187bb9a841c4802c0681e0f

Above mentioned commits were only backported to squid not to quincy and reef.
We might be seeing this issue due to missing these backports in quincy

Can you try cherry picking the relevant commits in the backport PR itself?

Actions

Copy link

Updated by Neeraj Pratap Singh over 1 year ago

Venky Shankar wrote in #note-2:

Neeraj Pratap Singh wrote in #note-1:

https://github.com/ceph/ceph/commit/f5433792ecb0d08cc4a845336b50651bc6909f4a
https://github.com/ceph/ceph/commit/464ef51525d673d38187bb9a841c4802c0681e0f

Above mentioned commits were only backported to squid not to quincy and reef.
We might be seeing this issue due to missing these backports in quincy

Can you try cherry picking the relevant commits in the backport PR itself?

https://github.com/ceph/ceph/pull/58724 is already merged, I have created another backport PR (mentioned in Pull Request ID) for quincy, with the cherry-picked commits

Actions

Copy link