Project

General

Profile

Actions

Fix #10135

closed

OSDMonitor: allow adding cache pools to cephfs pools already in use

Added by Greg Farnum over 9 years ago. Updated over 9 years ago.

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

0%

Source:
Community (user)
Tags:
Backport:
giant
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Right now we disallow this with _check_remove_tier(), I believe because we were worried about coordinating the switchover. The OSDs and rados clients handle this though, thanks to their force_resend_op stuff.

Actions #1

Updated by John Spray over 9 years ago

Yeah, we didn't think about this first time around because the focus was on cache tiers to EC pools, but it would make sense to allow adding cache tiers to replicated pools in use by cephfs.

Actions #2

Updated by John Spray over 9 years ago

  • Status changed from New to Fix Under Review
Actions #3

Updated by Greg Farnum over 9 years ago

  • Status changed from Fix Under Review to Pending Backport

merged to next in commit:25fc21b837ba74bab2f6bc921c78fb3c43993cf5

This also should go into giant (I think Firefly didn't have the issue, right?).

Actions #4

Updated by John Spray over 9 years ago

  • Status changed from Pending Backport to Fix Under Review
Actions #5

Updated by Greg Farnum over 9 years ago

  • Status changed from Fix Under Review to Resolved

26e8cf174b8e76b4282ce9d9c1af6ff12f5565a9

Actions

Also available in: Atom PDF