Project

General

Profile

Feature #7360

When EC ready add back test.sh test case for "mon: OSDMonitor: do not allow changing an erasure-coded pool's size"

Added by David Zafman over 6 years ago. Updated over 6 years ago.

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

70%

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

Description

Work in progress

Once OSDs can handle Erasure Coding we should add back the test.sh test case for attempting to change the size of an Erasure Coded pool.

This is just the test.sh change of commit c8c4cc6e81816069886af6bff968712993554759

commit c8c4cc6e81816069886af6bff968712993554759
Author: Joao Eduardo Luis <>
Date: Sun Feb 2 14:02:17 2014 +0000

mon: OSDMonitor: do not allow changing an erasure-coded pool's size
Fixes: 7277
Reviewed-by: Loic Dachary &lt;&gt;
Signed-off-by: Joao Eduardo Luis &lt;&gt;

diff --git a/qa/workunits/cephtool/test.sh b/qa/workunits/cephtool/test.sh
index 58d0e94..8419c7f 100755
--- a/qa/workunits/cephtool/test.sh
+++ b/qa/workunits/cephtool/test.sh
@ -369,6 +369,13 @ ceph osd pool set data size $new_size
ceph osd pool get data size | grep "size: $new_size"
ceph osd pool set data size $old_size

ceph osd crush rule create-erasure ec_ruleset
+ceph osd pool create pool_erasure 12 12 erasure crush_ruleset=ec_ruleset
+set +e
+ceph osd pool set pool_erasure size 4444 2>$TMPFILE
+check_response $? 38 'can not change the size'
+set -e

ceph osd pool set data hashpspool true
ceph osd pool set data hashpspool false
ceph osd pool set data hashpspool 0

Associated revisions

Revision 1340b360 (diff)
Added by Loic Dachary over 6 years ago

erasure-code: test that changing the pool size is not allowed

http://tracker.ceph.com/issues/7360 fixes #7360

Signed-off-by: Loic Dachary <>

History

#1 Updated by Loic Dachary over 6 years ago

  • Assignee changed from Joao Eduardo Luis to Loic Dachary

#2 Updated by Loic Dachary over 6 years ago

  • Status changed from New to 12

#3 Updated by Loic Dachary over 6 years ago

  • Description updated (diff)
  • % Done changed from 0 to 70

crashes, investigating why

#4 Updated by Sage Weil over 6 years ago

  • Tracker changed from Bug to Feature
  • Target version set to 0.78
  • Source changed from other to Development

#5 Updated by Sage Weil over 6 years ago

  • Status changed from 12 to Resolved

Also available in: Atom PDF