phoenix-oss/llama-stack-mirror
0
0
Fork 1
mirror of https://github.com/meta-llama/llama-stack.git synced 2025-12-12 04:00:42 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(ci): improve workflow logging and bot notifications

Browse source 
- Link pre-commit bot comment to workflow run instead of PR for better debugging
- Dump docker container logs before removal to ensure logs are actually captured
This commit is contained in:
Ashwin Bharambe 2025-10-20 22:02:55 -07:00
parent 122de785c4
commit dd5182ceed
3 changed files with 6 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

7
.github/actions/run-and-record-tests/action.yml vendored
View file

@ -86,10 +86,9 @@ runs:
if: ${{ always() }} if: ${{ always() }}
shell: bash shell: bash
run: | run: |
sudo docker logs ollama > ollama-${{ inputs.inference-mode }}.log || true # Ollama logs (if ollama container exists)
distro_name=$(echo "${{ inputs.stack-config }}" | sed 's/^docker://' | sed 's/^server://') sudo docker logs ollama > ollama-${{ inputs.inference-mode }}.log 2>&1 || true
stack_container_name="llama-stack-test-$distro_name" # Note: distro container logs are now dumped in integration-tests.sh before container is removed
sudo docker logs $stack_container_name > docker-${distro_name}-${{ inputs.inference-mode }}.log || true
- name: Upload logs - name: Upload logs
if: ${{ always() }} if: ${{ always() }}

2
.github/workflows/precommit-trigger.yml vendored
View file

@ -99,7 +99,7 @@ jobs:
owner: context.repo.owner, owner: context.repo.owner,
repo: context.repo.repo, repo: context.repo.repo,
issue_number: ${{ steps.check_author.outputs.pr_number }}, issue_number: ${{ steps.check_author.outputs.pr_number }},
body: `⏳ Running pre-commit hooks on PR #${{ steps.check_author.outputs.pr_number }}...` body: `⏳ Running [pre-commit hooks](https://github.com/${context.repo.owner}/${context.repo.repo}/actions/runs/${context.runId}) on PR #${{ steps.check_author.outputs.pr_number }}...`
}); });
- name: Checkout PR branch (same-repo) - name: Checkout PR branch (same-repo)

2
scripts/integration-tests.sh
View file

@ -238,6 +238,8 @@ if [[ "$STACK_CONFIG" == *"docker:"* && "$COLLECT_ONLY" == false ]]; then
echo "Stopping Docker container..." echo "Stopping Docker container..."
container_name="llama-stack-test-$DISTRO" container_name="llama-stack-test-$DISTRO"
if docker ps -a --format '{{.Names}}' | grep -q "^${container_name}$"; then if docker ps -a --format '{{.Names}}' | grep -q "^${container_name}$"; then
echo "Dumping container logs before stopping..."
docker logs "$container_name" > "docker-${DISTRO}-${INFERENCE_MODE}.log" 2>&1 || true
echo "Stopping and removing container: $container_name" echo "Stopping and removing container: $container_name"
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