From ccf6bf1b0de5aed1fd4891b90d99c09879a8277b Mon Sep 17 00:00:00 2001 From: Ralf Haferkamp Date: Mon, 28 Feb 2022 17:52:51 +0100 Subject: [PATCH] idm: Fix error handling on startup (#3243) * Fix idm to acutally return error during startup to the runtime * Add proper conditon to error log statement --- idm/pkg/command/server.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/idm/pkg/command/server.go b/idm/pkg/command/server.go index eee544ac1f..adfe31850a 100644 --- a/idm/pkg/command/server.go +++ b/idm/pkg/command/server.go @@ -40,8 +40,7 @@ func Server(cfg *config.Config) *cli.Command { }() defer cancel() - start(ctx, logger, cfg) - return nil + return start(ctx, logger, cfg) }, } } @@ -67,8 +66,9 @@ func start(ctx context.Context, logger log.Logger, cfg *config.Config) error { } if _, err := os.Stat(servercfg.BoltDBFile); errors.Is(err, os.ErrNotExist) { logger.Debug().Msg("Bootstrapping IDM database") - err = bootstrap(logger, cfg, servercfg) - logger.Error().Err(err).Msg("failed") + if err = bootstrap(logger, cfg, servercfg); err != nil { + logger.Error().Err(err).Msg("failed to bootstrap idm database") + } } svc, err := server.NewServer(&servercfg)