Project

General

Profile

Actions

Bug #20064

closed

allow empty data_extra_pool in zone placement

Added by Casey Bodley almost 7 years ago. Updated over 5 years ago.

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

0%

Source:
Tags:
Backport:
jewel kraken
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

RGWZoneParams::get_head_data_pool() and rgw_get_obj_data_pool() both use placement.data_extra_pool directly instead of calling placement.get_data_extra_pool(), which will return data_pool if extra_data_pool is empty.

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


Related issues 2 (0 open2 closed)

Copied to rgw - Backport #20136: jewel: allow empty data_extra_pool in zone placementRejectedActions
Copied to rgw - Backport #20137: kraken: allow empty data_extra_pool in zone placementRejectedCasey BodleyActions
Actions #1

Updated by Casey Bodley almost 7 years ago

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

Updated by Nathan Cutler almost 7 years ago

  • Copied to Backport #20136: jewel: allow empty data_extra_pool in zone placement added
Actions #3

Updated by Nathan Cutler almost 7 years ago

  • Copied to Backport #20137: kraken: allow empty data_extra_pool in zone placement added
Actions #4

Updated by Nathan Cutler almost 7 years ago

  • Status changed from Pending Backport to Need More Info
  • Assignee set to Casey Bodley

The master commit changes a line in the function "rgw_get_obj_data_pool" which was added by 9c1b828e838. That commit is not in jewel, and it's quite complicated, so it's unclear how to do this backport in a minimal way.

Actions #5

Updated by Nathan Cutler almost 7 years ago

  • Status changed from Need More Info to Pending Backport
Actions #6

Updated by Nathan Cutler over 5 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF