From bef7577fd9b4e0f8f63e2842fd71a502bbe9e366 Mon Sep 17 00:00:00 2001 From: BukeLy Date: Thu, 20 Nov 2025 00:35:03 +0800 Subject: [PATCH] fix: correct PostgreSQL environment variable name in E2E workflow Why this change is needed: E2E tests were failing with: "ValueError: Storage implementation 'PGKVStorage' requires the following environment variables: POSTGRES_DATABASE" The workflow was setting POSTGRES_DB but LightRAG's check_storage_env_vars() expects POSTGRES_DATABASE (matching ClientManager.get_config()). How it solves it: Changed environment variable name from POSTGRES_DB to POSTGRES_DATABASE in the "Run PostgreSQL E2E tests" step. Impact: - PGKVStorage, PGGraphStorage, and PGDocStatusStorage can now properly initialize using ClientManager's configuration - Fixes ValueError during LightRAG initialization Testing: Next E2E run should pass environment variable validation and proceed to actual test execution. --- .github/workflows/e2e-tests.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/e2e-tests.yml b/.github/workflows/e2e-tests.yml index a48edec7..98c9f5cc 100644 --- a/.github/workflows/e2e-tests.yml +++ b/.github/workflows/e2e-tests.yml @@ -73,7 +73,7 @@ jobs: POSTGRES_PORT: 5432 POSTGRES_USER: lightrag POSTGRES_PASSWORD: lightrag_test_password - POSTGRES_DB: lightrag_test + POSTGRES_DATABASE: lightrag_test run: | pytest tests/test_e2e_multi_instance.py -k "postgres" -v --tb=short -s timeout-minutes: 20