node: Possibly incorrect log level is used #1547

New issue

Open

opened 2024-12-09 09:45:36 +00:00 by a-savchuk · 2 comments

a-savchuk commented

2024-12-09 09:45:36 +00:00

Member

I believe these messages should have Error level instead of Info

 	if err != nil {
 		c.log.Info(ctx, logs.FrostFSNodeFailedToCreateNeoRPCClient,
 			zap.Any("endpoints", addresses),
 			zap.String("error", err.Error()),
 		)

	`if err := cli.SetGroupSignerScope(); err != nil {`
	`c.log.Info(ctx, logs.FrostFSNodeFailedToSetGroupSignerScopeContinueWithGlobal, zap.Error(err))`
	`}`

Describe the solution you'd like

Use Error log level instead of Info

Describe alternatives you've considered

Leave as is if the log level was chosen intentionally

## Is your feature request related to a problem? Please describe. I believe these messages should have `Error` level instead of `Info` https://git.frostfs.info/TrueCloudLab/frostfs-node/src/commit/7e542906ef9085a792e5ec5f7185bfdd9acecedb/cmd/frostfs-node/morph.go#L96-L100 https://git.frostfs.info/TrueCloudLab/frostfs-node/src/commit/7e542906ef9085a792e5ec5f7185bfdd9acecedb/cmd/frostfs-node/morph.go#L110-L112 ## Describe the solution you'd like Use `Error` log level instead of `Info` ## Describe alternatives you've considered Leave as is if the log level was chosen intentionally