Merge pull request #642 from kuba--/enh-638/bblfsh-tracing

Use global open tracer

Author: ajnavarro

Gopkg.lock

@@ -54,15 +54,15 @@
 
 [[constraint]]
   name = "gopkg.in/bblfsh/client-go.v3"
-  version = "3.1.0"
+  version = "3.2.0"
 
 [[override]]
   name = "gopkg.in/bblfsh/sdk.v1"
   version = "1.16.0"
 
 [[constraint]]
   name = "gopkg.in/bblfsh/sdk.v2"
-  version = "2.7.0"
+  version = "2.11.0"
 
 [[constraint]]
   name = "github.com/uber/jaeger-client-go"

Gopkg.toml

@@ -146,23 +146,19 @@ func (c *Server) Execute(args []string) error {
 				Fatal("unable to read jaeger environment")
 			return err
 		}
-
 		if cfg.ServiceName == "" {
 			cfg.ServiceName = TracerServiceName
 		}
 
 		logger := &jaegerLogrus{logrus.WithField("subsystem", "jaeger")}
 
-		t, closer, err := cfg.NewTracer(
-			config.Logger(logger),
-		)
-
+		closer, err := cfg.InitGlobalTracer(cfg.ServiceName, config.Logger(logger))
 		if err != nil {
-			logrus.WithField("error", err).Fatal("unable to initialize tracer")
+			logrus.WithField("error", err).Fatal("unable to initialize global tracer")
 			return err
 		}
 
-		tracer = t
+		tracer = opentracing.GlobalTracer()
 		defer closer.Close()
 
 		logrus.Info("tracing enabled")

cmd/gitbase/command/server.go

@@ -4,7 +4,7 @@
 
 `gitbase` optional dependencies that should be running on your system if you're planning on using certain functionality.
 
-- [bblfsh](https://github.com/bblfsh/bblfshd) >= 2.6.1 (only if you're planning to use the `UAST` functionality provided in gitbase)
+- [bblfsh](https://github.com/bblfsh/bblfshd) >= 2.10.0 (only if you're planning to use the `UAST` functionality provided in gitbase)
 
 ## Installing gitbase