Project

General

Profile

Actions

Bug #13477

closed

crush: crash if we see CRUSH_ITEM_NONE in early rule step

Added by Sage Weil over 8 years ago. Updated about 8 years ago.

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

0%

Source:
Community (user)
Tags:
Backport:
infernalis, hammer
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

see bugzilla https://bugzilla.redhat.com/show_bug.cgi?id=1231630

Description of problem:
Seeing Monitor Crash while creating erasure coded pool with wrong parameters

Version-Release number of selected component (if applicable):
ceph version 0.94.1 (e4bfad3a3c51054df7e537a724c8d0bf9be972ff)

How reproducible:
1/1

Steps to Reproduce:
1. Create a ec profile, with the below command.

ceph osd erasure-code-profile set myprofile plugin=lrc mapping=__DD__DD layers='[[ "_cDD_cDD", "" ],[ "cDDD____", "" ],[ "____cDDD", "" ],]' ruleset-steps='[ [ "choose", "datacenter", 3 ], [ "chooseleaf", "osd", 0] ]'

2. ceph osd pool create ecpool 12 12 erasure myprofile

Actual results: Monitoring is crashing.
BT:

ceph version 0.94.1 (e4bfad3a3c51054df7e537a724c8d0bf9be972ff)
1: /usr/bin/ceph-mon() [0x901e52]
2: (()+0xf130) [0x7f205a9ca130]
3: (crush_do_rule()+0x291) [0x833501]
4: (OSDMap::_pg_to_osds(pg_pool_t const&, pg_t, std::vector<int, std::allocator<int> >, int, unsigned int*) const+0xff) [0x77b76f]
5: (OSDMap::_pg_to_up_acting_osds(pg_t const&, std::vector<int, std::allocator<int> >, int, std::vector<int, std::allocator<int> >, int) const+0x104) [0x77be24]
6: (PGMonitor::map_pg_creates()+0x268) [0x65b748]
7: (PGMonitor::post_paxos_update()+0x25) [0x65bf35]
8: (Monitor::refresh_from_paxos(bool*)+0x221) [0x575721]
9: (Monitor::init_paxos()+0x95) [0x575ac5]
10: (Monitor::preinit()+0x7f1) [0x57a881]
11: (main()+0x24a1) [0x54d881]
12: (__libc_start_main()+0xf5) [0x7f20593d0af5]
13: /usr/bin/ceph-mon() [0x55d0f9]
NOTE: a copy of the executable, or `objdump -rdS <executable>` is needed to interpret this.

Files

cm (589 Bytes) cm Sage Weil, 10/13/2015 01:47 PM

Related issues 2 (0 open2 closed)

Copied to Ceph - Backport #13654: crush: crash if we see CRUSH_ITEM_NONE in early rule stepResolvedSage WeilActions
Copied to Ceph - Backport #13655: crush: crash if we see CRUSH_ITEM_NONE in early rule stepResolvedAbhishek VarshneyActions
Actions #1

Updated by Sage Weil over 8 years ago

  • Backport changed from hammer to infernalis, hammer
Actions #2

Updated by Sage Weil over 8 years ago

Good news is that latest code prevents the rule from being created in the first place (yay crushtool check). But crush shouldn't crash.

Actions #3

Updated by Sage Weil over 8 years ago

sample crush map attached

Actions #4

Updated by Sage Weil over 8 years ago

  • Status changed from 12 to Fix Under Review
Actions #5

Updated by Loïc Dachary over 8 years ago

  • Backport changed from infernalis, hammer to infernalis, hammer, firefly
Actions #6

Updated by Loïc Dachary over 8 years ago

  • Assignee set to Sage Weil
Actions #7

Updated by Sage Weil over 8 years ago

  • Status changed from Fix Under Review to Pending Backport
  • Assignee deleted (Sage Weil)
Actions #8

Updated by Nathan Cutler over 8 years ago

  • Copied to Backport #13653: crush: crash if we see CRUSH_ITEM_NONE in early rule step added
Actions #9

Updated by Nathan Cutler over 8 years ago

  • Copied to Backport #13654: crush: crash if we see CRUSH_ITEM_NONE in early rule step added
Actions #10

Updated by Nathan Cutler over 8 years ago

  • Copied to Backport #13655: crush: crash if we see CRUSH_ITEM_NONE in early rule step added
Actions #11

Updated by Loïc Dachary about 8 years ago

  • Backport changed from infernalis, hammer, firefly to infernalis, hammer
Actions #12

Updated by Loïc Dachary about 8 years ago

  • Copied to deleted (Backport #13653: crush: crash if we see CRUSH_ITEM_NONE in early rule step)
Actions #13

Updated by Loïc Dachary about 8 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF