Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 26 additions & 0 deletions tests/e2e/configuration/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,32 @@ This directory contains configuration files used for end-to-end testing of Light
- `server-mode/` - Configurations for testing when LCore connects to a separate Llama Stack service
- `library-mode/` - Configurations for testing when LCore embeds Llama Stack as a library

## Library mode uses unified configs (LCORE-2342)

The library-mode configurations use the unified single-file format: instead of
the legacy `llama_stack.library_client_config_path`, they carry

```yaml
llama_stack:
use_as_library_client: true
config:
profile: run.yaml
```

The harness/CI copies the provider-specific run config
(`tests/e2e/configs/run-<environment>.yaml`) to `./run.yaml` in the repo root,
and the active `lightspeed-stack.yaml` is also copied to the repo root — so the
relative `profile:` path resolves to that materialized file, which the unified
synthesizer consumes as its baseline. LS behavior is identical to the legacy
two-file path (requirement R7 in the config-merge design doc); the wiring that
selects a provider config stays unchanged.

The `tests/e2e/configs/run-*.yaml` files therefore serve a dual role: in
server mode they are the run configuration of the standalone Llama Stack
service, and in library mode they are consumed as the unified-mode synthesis
profile. No in-repo test config references them via the legacy mechanism
anymore.

Comment thread
coderabbitai[bot] marked this conversation as resolved.
## Common Configuration Features

### Default Configurations (`lightspeed-stack.yaml`)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,10 @@ service:
access_log: true
llama_stack:
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,10 @@ service:
access_log: true
llama_stack:
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,10 @@ service:
access_log: true
llama_stack:
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,10 @@ service:
access_log: true
llama_stack:
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,10 @@ service:
access_log: true
llama_stack:
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/invalid"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,10 @@ service:
llama_stack:
# Library mode - embeds llama-stack as library
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,10 @@ service:
llama_stack:
# Library mode - embeds llama-stack as library
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,10 @@ service:
llama_stack:
# Library mode - embeds llama-stack as library
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,10 @@ service:
llama_stack:
# Library mode - embeds llama-stack as library
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,10 @@ service:
llama_stack:
# Library mode - embeds llama-stack as library
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,10 @@ service:
llama_stack:
# Library mode - embeds llama-stack as library
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,10 @@ service:
llama_stack:
# Library mode - embeds llama-stack as library
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,10 @@ service:
access_log: true
llama_stack:
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,10 @@ service:

llama_stack:
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml

user_data_collection:
feedback_enabled: true
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,10 @@ service:
llama_stack:
# Library mode - embeds llama-stack as library
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,10 @@ service:
llama_stack:
# Library mode - embeds llama-stack as library
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
5 changes: 4 additions & 1 deletion tests/e2e/configuration/library-mode/lightspeed-stack.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,10 @@ service:
llama_stack:
# Library mode - embeds llama-stack as library
use_as_library_client: true
library_client_config_path: run.yaml
# Unified mode: run.yaml (materialized per provider by CI/the harness)
# is consumed as the synthesis profile instead of the legacy two-file path.
config:
profile: run.yaml
user_data_collection:
feedback_enabled: true
feedback_storage: "/tmp/data/feedback"
Expand Down
Loading