forked from rook/rook
After further consideration and working with the feature for a few weeks, the `configOverride` functionality seems to be more complexity than is worth it for the expected benefit. Users still have the most preferred option of setting configs in Ceph's config database using `ceph cofig set` or the dashboard, and the override ConfigMap still exists for fallback scenarios. The config database can still be used by Rook for setting necessary configs, but the user has no way of telling Rook to do so. Signed-off-by: Blaine Gardner <blaine.gardner@suse.com>
129 lines
4.2 KiB
Go
129 lines
4.2 KiB
Go
/*
|
|
Copyright 2019 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 config
|
|
|
|
import (
|
|
"fmt"
|
|
"strings"
|
|
"testing"
|
|
|
|
"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"
|
|
)
|
|
|
|
func TestMonStore_Set(t *testing.T) {
|
|
executor := &exectest.MockExecutor{}
|
|
ctx := &clusterd.Context{
|
|
Clientset: testop.New(1),
|
|
Executor: executor,
|
|
}
|
|
|
|
// create a mock command runner which creates a simple string of the command it ran, and allow
|
|
// us to cause it to return an error when it detects a keyword.
|
|
execedCmd := ""
|
|
execInjectErr := false
|
|
executor.MockExecuteCommandWithOutputFile =
|
|
func(debug bool, actionName string, command string, outfile string, args ...string) (string, error) {
|
|
execedCmd = command + " " + strings.Join(args, " ")
|
|
if execInjectErr {
|
|
return "output from cmd with error", fmt.Errorf("mocked error")
|
|
}
|
|
return "", nil
|
|
}
|
|
|
|
monStore := GetMonStore(ctx, "ns")
|
|
|
|
// setting with spaces converts to underscores
|
|
e := monStore.Set("global", "debug ms", "10")
|
|
assert.NoError(t, e)
|
|
assert.Contains(t, execedCmd, "config set global debug_ms 10")
|
|
|
|
// setting with dashes converts to underscores
|
|
e = monStore.Set("osd.0", "debug-osd", "20")
|
|
assert.NoError(t, e)
|
|
assert.Contains(t, execedCmd, " config set osd.0 debug_osd 20 ")
|
|
|
|
// setting with underscores stays the same
|
|
e = monStore.Set("mds.*", "debug_mds", "15")
|
|
assert.NoError(t, e)
|
|
assert.Contains(t, execedCmd, " config set mds.* debug_mds 15 ")
|
|
|
|
// errors returned as expected
|
|
execInjectErr = true
|
|
e = monStore.Set("mon.*", "unknown_setting", "10")
|
|
assert.Error(t, e)
|
|
assert.Contains(t, execedCmd, " config set mon.* unknown_setting 10 ")
|
|
}
|
|
|
|
func TestMonStore_SetAll(t *testing.T) {
|
|
executor := &exectest.MockExecutor{}
|
|
ctx := &clusterd.Context{
|
|
Clientset: testop.New(1),
|
|
Executor: executor,
|
|
}
|
|
|
|
// create a mock command runner which creates a simple string of the command it ran, and allow
|
|
// us to cause it to return an error when it detects a keyword.
|
|
execedCmds := []string{}
|
|
execInjectErrOnKeyword := "donotinjectanerror"
|
|
executor.MockExecuteCommandWithOutputFile =
|
|
func(debug bool, actionName string, command string, outfile string, args ...string) (string, error) {
|
|
execedCmd := command + " " + strings.Join(args, " ")
|
|
execedCmds = append(execedCmds, execedCmd)
|
|
k := execInjectErrOnKeyword
|
|
if strings.Contains(execedCmd, k) {
|
|
return "output from cmd with error on keyword: " + k, fmt.Errorf("mocked error on keyword: " + k)
|
|
}
|
|
return "", nil
|
|
}
|
|
|
|
monStore := GetMonStore(ctx, "ns")
|
|
|
|
cfgOverrides := []Option{
|
|
configOverride("global", "debug ms", "10"), // setting w/ spaces converts to underscores
|
|
configOverride("osd.0", "debug-osd", "20"), // setting w/ dashes converts to underscores
|
|
configOverride("mds.*", "debug_mds", "15"), // setting w/ underscores remains the same
|
|
}
|
|
|
|
// commands w/ no error
|
|
e := monStore.SetAll(cfgOverrides...)
|
|
assert.NoError(t, e)
|
|
assert.Len(t, execedCmds, 3)
|
|
assert.Contains(t, execedCmds[0], " global debug_ms 10 ")
|
|
assert.Contains(t, execedCmds[1], " osd.0 debug_osd 20 ")
|
|
assert.Contains(t, execedCmds[2], " mds.* debug_mds 15 ")
|
|
|
|
// commands w/ one error
|
|
// keep cfgOverrides from last test
|
|
execInjectErrOnKeyword = "debug_osd"
|
|
execedCmds = execedCmds[:0] // empty execedCmds slice
|
|
e = monStore.SetAll(cfgOverrides...)
|
|
assert.Error(t, e)
|
|
// Rook should not return error before trying to set all config overrides
|
|
assert.Len(t, execedCmds, 3)
|
|
|
|
// all commands return error
|
|
// keep cfgOverrides
|
|
execInjectErrOnKeyword = "debug"
|
|
execedCmds = execedCmds[:0]
|
|
e = monStore.SetAll(cfgOverrides...)
|
|
assert.Error(t, e)
|
|
assert.Len(t, execedCmds, 3)
|
|
}
|