Project

General

Profile

Bug #24644

cephfs-journal-tool: wrong layout info used

Added by cory gu over 4 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
% Done:

0%

Source:
Community (dev)
Tags:
Backport:
mimic,luminous
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
tools
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

when cephfs-journal-tool import journal, it uses default layout
to get object_size, this is wrong. Because default objetct_size
can be changed by Config option change mds_log_segment_size when
cluster initialized.


Related issues

Copied to CephFS - Backport #24933: mimic: cephfs-journal-tool: wrong layout info used Resolved
Copied to CephFS - Backport #24934: luminous: cephfs-journal-tool: wrong layout info used Resolved

History

#1 Updated by cory gu over 4 years ago

The fix is save the layout info to header during journal export,
When improt journal, first try to get layout from cluster, if failed,
then get layout info from dumped journal file, if still failed, set
layout to default.

https://github.com/ceph/ceph/pull/22211

#2 Updated by Patrick Donnelly over 4 years ago

  • Project changed from Ceph to CephFS
  • Subject changed from cephfs-journal-tool use wrong layout info to cephfs-journal-tool: wrong layout info used
  • Assignee set to cory gu
  • Target version set to v14.0.0
  • Source set to Community (dev)
  • Backport set to mimic,luminous
  • Component(FS) tools added

#3 Updated by Patrick Donnelly over 4 years ago

  • Status changed from New to Fix Under Review

#4 Updated by Patrick Donnelly over 4 years ago

  • Status changed from Fix Under Review to Pending Backport

#5 Updated by Nathan Cutler over 4 years ago

  • Copied to Backport #24933: mimic: cephfs-journal-tool: wrong layout info used added

#6 Updated by Nathan Cutler over 4 years ago

  • Copied to Backport #24934: luminous: cephfs-journal-tool: wrong layout info used added

#7 Updated by Patrick Donnelly about 4 years ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF