Skip to content

Commit 48875d3

Browse files
Merge pull request #1006 from flavianmissi/fix-panic-move-blobs
OCPBUGS-29932: cmd/move-blobs: log and exit 1 on error instead of panic
2 parents 41ea2a9 + f3334ec commit 48875d3

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

cmd/move-blobs/main.go

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ package main
33
import (
44
"context"
55
"fmt"
6+
"log"
67
"net/url"
78
"os"
89
"strings"
@@ -22,7 +23,7 @@ import (
2223
func main() {
2324
opts := getConfigOpts()
2425
if err := validate(opts); err != nil {
25-
panic(err)
26+
log.Fatal(err)
2627
}
2728

2829
// if the environment specific configs are not given, assume
@@ -32,25 +33,25 @@ func main() {
3233
}
3334

3435
if err := createASHEnvironmentFile(opts); err != nil {
35-
panic(err)
36+
log.Fatal(err)
3637
}
3738

3839
cloudConfig, err := getCloudConfig(opts.environment)
3940
if err != nil {
40-
panic(err)
41+
log.Fatal(err)
4142
}
4243

4344
client, err := getClient(cloudConfig, opts)
4445
if err != nil {
45-
panic(err)
46+
log.Fatal(err)
4647
}
4748
ctx := context.Background()
4849
_, err = moveBlobs(ctx, client, &moveBlobOpts{
4950
source: "docker",
5051
dest: "/docker",
5152
})
5253
if err != nil {
53-
panic(err)
54+
log.Fatal(err)
5455
}
5556
}
5657

0 commit comments

Comments
 (0)