Skip to content

Commit

Permalink
Merge pull request kubernetes#30184 from ping035627/ping035627-patch-…
Browse files Browse the repository at this point in the history
…0806

Automatic merge from submit-queue

some optimization for server.go

The PR modified two places:
1) Optimise the code style according to the go style guide;
2) Optimise the log.
  • Loading branch information
Kubernetes Submit Queue authored Aug 12, 2016
2 parents 19b2dd1 + 9f3d83e commit d933f07
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions cmd/kube-dns/app/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -91,8 +91,7 @@ func newKubeClient(dnsConfig *options.KubeDNSConfig) (clientset.Interface, error
}
}

glog.Infof("Using %s for kubernetes master", config.Host)
glog.Infof("Using kubernetes API %v", config.GroupVersion)
glog.Infof("Using %s for kubernetes master, kubernetes API: %v", config.Host, config.GroupVersion)
return clientset.NewForConfig(config)
}

Expand Down Expand Up @@ -142,8 +141,8 @@ func (d *KubeDNSServer) startSkyDNSServer() {
s := server.New(d.kd, skydnsConfig)
if err := metrics.Metrics(); err != nil {
glog.Fatalf("skydns: %s", err)
} else {
glog.Infof("skydns: metrics enabled on :%s%s", metrics.Port, metrics.Path)
}
glog.Infof("skydns: metrics enabled on : %s:%s", metrics.Path, metrics.Port)

go s.Run()
}

0 comments on commit d933f07

Please sign in to comment.