Merge pull request #13 from smacker/fix_analyzer_on_staging

Fix analyzer on staging

Author: Rafael Porres Molina

.helm_staging.yml

@@ -1,5 +1,6 @@
 app:
   dataServiceUrl: ipv4://lookout-work:10301
+  port: 9930
 
 nodeSelector:
   srcd.host/app: lookout

Gopkg.lock

@@ -39,7 +39,7 @@
 
 [[constraint]]
   name = "gopkg.in/src-d/lookout-sdk.v0"
-  version = "0.1.2"
+  version = "0.3.0"
 
 [prune]
   go-tests = true

Gopkg.toml

@@ -9,7 +9,7 @@ Install the analyzer, and run it:
 
 ```shell
 $ go install github.com/src-d/lookout-terraform-analyzer/cmd/lookout-terraform-analyzer
-$ PORT=2001 lookout-terraform-analyzer
+$ lookout-terraform-analyzer
 ```
 
 The analyzer will start listening for pull review requests from lookout.
@@ -28,7 +28,7 @@ $ wget -O - https://raw.githubusercontent.com/src-d/lookout-sdk/master/_tools/in
 And then run:
 
 ```shell
-$ ./lookout-sdk review ipv4://localhost:2001
+$ ./lookout-sdk review
 ```
 
-It will mock a Pull Request containing the changes made by `HEAD` over `HEAD~1`, and it will send it to `lookout-terraform-analyzer` that you ran in the [previous step](#usage) and is listening under the specified port (dafault is `2001`)
+It will mock a Pull Request containing the changes made by `HEAD` over `HEAD~1`, and it will send it to `lookout-terraform-analyzer` that you ran in the [previous step](#usage).

README.md

@@ -1,6 +1,7 @@
 package main
 
 import (
+	"context"
 	"fmt"
 	"net"
 	"os"
@@ -11,7 +12,7 @@ import (
 
 	"github.com/kelseyhightower/envconfig"
 	"google.golang.org/grpc"
-	"gopkg.in/src-d/go-log.v1"
+	log "gopkg.in/src-d/go-log.v1"
 )
 
 var (
@@ -24,8 +25,8 @@ const maxMessageSize = 100 * 1024 * 1024 //100mb
 
 type config struct {
 	Host           string `envconfig:"HOST" default:"0.0.0.0"`
-	Port           int    `envconfig:"PORT" default:"2001"`
-	DataServiceURL string `envconfig:"DATA_SERVICE_URL" default:"localhost:10301"`
+	Port           int    `envconfig:"PORT" default:"9930"`
+	DataServiceURL string `envconfig:"DATA_SERVICE_URL" default:"ipv4://localhost:10301"`
 	LogLevel       string `envconfig:"LOG_LEVEL" default:"info"`
 }
 
@@ -36,9 +37,14 @@ func main() {
 	log.DefaultFactory = &log.LoggerFactory{Level: conf.LogLevel}
 	log.DefaultLogger = log.New(nil)
 
-	grpcAddr := conf.DataServiceURL
+	grpcAddr, err := pb.ToGoGrpcAddress(conf.DataServiceURL)
+	if err != nil {
+		log.Errorf(err, "failed to parse DataService address %s", conf.DataServiceURL)
+		return
+	}
 
-	conn, err := grpc.Dial(
+	conn, err := pb.DialContext(
+		context.Background(),
 		grpcAddr,
 		grpc.WithInsecure(),
 		grpc.WithDefaultCallOptions(grpc.FailFast(false)),