Merge pull request #116 from march1993/master

fix some issues

cmd/cmd.go

@@ -30,23 +30,24 @@ import (
 )
 
 func NewCommand() *cobra.Command {
-	var cli *client.Client
+	var cli client.Client
 	var host string = "tcp://127.0.0.1:23457"
 	var cmd = &cobra.Command{
 		Use:   "gotgt",
 		Short: "Gotgt is a very fast and stable SCSI target framework",
 		Long:  ``,
-		RunE: func(cmd *cobra.Command, args []string) error {
+		PersistentPreRunE: func(cmd *cobra.Command, args []string) error {
 			httpClient, err := newHTTPClient(host)
 			if err != nil {
 				fmt.Fprintf(os.Stderr, "%v", err)
 				return err
 			}
 
-			cli, err = client.NewClient(host, version.Version, httpClient, nil)
-			if err != nil {
+			if _cli, err := client.NewClient(host, version.Version, httpClient, nil); err != nil {
 				fmt.Fprintf(os.Stderr, "%v", err)
 				return err
+			} else {
+				cli = *_cli
 			}
 			// Do Stuff Here
 			return nil
@@ -56,10 +57,10 @@ func NewCommand() *cobra.Command {
 	cmd.PersistentFlags().StringVar(&host, "host", host, "Endpoint for SCSI target daemon")
 	cmd.AddCommand(
 		newDaemonCommand(),
-		newCreateCommand(cli),
-		newRemoveCommand(cli),
-		newListCommand(cli),
-		newVersionCommand(cli),
+		newCreateCommand(&cli),
+		newRemoveCommand(&cli),
+		newListCommand(&cli),
+		newVersionCommand(&cli),
 	)
 	return cmd
 }

examples/config.json

@@ -1,27 +1,29 @@
 {
-  "storages":[
+  "storages": [
     {
-      "deviceID":1000,
-      "path":"file:/var/tmp/disk.img",
-      "online":true,
-      "thinProvisioning":false,
-      "blockShift": 9,
+      "deviceID": 1000,
+      "path": "file:/var/tmp/disk.img",
+      "online": true,
+      "thinProvisioning": false,
+      "blockShift": 9
     }
   ],
-  "iscsiportals":[
+  "iscsiportals": [
     {
-      "id":0,
-      "portal":"127.0.0.1:3260"
+      "id": 0,
+      "portal": "127.0.0.1:3260"
     }
   ],
-  "iscsitargets":{
-    "iqn.2016-09.com.gotgt.gostor:02:example-tgt-0":{
-      "tpgts":{
-        "1":[0]
+  "iscsitargets": {
+    "iqn.2016-09.com.gotgt.gostor:02:example-tgt-0": {
+      "tpgts": {
+        "1": [
+          0
+        ]
       },
-      "luns":{
-        "0":1000
+      "luns": {
+        "0": 1000
       }
     }
   }
-}
+}

pkg/api/client/target_list.go

@@ -5,7 +5,7 @@ Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.
 You may obtain a copy of the License at
 
-    http://www.apache.org/licenses/LICENSE-2.0
+	http://www.apache.org/licenses/LICENSE-2.0
 
 Unless required by applicable law or agreed to in writing, software
 distributed under the License is distributed on an "AS IS" BASIS,

pkg/apiserver/apiserver.go

@@ -39,7 +39,7 @@ import (
 
 // versionMatcher defines a variable matcher to be parsed by the router
 // when a request is about to be served.
-const versionMatcher = "/v{version:[0-9.]+}"
+const versionMatcher = "/v{version:[0-9.]+(?:-dirty)}"
 
 // Config provides the configuration for the API server
 type Config struct {

pkg/version/version.go

@@ -23,7 +23,7 @@ import (
 	"runtime"
 )
 
-const (
+var (
 	// Version shows the version of gotgt.
 	Version = "Not provided."
 	// SCSIVersion string MUST be shorter than 4 characters