mirror of
https://github.com/meta-llama/llama-stack.git
synced 2025-12-11 19:56:03 +00:00
fix: Fix missing telemetry env vars in server mode
LLAMA_STACK_TEST_COLLECTOR_PORT and OTEL_EXPORTER_OTLP_ENDPOINT were only set in docker mode, causing the telemetry tests to fail in "server" record mode. Closes: #3993 Signed-off-by: Derek Higgins <derekh@redhat.com>
This commit is contained in:
parent
5e20938832
commit
a80ebf18e3
1 changed files with 5 additions and 4 deletions
|
|
@ -159,6 +159,11 @@ if [[ "$COLLECT_ONLY" == false ]]; then
|
||||||
if [[ "$STACK_CONFIG" == server:* ]] || [[ "$STACK_CONFIG" == docker:* ]]; then
|
if [[ "$STACK_CONFIG" == server:* ]] || [[ "$STACK_CONFIG" == docker:* ]]; then
|
||||||
export LLAMA_STACK_TEST_STACK_CONFIG_TYPE="server"
|
export LLAMA_STACK_TEST_STACK_CONFIG_TYPE="server"
|
||||||
echo "Setting stack config type: server"
|
echo "Setting stack config type: server"
|
||||||
|
|
||||||
|
# Configure telemetry collector port for both server and docker modes
|
||||||
|
COLLECTOR_PORT=4317
|
||||||
|
export LLAMA_STACK_TEST_COLLECTOR_PORT="${COLLECTOR_PORT}"
|
||||||
|
export OTEL_EXPORTER_OTLP_ENDPOINT="http://localhost:${COLLECTOR_PORT}"
|
||||||
else
|
else
|
||||||
export LLAMA_STACK_TEST_STACK_CONFIG_TYPE="library_client"
|
export LLAMA_STACK_TEST_STACK_CONFIG_TYPE="library_client"
|
||||||
echo "Setting stack config type: library_client"
|
echo "Setting stack config type: library_client"
|
||||||
|
|
@ -284,10 +289,6 @@ if [[ "$STACK_CONFIG" == *"docker:"* && "$COLLECT_ONLY" == false ]]; then
|
||||||
docker stop "$container_name" 2>/dev/null || true
|
docker stop "$container_name" 2>/dev/null || true
|
||||||
docker rm "$container_name" 2>/dev/null || true
|
docker rm "$container_name" 2>/dev/null || true
|
||||||
|
|
||||||
# Configure telemetry collector port shared between host and container
|
|
||||||
COLLECTOR_PORT=4317
|
|
||||||
export LLAMA_STACK_TEST_COLLECTOR_PORT="${COLLECTOR_PORT}"
|
|
||||||
|
|
||||||
# Build environment variables for docker run
|
# Build environment variables for docker run
|
||||||
DOCKER_ENV_VARS=""
|
DOCKER_ENV_VARS=""
|
||||||
DOCKER_ENV_VARS="$DOCKER_ENV_VARS -e LLAMA_STACK_TEST_INFERENCE_MODE=$INFERENCE_MODE"
|
DOCKER_ENV_VARS="$DOCKER_ENV_VARS -e LLAMA_STACK_TEST_INFERENCE_MODE=$INFERENCE_MODE"
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue