Skip to content
This repository was archived by the owner on Mar 22, 2018. It is now read-only.

Commit a303cc3

Browse files
committed
Fix logs and add version information
1 parent e5b66e7 commit a303cc3

1 file changed

Lines changed: 26 additions & 13 deletions

File tree

cmd/cinder-provisioner/main.go

Lines changed: 26 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -17,39 +17,52 @@ limitations under the License.
1717
package main
1818

1919
import (
20-
flag "github.com/spf13/pflag"
20+
"flag"
21+
"github.com/golang/glog"
22+
"github.com/spf13/pflag"
2123

2224
"git.openstack.org/openstack/openstack-cloud-controller-manager/pkg/volume/cinder/provisioner"
23-
"github.com/golang/glog"
2425
"github.com/kubernetes-incubator/external-storage/lib/controller"
2526

2627
"k8s.io/apimachinery/pkg/util/wait"
28+
kflag "k8s.io/apiserver/pkg/util/flag"
29+
"k8s.io/apiserver/pkg/util/logs"
2730
"k8s.io/client-go/kubernetes"
2831
"k8s.io/client-go/rest"
2932
"k8s.io/client-go/tools/clientcmd"
3033
)
3134

3235
var (
33-
master = flag.String("master", "", "Master URL")
34-
kubeconfig = flag.String("kubeconfig", "", "Absolute path to the kubeconfig")
35-
id = flag.String("id", "", "Unique provisioner identity")
36-
cloudconfig = flag.String("cloud-config", "", "Path to OpenStack config file")
36+
master string
37+
kubeconfig string
38+
id string
39+
cloudconfig string
40+
version string
3741
)
3842

3943
func main() {
40-
flag.Parse()
41-
flag.Set("logtostderr", "true")
44+
flag.CommandLine.Parse([]string{})
45+
pflag.StringVar(&master, "master", "", "Master URL")
46+
pflag.StringVar(&kubeconfig, "kubeconfig", "", "Absolute path to the kubeconfig")
47+
pflag.StringVar(&id, "id", "", "Unique provisioner identity")
48+
pflag.StringVar(&cloudconfig, "cloud-config", "", "Path to OpenStack config file")
49+
50+
kflag.InitFlags()
51+
logs.InitLogs()
52+
defer logs.FlushLogs()
53+
54+
glog.V(1).Infof("cinder-provisioner version: %s", version)
4255

4356
var config *rest.Config
4457
var err error
45-
if *master != "" || *kubeconfig != "" {
46-
config, err = clientcmd.BuildConfigFromFlags(*master, *kubeconfig)
58+
if master != "" || kubeconfig != "" {
59+
config, err = clientcmd.BuildConfigFromFlags(master, kubeconfig)
4760
} else {
4861
config, err = rest.InClusterConfig()
4962
}
5063
prID := provisioner.ProvisionerName
51-
if *id != "" {
52-
prID = *id
64+
if id != "" {
65+
prID = id
5366
}
5467
if err != nil {
5568
glog.Fatalf("Failed to create config: %v", err)
@@ -68,7 +81,7 @@ func main() {
6881

6982
// Create the provisioner: it implements the Provisioner interface expected by
7083
// the controller
71-
cinderProvisioner, err := provisioner.NewCinderProvisioner(clientset, prID, *cloudconfig)
84+
cinderProvisioner, err := provisioner.NewCinderProvisioner(clientset, prID, cloudconfig)
7285
if err != nil {
7386
glog.Fatalf("Error creating Cinder provisioner: %v", err)
7487
}

0 commit comments

Comments
 (0)