Merge pull request #469 from versity/ben/cmd_exit_status

fix: return success exit status if shutdown succeeds
This commit is contained in:
Ben McClelland
2024-03-19 08:14:20 -07:00
committed by GitHub
+6 -1
View File
@@ -492,7 +492,6 @@ Loop:
for {
select {
case <-ctx.Done():
err = ctx.Err()
break Loop
case err = <-c:
break Loop
@@ -512,12 +511,18 @@ Loop:
err = iam.Shutdown()
if err != nil {
if saveErr == nil {
saveErr = err
}
fmt.Fprintf(os.Stderr, "shutdown iam: %v\n", err)
}
if logger != nil {
err := logger.Shutdown()
if err != nil {
if saveErr == nil {
saveErr = err
}
fmt.Fprintf(os.Stderr, "shutdown logger: %v\n", err)
}
}