milvus/pkg/config/manager.go
wei liu 1f759837c4
fix: remove Unnecessary lock in config manager (#29836)
issue: #29709 #291712
to avoid concurrent recursive RLock and Lock cause deadlock, This PR
remove the unnecessary lock in config manager

---------

Signed-off-by: Wei Liu <wei.liu@zilliz.com>
2024-01-11 13:48:49 +08:00

380 lines
9.6 KiB
Go

// Licensed to the LF AI & Data foundation under one
// or more contributor license agreements. See the NOTICE file
// distributed with this work for additional information
// regarding copyright ownership. The ASF licenses this file
// to you 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"
"github.com/cockroachdb/errors"
"go.uber.org/zap"
"github.com/milvus-io/milvus/pkg/log"
"github.com/milvus-io/milvus/pkg/util/typeutil"
)
const (
TombValue = "TOMB_VAULE"
)
type Filter func(key string) (string, bool)
func WithSubstr(substring string) Filter {
substring = strings.ToLower(substring)
return func(key string) (string, bool) {
return key, strings.Contains(key, substring)
}
}
func WithPrefix(prefix string) Filter {
prefix = strings.ToLower(prefix)
return func(key string) (string, bool) {
return key, strings.HasPrefix(key, prefix)
}
}
func WithOneOfPrefixs(prefixs ...string) Filter {
for id, prefix := range prefixs {
prefixs[id] = strings.ToLower(prefix)
}
return func(key string) (string, bool) {
for _, prefix := range prefixs {
if strings.HasPrefix(key, prefix) {
return key, true
}
}
return key, false
}
}
func RemovePrefix(prefix string) Filter {
prefix = strings.ToLower(prefix)
return func(key string) (string, bool) {
return strings.Replace(key, prefix, "", 1), true
}
}
func filterate(key string, filters ...Filter) (string, bool) {
var ok bool
for _, filter := range filters {
key, ok = filter(key)
if !ok {
return key, ok
}
}
return key, ok
}
type Manager struct {
Dispatcher *EventDispatcher
sources *typeutil.ConcurrentMap[string, Source]
keySourceMap *typeutil.ConcurrentMap[string, string] // store the key to config source, example: key is A.B.C and source is file which means the A.B.C's value is from file
overlays *typeutil.ConcurrentMap[string, string] // store the highest priority configs which modified at runtime
forbiddenKeys *typeutil.ConcurrentSet[string]
}
func NewManager() *Manager {
return &Manager{
Dispatcher: NewEventDispatcher(),
sources: typeutil.NewConcurrentMap[string, Source](),
keySourceMap: typeutil.NewConcurrentMap[string, string](),
overlays: typeutil.NewConcurrentMap[string, string](),
forbiddenKeys: typeutil.NewConcurrentSet[string](),
}
}
func (m *Manager) GetConfig(key string) (string, error) {
realKey := formatKey(key)
v, ok := m.overlays.Get(realKey)
if ok {
if v == TombValue {
return "", fmt.Errorf("key not found %s", key)
}
return v, nil
}
sourceName, ok := m.keySourceMap.Get(realKey)
if !ok {
return "", fmt.Errorf("key not found: %s", key)
}
return m.getConfigValueBySource(realKey, sourceName)
}
// GetConfigs returns all the key values
func (m *Manager) GetConfigs() map[string]string {
config := make(map[string]string)
m.keySourceMap.Range(func(key, value string) bool {
sValue, err := m.GetConfig(key)
if err != nil {
return true
}
config[key] = sValue
return true
})
m.overlays.Range(func(key, value string) bool {
config[key] = value
return true
})
return config
}
func (m *Manager) GetBy(filters ...Filter) map[string]string {
matchedConfig := make(map[string]string)
for key, value := range m.GetConfigs() {
newkey, ok := filterate(key, filters...)
if ok {
matchedConfig[newkey] = value
}
}
return matchedConfig
}
func (m *Manager) FileConfigs() map[string]string {
config := make(map[string]string)
m.sources.Range(func(key string, value Source) bool {
if s, ok := value.(*FileSource); ok {
config, _ = s.GetConfigurations()
return false
}
return true
})
return config
}
func (m *Manager) Close() {
m.sources.Range(func(key string, value Source) bool {
value.Close()
return true
})
}
func (m *Manager) AddSource(source Source) error {
sourceName := source.GetSourceName()
_, ok := m.sources.Get(sourceName)
if ok {
err := errors.New("duplicate source supplied")
return err
}
m.sources.Insert(sourceName, source)
err := m.pullSourceConfigs(sourceName)
if err != nil {
err = fmt.Errorf("failed to load %s cause: %x", sourceName, err)
return err
}
source.SetEventHandler(m)
return nil
}
// Update config at runtime, which can be called by others
// The most used scenario is UT
func (m *Manager) SetConfig(key, value string) {
m.overlays.Insert(formatKey(key), value)
}
func (m *Manager) SetMapConfig(key, value string) {
m.overlays.Insert(strings.ToLower(key), value)
}
// Delete config at runtime, which has the highest priority to override all other sources
func (m *Manager) DeleteConfig(key string) {
m.overlays.Insert(formatKey(key), TombValue)
}
// Remove the config which set at runtime, use config from sources
func (m *Manager) ResetConfig(key string) {
m.overlays.Remove(formatKey(key))
}
// Ignore any of update events, which means the config cannot auto refresh anymore
func (m *Manager) ForbidUpdate(key string) {
m.forbiddenKeys.Insert(formatKey(key))
}
func (m *Manager) UpdateSourceOptions(opts ...Option) {
var options Options
for _, opt := range opts {
opt(&options)
}
m.sources.Range(func(key string, value Source) bool {
value.UpdateOptions(options)
return true
})
}
// Do not use it directly, only used when add source and unittests.
func (m *Manager) pullSourceConfigs(source string) error {
configSource, ok := m.sources.Get(source)
if !ok {
return errors.New("invalid source or source not added")
}
configs, err := configSource.GetConfigurations()
if err != nil {
log.Info("Get configuration by items failed", zap.Error(err))
return err
}
sourcePriority := configSource.GetPriority()
for key := range configs {
sourceName, ok := m.keySourceMap.Get(key)
if !ok { // if key do not exist then add source
m.keySourceMap.Insert(key, source)
continue
}
currentSource, ok := m.sources.Get(sourceName)
if !ok {
m.keySourceMap.Insert(key, source)
continue
}
currentSrcPriority := currentSource.GetPriority()
if currentSrcPriority > sourcePriority { // lesser value has high priority
m.keySourceMap.Insert(key, source)
}
}
return nil
}
func (m *Manager) getConfigValueBySource(configKey, sourceName string) (string, error) {
source, ok := m.sources.Get(sourceName)
if !ok {
return "", ErrKeyNotFound
}
return source.GetConfigurationByKey(configKey)
}
func (m *Manager) updateEvent(e *Event) error {
// refresh all configuration one by one
if e.HasUpdated {
return nil
}
switch e.EventType {
case CreateType, UpdateType:
sourceName, ok := m.keySourceMap.Get(e.Key)
if !ok {
m.keySourceMap.Insert(e.Key, e.EventSource)
e.EventType = CreateType
} else if sourceName == e.EventSource {
e.EventType = UpdateType
} else if sourceName != e.EventSource {
prioritySrc := m.getHighPrioritySource(sourceName, e.EventSource)
if prioritySrc != nil && prioritySrc.GetSourceName() == sourceName {
// if event generated from less priority source then ignore
log.Info(fmt.Sprintf("the event source %s's priority is less then %s's, ignore",
e.EventSource, sourceName))
return ErrIgnoreChange
}
m.keySourceMap.Insert(e.Key, e.EventSource)
e.EventType = UpdateType
}
case DeleteType:
sourceName, ok := m.keySourceMap.Get(e.Key)
if !ok || sourceName != e.EventSource {
// if delete event generated from source not maintained ignore it
log.Info(fmt.Sprintf("the event source %s (expect %s) is not maintained, ignore",
e.EventSource, sourceName))
return ErrIgnoreChange
} else if sourceName == e.EventSource {
// find less priority source or delete key
source := m.findNextBestSource(e.Key, sourceName)
if source == nil {
m.keySourceMap.Remove(e.Key)
} else {
m.keySourceMap.Insert(e.Key, source.GetSourceName())
}
}
}
log.Info("receive update event", zap.Any("event", e))
e.HasUpdated = true
return nil
}
// OnEvent Triggers actions when an event is generated
func (m *Manager) OnEvent(event *Event) {
if m.forbiddenKeys.Contain(formatKey(event.Key)) {
log.Info("ignore event for forbidden key", zap.String("key", event.Key))
return
}
err := m.updateEvent(event)
if err != nil {
log.Warn("failed in updating event with error", zap.Error(err), zap.Any("event", event))
return
}
m.Dispatcher.Dispatch(event)
}
func (m *Manager) GetIdentifier() string {
return "Manager"
}
func (m *Manager) findNextBestSource(configKey string, sourceName string) Source {
var rSource Source
m.sources.Range(func(key string, value Source) bool {
if value.GetSourceName() == sourceName {
return true
}
_, err := value.GetConfigurationByKey(configKey)
if err != nil {
return true
}
if rSource == nil {
rSource = value
return true
}
if value.GetPriority() < rSource.GetPriority() { // less value has high priority
rSource = value
}
return true
})
return rSource
}
func (m *Manager) getHighPrioritySource(srcNameA, srcNameB string) Source {
sourceA, okA := m.sources.Get(srcNameA)
sourceB, okB := m.sources.Get(srcNameB)
if !okA && !okB {
return nil
} else if !okA {
return sourceB
} else if !okB {
return sourceA
}
if sourceA.GetPriority() < sourceB.GetPriority() { // less value has high priority
return sourceA
}
return sourceB
}