Bug #51857: client: make sure only to update dir dist from auth mds - CephFS - Ceph

Actions

Copy link

Bug #51857

closed

client: make sure only to update dir dist from auth mds

Added by yantao xue over 2 years ago. Updated 12 months ago.

Status:

Resolved

Priority:

Normal

Assignee:

yantao xue

Category:

Target version:

Ceph - v17.0.0

% Done:

100%

Source:

Community (dev)

Tags:

backport_processed

Backport:

pacific,octopus

Regression:

Severity:

3 - minor

Reviewed:

Affected Versions:

Ceph - v17.0.0

ceph-qa-suite:

Component(FS):

Client

Labels (FS):

Pull request ID:

42499

Crash signature (v1):

Crash signature (v2):

Description

In mds/Server::set_trace_dist, if (dir->is_auth() && !forward_all_requests_to_auth) dir->get_dist_spec(ds.dist, whoami);
On client side, in Client::update_dir_dist, it updates the replica info from the ds.dist, regardless of whether the reply comes from the replica or the auth mds. Because the replica mds dose not spec the ds.dist, so client will clear the replica info what have been set by the auth mds when received reply from the replica mds.

Related issues 2 (0 open — 2 closed)

Actions

Copy link

Updated by Kefu Chai over 2 years ago

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

Actions

Copy link

Updated by Patrick Donnelly over 2 years ago

Project changed from Ceph to CephFS
Subject changed from client:make sure only to update dir dist from auth mds to client: make sure only to update dir dist from auth mds
Assignee set to yantao xue
Target version set to v17.0.0
Source set to Community (dev)
Component(FS) Client added