Actions
Bug #46902
closedmds: CInode::maybe_export_pin is broken
% Done:
0%
Source:
Development
Tags:
Backport:
pacific,octopus,nautilus
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
MDS
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Description
void CInode::maybe_export_pin(bool update)
{
if (!g_conf()->mds_bal_export_pin)
return;
if (!is_dir() || !is_normal()) // this always return true
return;
Updated by Patrick Donnelly over 3 years ago
- Assignee set to Zheng Yan
- Target version set to v16.0.0
- Source set to Development
- Backport set to octopus,nautilus
- Component(FS) MDS added
Zheng Yan wrote:
void CInode::maybe_export_pin(bool update) {
if (!g_conf()->mds_bal_export_pin)
return;
if (!is_dir() || !is_normal()) // this always return true
return;
Maybe the boolean evaluator in my brain is broken today but this looks right to me.
Updated by Patrick Donnelly over 3 years ago
- Target version changed from v16.0.0 to v17.0.0
- Backport changed from octopus,nautilus to pacific,octopus,nautilus
Updated by Patrick Donnelly over 2 years ago
- Status changed from New to Rejected
Actions