CEPH Journal is created unsafely

Bug #1260546 reported by Andrew Woodward
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Ryan Moe

Bug Description

We prepare a raw partition for each CEPH journal device specified. Then the NUM_OSDS / JOURNAL_DEVS to assign a number of OSD's to each journal device. This is NOT OK, the OSD's can not share raw devices

<xarses> sagewk: hmm, multiple osd's appear to work with the same raw journal device
<xarses> (dumpling)
<sagewk> oh hmm, yeah it's not actually locking it.
<sagewk> that should be fixed
<sagewk> they may appear to work that way but will fail spectacularly on replay

pmanager needs to be updated so that the each journal devices is sub-divided into enough devices to provide a 1-1 mapping of devices.

pmanager should allocate for each partition the between 12-2Gb (less if less than 10% of the drive will remain free) or statically 2Gb for now if complicated to implement.

Case 1:

Journal: sdc
OSD: sdd sde sdf sdg

result pmanager should create sdc1 sdc2 sdc3 sdc4

Case 2:
Journal sdb sdc
OSD: sdd sde sdf sdg

result pmanager should create sdb1 sdb2 sdc1 sdc2

---

the fact should already be capable of pairing the OSD's and journals correctly with out modification

Andrew Woodward (xarses)
description: updated
description: updated
Revision history for this message
Andrey Korolyov (xdeller) wrote :

Can we just use file-based targets and one partition for every journal?

Ryan Moe (rmoe)
Changed in fuel:
assignee: nobody → Ryan Moe (rmoe)
Revision history for this message
Andrew Woodward (xarses) wrote :

Its possible, but from IRC again.
<ishkabob> hey guys, whats the recommended filesystem for a separate journal drive?
<sagewk> ishkabob: use a raw partition (no fs)

Changed in fuel:
milestone: 4.0 → 4.1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/62497

Changed in fuel:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/62497
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=24c1fc9a315a4fbbef03720d7b0f6d756b78fbdd
Submitter: Jenkins
Branch: master

commit 24c1fc9a315a4fbbef03720d7b0f6d756b78fbdd
Author: Ryan Moe <email address hidden>
Date: Mon Dec 16 15:02:14 2013 -0800

    Allocate separate journal partitions for each OSD

    Change-Id: Ie9cde42620ba8f63d4ef3f8b36786a9a49317e83
    Closes-bug: #1260546

Changed in fuel:
status: In Progress → Fix Committed
Andrew Woodward (xarses)
Changed in fuel:
milestone: 4.1 → 4.0
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.