Skip to content

Commit 73f2d43

Browse files
committed
Update CI workflows to use 'docker compose' for log retrieval and added container status check
- Replaced 'docker logs' commands with 'docker compose -f docker-compose-test.yml logs' for consistency across workflows. - Added a check for running containers before saving logs to enhance debugging capabilities.
1 parent e2886e9 commit 73f2d43

File tree

4 files changed

+28
-20
lines changed

4 files changed

+28
-20
lines changed

.github/workflows/full-tests-with-api.yml

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -120,14 +120,17 @@ jobs:
120120
if: always()
121121
working-directory: backends/advanced
122122
run: |
123+
echo "Checking running containers..."
124+
docker compose -f docker-compose-test.yml ps -a
125+
echo ""
123126
echo "Saving service logs to files..."
124127
mkdir -p logs
125-
docker logs chronicle-backend-test > logs/backend.log 2>&1 || true
126-
docker logs workers-test > logs/workers.log 2>&1 || true
127-
docker logs mongo-test > logs/mongo.log 2>&1 || true
128-
docker logs redis-test > logs/redis.log 2>&1 || true
129-
docker logs qdrant-test > logs/qdrant.log 2>&1 || true
130-
docker logs speaker-service-test > logs/speaker.log 2>&1 || true
128+
docker compose -f docker-compose-test.yml logs chronicle-backend-test > logs/backend.log 2>&1 || true
129+
docker compose -f docker-compose-test.yml logs workers-test > logs/workers.log 2>&1 || true
130+
docker compose -f docker-compose-test.yml logs mongo-test > logs/mongo.log 2>&1 || true
131+
docker compose -f docker-compose-test.yml logs redis-test > logs/redis.log 2>&1 || true
132+
docker compose -f docker-compose-test.yml logs qdrant-test > logs/qdrant.log 2>&1 || true
133+
docker compose -f docker-compose-test.yml logs speaker-service-test > logs/speaker.log 2>&1 || true
131134
echo "✓ Logs saved to backends/advanced/logs/"
132135
ls -lh logs/
133136

.github/workflows/pr-tests-with-api.yml

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -114,14 +114,17 @@ jobs:
114114
if: always()
115115
working-directory: backends/advanced
116116
run: |
117+
echo "Checking running containers..."
118+
docker compose -f docker-compose-test.yml ps -a
119+
echo ""
117120
echo "Saving service logs to files..."
118121
mkdir -p logs
119-
docker logs chronicle-backend-test > logs/backend.log 2>&1 || true
120-
docker logs workers-test > logs/workers.log 2>&1 || true
121-
docker logs mongo-test > logs/mongo.log 2>&1 || true
122-
docker logs redis-test > logs/redis.log 2>&1 || true
123-
docker logs qdrant-test > logs/qdrant.log 2>&1 || true
124-
docker logs speaker-service-test > logs/speaker.log 2>&1 || true
122+
docker compose -f docker-compose-test.yml logs chronicle-backend-test > logs/backend.log 2>&1 || true
123+
docker compose -f docker-compose-test.yml logs workers-test > logs/workers.log 2>&1 || true
124+
docker compose -f docker-compose-test.yml logs mongo-test > logs/mongo.log 2>&1 || true
125+
docker compose -f docker-compose-test.yml logs redis-test > logs/redis.log 2>&1 || true
126+
docker compose -f docker-compose-test.yml logs qdrant-test > logs/qdrant.log 2>&1 || true
127+
docker compose -f docker-compose-test.yml logs speaker-service-test > logs/speaker.log 2>&1 || true
125128
echo "✓ Logs saved to backends/advanced/logs/"
126129
ls -lh logs/
127130

.github/workflows/robot-tests.yml

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -89,14 +89,17 @@ jobs:
8989
if: always()
9090
working-directory: backends/advanced
9191
run: |
92+
echo "Checking running containers..."
93+
docker compose -f docker-compose-test.yml ps -a
94+
echo ""
9295
echo "Saving service logs to files..."
9396
mkdir -p logs
94-
docker logs chronicle-backend-test > logs/backend.log 2>&1 || true
95-
docker logs workers-test > logs/workers.log 2>&1 || true
96-
docker logs mongo-test > logs/mongo.log 2>&1 || true
97-
docker logs redis-test > logs/redis.log 2>&1 || true
98-
docker logs qdrant-test > logs/qdrant.log 2>&1 || true
99-
docker logs speaker-service-test > logs/speaker.log 2>&1 || true
97+
docker compose -f docker-compose-test.yml logs chronicle-backend-test > logs/backend.log 2>&1 || true
98+
docker compose -f docker-compose-test.yml logs workers-test > logs/workers.log 2>&1 || true
99+
docker compose -f docker-compose-test.yml logs mongo-test > logs/mongo.log 2>&1 || true
100+
docker compose -f docker-compose-test.yml logs redis-test > logs/redis.log 2>&1 || true
101+
docker compose -f docker-compose-test.yml logs qdrant-test > logs/qdrant.log 2>&1 || true
102+
docker compose -f docker-compose-test.yml logs speaker-service-test > logs/speaker.log 2>&1 || true
100103
echo "✓ Logs saved to backends/advanced/logs/"
101104
ls -lh logs/
102105

extras/speaker-recognition/src/simple_speaker_recognition/api/routers/identification.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -320,9 +320,8 @@ async def diarize_identify_match(
320320
try:
321321
transcript = json.loads(transcript_data)
322322
words = transcript.get("words", [])
323-
full_text = transcript.get("text", "")
324323
except json.JSONDecodeError as e:
325-
raise HTTPException(400, f"Invalid transcript_data JSON: {str(e)}")
324+
raise HTTPException(400, f"Invalid transcript_data JSON: {str(e)}") from e
326325

327326
if not words:
328327
raise HTTPException(400, "No words found in transcript_data")

0 commit comments

Comments
 (0)