Merge pull request #247 from movio/opentelemetry

feat: OpenTelemetry env url

docs/configuration.md

@@ -98,7 +98,7 @@ Sample configuration:
     - Default: `false`
     - Supports hot-reload: No
   - `endpoint`: OpenTelemetry collector endpoint.
-    - Default: If no endpoint is specified, telemetry is disabled.
+    - Default: If no endpoint is specified, telemetry is disabled. Bramble will check for `BRAMBLE_OTEL_ENDPOINT` environment variable and use it if set.
     - Supports hot-reload: No
   - `serviceName`: Service name to use for OpenTelemetry.
     - Default: `bramble`

telemetry.go

@@ -3,6 +3,7 @@ package bramble
 import (
 	"context"
 	"errors"
+	"os"
 
 	"github.com/sirupsen/logrus"
 	"go.opentelemetry.io/otel"
@@ -41,6 +42,11 @@ func (e *TelemetryErrHandler) Handle(err error) {
 // returns a shutdown function that should be called when the application
 // terminates.
 func InitTelemetry(ctx context.Context, cfg TelemetryConfig) (func(context.Context) error, error) {
+	endpoint := os.Getenv("BRAMBLE_OTEL_ENDPOINT")
+	if endpoint != "" {
+		cfg.Endpoint = endpoint
+	}
+
 	// If telemetry is disabled, return a no-op shutdown function. The standard
 	// behaviour of the application will not be affected, since a
 	// `NoopTracerProvider` is used by default.