Project

General

Profile

Actions

Documentation #11082

closed

Crushmap documentation still references raid4

Added by Thomas Johnson about 9 years ago. Updated over 8 years ago.

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

0%

Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

The docs for the Crushmap Rules still reference the legacy RAID-4 implementation.

grep --color -Eir 'raid[ _-]?4' ceph-working-copy
doc/rados/operations/crush-map.rst:496:centers for 3-way mirroring, and yet another rule for RAID-4 over six storage
doc/rados/operations/crush-map.rst:506: type [ replicated | raid4 ]
src/mon/OSDMonitor.cc:3962: * @param pool_type TYPE_ERASURE, TYPE_REP or TYPE_RAID4

Actions #1

Updated by Aron Gunn over 8 years ago

Checked the "doc/rados/operations/crush-map.rst" file, and it looks like all the RAID 4 references have already been removed.

$ grep -n --color -i raid doc/rados/operations/crush-map.rst
253:As a general rule, an OSD daemon maps to a single storage drive or to a RAID.
531: or a RAID.

But the "src/mon/OSDMonitor.cc" file still does have a reference to RAID-4.

$ grep -n --color -i raid src/mon/OSDMonitor.cc
4363: * @param pool_type TYPE_ERASURE, TYPE_REP or TYPE_RAID4

Removed the "pool_type" option "TYPE_RAID4".

New line: * @param pool_type TYPE_ERASURE, or TYPE_REP

This is now consistent with the current documentation1.

[1] - http://ceph.com/docs/master/rados/operations/pools/

Actions #2

Updated by John Wilkins over 8 years ago

  • Status changed from New to Resolved

Fixed by Aron Gunn.

Actions

Also available in: Atom PDF