Project

General

Profile

Actions

Feature #9589

closed

erasure-code: query plugin for erasure-code-profile defaults

Added by Loïc Dachary over 9 years ago. Updated almost 9 years ago.

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

0%

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

Description

When a parameter is missing from an erasure-code-profile (ruleset-failure-domain for instance) it falls back to the default of the plugin. The defaults are documented ( http://ceph.com/docs/master/rados/operations/erasure-code-isa/ for instance ) but they do not show when displaying the erasure-code-profile which is confusing for the user.

The get erasure-code-profile should display both the content of the profile and the parameters that are set implicitly by the plugin.

An method should be added to the interface of plugins to validate an erasure code profile and add the default values. This would also make the stored profile less fragile because it would not depend on parameters implicitly provided by the plugin. The profile introspection function should also be the only one providing defaults. When creating an ErasureCodeInstance via the factory, the requirement could be that all parameters need to be provided.


Related issues 1 (0 open1 closed)

Related to Ceph - Bug #9579: Default parameters are not getting initialized for EC profile using isa EC pluginWon't FixLoïc Dachary09/24/2014

Actions
Actions #1

Updated by Loïc Dachary almost 9 years ago

  • Status changed from 12 to In Progress
Actions #2

Updated by Loïc Dachary almost 9 years ago

  • Status changed from In Progress to Fix Under Review
  • Assignee set to Loïc Dachary
Actions #3

Updated by Loïc Dachary almost 9 years ago

  • Status changed from Fix Under Review to Resolved
Actions

Also available in: Atom PDF