Files
my-rook-config/pkg/operator/ceph/pool/controller_test.go
T
travisn 9dcddb5bed ceph: upgrade all daemons during ceph upgrade
The CephCluster CR contains settings that are needed by other
CRs to configure the Ceph daemons. When the CephCluster CR
is updated, the updates will now be passed on to each of the
CR controllers to ensure the daemons are updated properly
without requiring an operator restart.

When calling the controllers from another controller,
we ensure that only a single goroutine is handling
CRs at any given time to prevent contention across
multiple CRs of the same type.

Signed-off-by: travisn <tnielsen@redhat.com>
2019-04-23 13:36:45 -06:00

284 lines
9.2 KiB
Go

/*
Copyright 2016 The Rook Authors. All rights reserved.
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/
package pool
import (
"fmt"
"testing"
cephv1 "github.com/rook/rook/pkg/apis/ceph.rook.io/v1"
cephbeta "github.com/rook/rook/pkg/apis/ceph.rook.io/v1beta1"
rookfake "github.com/rook/rook/pkg/client/clientset/versioned/fake"
"github.com/rook/rook/pkg/clusterd"
testop "github.com/rook/rook/pkg/operator/test"
exectest "github.com/rook/rook/pkg/util/exec/test"
"github.com/stretchr/testify/assert"
"k8s.io/apimachinery/pkg/api/errors"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
)
func TestValidatePool(t *testing.T) {
context := &clusterd.Context{Executor: &exectest.MockExecutor{}}
// must specify some replication or EC settings
p := cephv1.CephBlockPool{ObjectMeta: metav1.ObjectMeta{Name: "mypool", Namespace: "myns"}}
err := ValidatePool(context, &p)
assert.NotNil(t, err)
// must specify name
p = cephv1.CephBlockPool{ObjectMeta: metav1.ObjectMeta{Namespace: "myns"}}
err = ValidatePool(context, &p)
assert.NotNil(t, err)
// must specify namespace
p = cephv1.CephBlockPool{ObjectMeta: metav1.ObjectMeta{Name: "mypool"}}
err = ValidatePool(context, &p)
assert.NotNil(t, err)
// must not specify both replication and EC settings
p = cephv1.CephBlockPool{ObjectMeta: metav1.ObjectMeta{Name: "mypool", Namespace: "myns"}}
p.Spec.Replicated.Size = 1
p.Spec.ErasureCoded.CodingChunks = 2
p.Spec.ErasureCoded.DataChunks = 3
err = ValidatePool(context, &p)
assert.NotNil(t, err)
// succeed with replication settings
p = cephv1.CephBlockPool{ObjectMeta: metav1.ObjectMeta{Name: "mypool", Namespace: "myns"}}
p.Spec.Replicated.Size = 1
err = ValidatePool(context, &p)
assert.Nil(t, err)
// succeed with ec settings
p = cephv1.CephBlockPool{ObjectMeta: metav1.ObjectMeta{Name: "mypool", Namespace: "myns"}}
p.Spec.ErasureCoded.CodingChunks = 1
p.Spec.ErasureCoded.DataChunks = 2
err = ValidatePool(context, &p)
assert.Nil(t, err)
}
func TestValidateCrushProperties(t *testing.T) {
executor := &exectest.MockExecutor{}
context := &clusterd.Context{Executor: executor}
executor.MockExecuteCommandWithOutputFile = func(debug bool, actionName, command, outputFile string, args ...string) (string, error) {
logger.Infof("Command: %s %v", command, args)
if args[1] == "crush" && args[2] == "dump" {
return `{"types":[{"type_id": 0,"name": "osd"}],"buckets":[{"id": -1,"name":"default"},{"id": -2,"name":"good"}]}`, nil
}
return "", fmt.Errorf("unexpected ceph command '%v'", args)
}
// succeed with a failure domain that exists
p := &cephv1.CephBlockPool{
ObjectMeta: metav1.ObjectMeta{Name: "mypool", Namespace: "myns"},
Spec: cephv1.PoolSpec{
Replicated: cephv1.ReplicatedSpec{Size: 1},
FailureDomain: "osd",
},
}
err := ValidatePool(context, p)
assert.Nil(t, err)
// fail with a failure domain that doesn't exist
p.Spec.FailureDomain = "doesntexist"
err = ValidatePool(context, p)
assert.NotNil(t, err)
// fail with a crush root that doesn't exist
p.Spec.FailureDomain = "osd"
p.Spec.CrushRoot = "bad"
err = ValidatePool(context, p)
assert.NotNil(t, err)
// fail with a crush root that does exist
p.Spec.CrushRoot = "good"
err = ValidatePool(context, p)
assert.Nil(t, err)
}
func TestCreatePool(t *testing.T) {
executor := &exectest.MockExecutor{
MockExecuteCommandWithOutputFile: func(debug bool, actionName, command, outfile string, args ...string) (string, error) {
if command == "ceph" && args[1] == "erasure-code-profile" {
return `{"k":"2","m":"1","plugin":"jerasure","technique":"reed_sol_van"}`, nil
}
return "", nil
},
}
context := &clusterd.Context{Executor: executor}
p := &cephv1.CephBlockPool{ObjectMeta: metav1.ObjectMeta{Name: "mypool", Namespace: "myns"}}
p.Spec.Replicated.Size = 1
exists, err := poolExists(context, p)
assert.False(t, exists)
err = createPool(context, p)
assert.Nil(t, err)
// fail if both replication and EC are specified
p.Spec.ErasureCoded.CodingChunks = 2
p.Spec.ErasureCoded.DataChunks = 2
err = createPool(context, p)
assert.NotNil(t, err)
// succeed with EC
p.Spec.Replicated.Size = 0
err = createPool(context, p)
assert.Nil(t, err)
}
func TestUpdatePool(t *testing.T) {
// the pool did not change for properties that are updatable
old := cephv1.PoolSpec{FailureDomain: "osd", ErasureCoded: cephv1.ErasureCodedSpec{CodingChunks: 2, DataChunks: 2}}
new := cephv1.PoolSpec{FailureDomain: "host", ErasureCoded: cephv1.ErasureCodedSpec{CodingChunks: 3, DataChunks: 3}}
changed := poolChanged(old, new)
assert.False(t, changed)
// the pool changed for properties that are updatable
old = cephv1.PoolSpec{FailureDomain: "osd", Replicated: cephv1.ReplicatedSpec{Size: 1}}
new = cephv1.PoolSpec{FailureDomain: "osd", Replicated: cephv1.ReplicatedSpec{Size: 2}}
changed = poolChanged(old, new)
assert.True(t, changed)
}
func TestDeletePool(t *testing.T) {
executor := &exectest.MockExecutor{
MockExecuteCommandWithOutputFile: func(debug bool, actionName, command, outfile string, args ...string) (string, error) {
if command == "ceph" && args[1] == "lspools" {
return `[{"poolnum":1,"poolname":"mypool"}]`, nil
} else if command == "ceph" && args[1] == "pool" && args[2] == "get" {
return `{"pool": "mypool","pool_id": 1,"size":1}`, nil
}
return "", nil
},
}
context := &clusterd.Context{Executor: executor}
// delete a pool that exists
p := &cephv1.CephBlockPool{ObjectMeta: metav1.ObjectMeta{Name: "mypool", Namespace: "myns"}}
exists, err := poolExists(context, p)
assert.Nil(t, err)
assert.True(t, exists)
err = deletePool(context, p)
assert.Nil(t, err)
// succeed even if the pool doesn't exist
p = &cephv1.CephBlockPool{ObjectMeta: metav1.ObjectMeta{Name: "otherpool", Namespace: "myns"}}
exists, err = poolExists(context, p)
assert.Nil(t, err)
assert.False(t, exists)
err = deletePool(context, p)
assert.Nil(t, err)
}
func TestGetPoolObject(t *testing.T) {
// get a current version pool object, should return with no error and no migration needed
pool, migrationNeeded, err := getPoolObject(&cephv1.CephBlockPool{})
assert.NotNil(t, pool)
assert.False(t, migrationNeeded)
assert.Nil(t, err)
// get a legacy version pool object, should return with no error and yes migration needed
pool, migrationNeeded, err = getPoolObject(&cephbeta.Pool{})
assert.NotNil(t, pool)
assert.True(t, migrationNeeded)
assert.Nil(t, err)
// try to get an object that isn't a pool, should return with an error
pool, migrationNeeded, err = getPoolObject(&map[string]string{})
assert.Nil(t, pool)
assert.False(t, migrationNeeded)
assert.NotNil(t, err)
}
func TestMigratePoolObject(t *testing.T) {
// create a legacy pool that will get migrated
legacyPool := &cephbeta.Pool{
ObjectMeta: metav1.ObjectMeta{
Name: "legacy-pool-861",
Namespace: "rook-267",
},
}
// create fake core and rook clientsets and a pool controller
clientset := testop.New(3)
context := &clusterd.Context{
Clientset: clientset,
RookClientset: rookfake.NewSimpleClientset(legacyPool),
}
controller := NewPoolController(context, legacyPool.Namespace)
// convert the legacy pool object in memory and assert that a migration is needed
convertedPool, migrationNeeded, err := getPoolObject(legacyPool)
assert.NotNil(t, convertedPool)
assert.True(t, migrationNeeded)
assert.Nil(t, err)
// perform the migration of the converted legacy pool
err = controller.migratePoolObject(convertedPool, legacyPool)
assert.Nil(t, err)
// assert that a current pool object was created via the migration
migratedPool, err := context.RookClientset.CephV1().CephBlockPools(legacyPool.Namespace).Get(
legacyPool.Name, metav1.GetOptions{})
assert.NotNil(t, migratedPool)
assert.Nil(t, err)
// assert that the legacy pool object was deleted
_, err = context.RookClientset.CephV1beta1().Pools(legacyPool.Namespace).Get(legacyPool.Name, metav1.GetOptions{})
assert.NotNil(t, err)
assert.True(t, errors.IsNotFound(err))
}
func TestConvertLegacyPool(t *testing.T) {
legacyPool := cephbeta.Pool{
ObjectMeta: metav1.ObjectMeta{
Name: "legacy-pool-383",
Namespace: "rook-215",
},
Spec: cephbeta.PoolSpec{
FailureDomain: "fd202",
CrushRoot: "root329",
Replicated: cephbeta.ReplicatedSpec{Size: 5},
ErasureCoded: cephbeta.ErasureCodedSpec{
CodingChunks: 5,
DataChunks: 10,
Algorithm: "ec-algorithm-367",
},
},
}
expectedPool := cephv1.CephBlockPool{
ObjectMeta: metav1.ObjectMeta{
Name: "legacy-pool-383",
Namespace: "rook-215",
},
Spec: cephv1.PoolSpec{
FailureDomain: "fd202",
CrushRoot: "root329",
Replicated: cephv1.ReplicatedSpec{Size: 5},
ErasureCoded: cephv1.ErasureCodedSpec{
CodingChunks: 5,
DataChunks: 10,
Algorithm: "ec-algorithm-367",
},
},
}
assert.Equal(t, expectedPool, *convertRookLegacyPool(&legacyPool))
}