Skip to content
This repository was archived by the owner on Apr 17, 2019. It is now read-only.

Commit 0985397

Browse files
authored
Merge pull request #1215 from mwielgus/cloud-error
Cluster-autoscaler: cloud config error handling
2 parents fe721a9 + c4f2001 commit 0985397

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

cluster-autoscaler/cluster_autoscaler.go

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -82,17 +82,18 @@ func main() {
8282

8383
// GCE Manager
8484
var gceManager *gce.GceManager
85+
var gceError error
8586
if *cloudConfig != "" {
86-
config, err := os.Open(*cloudConfig)
87-
if err != nil {
87+
config, fileErr := os.Open(*cloudConfig)
88+
if fileErr != nil {
8889
glog.Fatalf("Couldn't open cloud provider configuration %s: %#v", *cloudConfig, err)
8990
}
9091
defer config.Close()
91-
gceManager, err = gce.CreateGceManager(migConfigs, config)
92+
gceManager, gceError = gce.CreateGceManager(migConfigs, config)
9293
} else {
93-
gceManager, err = gce.CreateGceManager(migConfigs, nil)
94+
gceManager, gceError = gce.CreateGceManager(migConfigs, nil)
9495
}
95-
if err != nil {
96+
if gceError != nil {
9697
glog.Fatalf("Failed to create GCE Manager: %v", err)
9798
}
9899

0 commit comments

Comments
 (0)