[#1361] linter: fix funlen
All checks were successful
DCO action / DCO (pull_request) Successful in 53s
Tests and linters / Run gofumpt (pull_request) Successful in 1m19s
Vulncheck / Vulncheck (pull_request) Successful in 1m51s
Pre-commit hooks / Pre-commit (pull_request) Successful in 2m23s
Build / Build Components (pull_request) Successful in 2m31s
Tests and linters / Staticcheck (pull_request) Successful in 2m39s
Tests and linters / gopls check (pull_request) Successful in 2m55s
Tests and linters / Lint (pull_request) Successful in 3m51s
Tests and linters / Tests (pull_request) Successful in 4m41s
Tests and linters / Tests with -race (pull_request) Successful in 4m53s
All checks were successful
DCO action / DCO (pull_request) Successful in 53s
Tests and linters / Run gofumpt (pull_request) Successful in 1m19s
Vulncheck / Vulncheck (pull_request) Successful in 1m51s
Pre-commit hooks / Pre-commit (pull_request) Successful in 2m23s
Build / Build Components (pull_request) Successful in 2m31s
Tests and linters / Staticcheck (pull_request) Successful in 2m39s
Tests and linters / gopls check (pull_request) Successful in 2m55s
Tests and linters / Lint (pull_request) Successful in 3m51s
Tests and linters / Tests (pull_request) Successful in 4m41s
Tests and linters / Tests with -race (pull_request) Successful in 4m53s
Signed-off-by: Aleksey Savaitan <a.savaitan@yadro.com>
This commit is contained in:
parent
3df584043e
commit
7bbe90803c
1 changed files with 31 additions and 24 deletions
|
@ -1278,7 +1278,6 @@ func (c *cfg) reloadConfig(ctx context.Context) {
|
||||||
|
|
||||||
// all the components are expected to support
|
// all the components are expected to support
|
||||||
// Logger's dynamic reconfiguration approach
|
// Logger's dynamic reconfiguration approach
|
||||||
var components []dCmp
|
|
||||||
|
|
||||||
// Logger
|
// Logger
|
||||||
|
|
||||||
|
@ -1288,6 +1287,36 @@ func (c *cfg) reloadConfig(ctx context.Context) {
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
|
components := c.getComponents(ctx, logPrm)
|
||||||
|
|
||||||
|
// Storage Engine
|
||||||
|
|
||||||
|
var rcfg engine.ReConfiguration
|
||||||
|
for _, optsWithID := range c.shardOpts(ctx) {
|
||||||
|
rcfg.AddShard(optsWithID.configID, append(optsWithID.shOpts, shard.WithTombstoneSource(c.createTombstoneSource())))
|
||||||
|
}
|
||||||
|
|
||||||
|
err = c.cfgObject.cfgLocalStorage.localStorage.Reload(ctx, rcfg)
|
||||||
|
if err != nil {
|
||||||
|
c.log.Error(logs.FrostFSNodeStorageEngineConfigurationUpdate, zap.Error(err))
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, component := range components {
|
||||||
|
err = component.reloadFunc()
|
||||||
|
if err != nil {
|
||||||
|
c.log.Error(logs.FrostFSNodeUpdatedConfigurationApplying,
|
||||||
|
zap.String("component", component.name),
|
||||||
|
zap.Error(err))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
c.log.Info(logs.FrostFSNodeConfigurationHasBeenReloadedSuccessfully)
|
||||||
|
}
|
||||||
|
|
||||||
|
func (c *cfg) getComponents(ctx context.Context, logPrm *logger.Prm) []dCmp {
|
||||||
|
var components []dCmp
|
||||||
|
|
||||||
components = append(components, dCmp{"logger", logPrm.Reload})
|
components = append(components, dCmp{"logger", logPrm.Reload})
|
||||||
components = append(components, dCmp{"runtime", func() error {
|
components = append(components, dCmp{"runtime", func() error {
|
||||||
setRuntimeParameters(c)
|
setRuntimeParameters(c)
|
||||||
|
@ -1321,29 +1350,7 @@ func (c *cfg) reloadConfig(ctx context.Context) {
|
||||||
components = append(components, dCmp{cmp.name, func() error { return cmp.reload(ctx) }})
|
components = append(components, dCmp{cmp.name, func() error { return cmp.reload(ctx) }})
|
||||||
}
|
}
|
||||||
|
|
||||||
// Storage Engine
|
return components
|
||||||
|
|
||||||
var rcfg engine.ReConfiguration
|
|
||||||
for _, optsWithID := range c.shardOpts(ctx) {
|
|
||||||
rcfg.AddShard(optsWithID.configID, append(optsWithID.shOpts, shard.WithTombstoneSource(c.createTombstoneSource())))
|
|
||||||
}
|
|
||||||
|
|
||||||
err = c.cfgObject.cfgLocalStorage.localStorage.Reload(ctx, rcfg)
|
|
||||||
if err != nil {
|
|
||||||
c.log.Error(logs.FrostFSNodeStorageEngineConfigurationUpdate, zap.Error(err))
|
|
||||||
return
|
|
||||||
}
|
|
||||||
|
|
||||||
for _, component := range components {
|
|
||||||
err = component.reloadFunc()
|
|
||||||
if err != nil {
|
|
||||||
c.log.Error(logs.FrostFSNodeUpdatedConfigurationApplying,
|
|
||||||
zap.String("component", component.name),
|
|
||||||
zap.Error(err))
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
c.log.Info(logs.FrostFSNodeConfigurationHasBeenReloadedSuccessfully)
|
|
||||||
}
|
}
|
||||||
|
|
||||||
func (c *cfg) reloadPools() error {
|
func (c *cfg) reloadPools() error {
|
||||||
|
|
Loading…
Reference in a new issue