From 38906420c48cfbc6a55e0dc00ec21a8a8fd7b95a Mon Sep 17 00:00:00 2001 From: Kian Jones Date: Tue, 9 Sep 2025 10:46:45 -0700 Subject: [PATCH] revert to old docker integration test to unblock --- .../workflows/docker-integration-tests.yaml | 87 +++++++++++++------ 1 file changed, 60 insertions(+), 27 deletions(-) diff --git a/.github/workflows/docker-integration-tests.yaml b/.github/workflows/docker-integration-tests.yaml index 36add10e..4dceb127 100644 --- a/.github/workflows/docker-integration-tests.yaml +++ b/.github/workflows/docker-integration-tests.yaml @@ -1,33 +1,66 @@ name: Run Docker integration tests on: + push: + branches: [ main ] pull_request: - branches: - - main - pull_request_target: - branches: - - main - types: [labeled] - -concurrency: - group: docker-tests-${{ github.ref }} - cancel-in-progress: true + branches: [ main ] jobs: - docker-tests: - # Run on pull_request OR on pull_request_target only when labeled "safe to test" - if: github.event_name == 'pull_request' || (github.event_name == 'pull_request_target' && contains(github.event.pull_request.labels.*.name, 'safe to test')) - uses: ./.github/workflows/reusable-test-workflow.yml - with: - test-type: 'docker' - install-args: '--extra dev --extra postgres --extra sqlite' - timeout-minutes: 15 - use-docker: true - runner: '["self-hosted", "medium"]' - ref: ${{ github.event.pull_request.head.sha || github.sha }} - changed-files-pattern: | - apps/core/** - libs/config-core-deploy/** - .github/workflows/reusable-test-workflow.yml - .github/workflows/docker-integration-tests.yaml - secrets: inherit + test: + runs-on: ubuntu-latest + timeout-minutes: 15 + steps: + - name: Checkout + uses: actions/checkout@v4 + + - name: Set up python 3.11 + id: setup-python + uses: actions/setup-python@v5 + with: + python-version: 3.11 + + - name: Install uv + uses: astral-sh/setup-uv@v6 + with: + enable-cache: true + + - name: Set permissions for log directory + run: | + mkdir -p /home/runner/.letta/logs + sudo chown -R $USER:$USER /home/runner/.letta/logs + chmod -R 755 /home/runner/.letta/logs + + - name: Build and run docker dev server + env: + LETTA_PG_DB: letta + LETTA_PG_USER: letta + LETTA_PG_PASSWORD: letta + LETTA_PG_PORT: 8888 + OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }} + + run: | + docker compose -f dev-compose.yaml up --build -d + + - name: Wait for service + run: bash scripts/wait_for_service.sh http://localhost:8283 -- echo "Service is ready" + + - name: Run tests with pytest + env: + LETTA_PG_DB: letta + LETTA_PG_USER: letta + LETTA_PG_PASSWORD: letta + LETTA_PG_PORT: 8888 + LETTA_SERVER_PASS: test_server_token + LETTA_SERVER_URL: http://localhost:8283 + OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }} + PYTHONPATH: ${{ github.workspace }}:${{ env.PYTHONPATH }} + run: | + uv sync --extra dev --extra postgres --extra sqlite + uv run pytest -s tests/test_client.py + + - name: Print docker logs if tests fail + if: failure() + run: | + echo "Printing Docker Logs..." + docker compose -f dev-compose.yaml logs