Skip to content

Commit

Permalink
changing default otlp endpoint
Browse files Browse the repository at this point in the history
changing default otlp endpoint host to be consistent with other apps, and to include http:// which seems to be necessary
  • Loading branch information
EvanDietzMorris committed Nov 4, 2024
1 parent 363570b commit 4097d81
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/otel_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ def configure_otel(service_name, APP):
# these supresses such warnings.
logging.captureWarnings(capture=True)
warnings.filterwarnings("ignore",category=ResourceWarning)
jaeger_host = os.environ.get('JAEGER_HOST', 'jaeger-otel-collector')
jaeger_host = os.environ.get('JAEGER_HOST', 'http://localhost/').rstrip('/')
jaeger_port = int(os.environ.get('JAEGER_PORT', '4317'))
jaeger_endpoint = f'{jaeger_host}:{jaeger_port}'
otlp_exporter = OTLPSpanExporter(endpoint=jaeger_endpoint)
Expand Down

0 comments on commit 4097d81

Please sign in to comment.