Project

General

Profile

Actions

Subtask #5878

closed

Feature #4929: Erasure encoded placement group

Subtask #5877: Plugable erasure code library

erasure plugin mechanism and abstract API

Added by Loïc Dachary over 10 years ago. Updated about 10 years ago.

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

100%

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

Description

work in progress

The abstract API is provided by dynamically loaded plugins.
  • Abstract Interface
  • Plugin loader
  • unit tests
    • in the test file load the plugin from source
    • simple minded xor based erasure code plugin that only supports M=2,K=1
    • the plugin can be directed to trigger errors for tests purposes
    • implement the test plugin to be the basis of a documented example to follow when implementing a new plugin

Discussions


Related issues 1 (0 open1 closed)

Related to Ceph - Feature #6000: EC: [link] erasure plugin mechanism and abstract APIResolvedLoïc Dachary08/15/2013

Actions
Actions #1

Updated by Loïc Dachary over 10 years ago

  • Category set to OSD
Actions #2

Updated by Loïc Dachary over 10 years ago

  • Source changed from other to Development
Actions #3

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #4

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #5

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #6

Updated by Loïc Dachary over 10 years ago

  • Assignee set to Loïc Dachary
Actions #7

Updated by Loïc Dachary over 10 years ago

  • Status changed from New to In Progress
Actions #8

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #9

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #10

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #11

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #12

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #13

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #14

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #15

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #16

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #17

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #18

Updated by Wido den Hollander over 10 years ago

I haven't looked at it in-depth, but one thing I noticed is that Reed-Solomon is always spelled with the first two letters capatalized.

Will this be case sensitve? I would suggest not, since that will confuse users. I personally dislike CLI tools which are case sensitive for key=value parameters. I always prefer lowercase.

Actions #19

Updated by Loïc Dachary over 10 years ago

Wido den Hollander wrote:

Will this be case sensitve? I would suggest not, since that will confuse users. I personally dislike CLI tools which are case sensitive for key=value parameters. I always prefer lowercase.

In this case Reed-Solomon is the value interpreted by the plugin and I agree with you : it would be better if case is ignored :-)

Actions #20

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #21

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #22

Updated by Loïc Dachary over 10 years ago

  • % Done changed from 0 to 50
Actions #23

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #24

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
  • Status changed from In Progress to Fix Under Review
  • % Done changed from 50 to 90
Actions #25

Updated by Loïc Dachary over 10 years ago

  • Description updated (diff)
Actions #26

Updated by Loïc Dachary over 10 years ago

  • Status changed from Fix Under Review to Resolved
  • % Done changed from 90 to 100
  • Translation missing: en.field_remaining_hours set to 0.00
Actions #27

Updated by Loïc Dachary about 10 years ago

  • Estimated time set to 0:00 h
Actions

Also available in: Atom PDF