🔧 Fix health checks: Use IPv4 address + Add debugging

🌐 Network Fix:
- Change localhost to 127.0.0.1 for all health check URLs
- Prevents IPv6 resolution issues in CI environment
- Ensures consistent IPv4 connectivity to container

🔍 Debugging Improvements:
- Check if container is running with docker ps
- Show recent container logs before health checks
- Better troubleshooting information for failures

📋 Updated Endpoints:
- http://127.0.0.1:8080/health
- http://127.0.0.1:8080/docs
- http://127.0.0.1:8080/stations
- http://127.0.0.1:8080/metrics

 Should resolve curl connection failures to localhost
This commit is contained in:
2025-08-13 12:15:36 +07:00
parent 18f77530ec
commit cd59236473

View File

@@ -3,16 +3,16 @@ name: Release - Northern Thailand Ping River Monitor
on: on:
push: push:
tags: tags:
- 'v*.*.*' - "v*.*.*"
workflow_dispatch: workflow_dispatch:
inputs: inputs:
version: version:
description: 'Release version (e.g., v3.1.3)' description: "Release version (e.g., v3.1.3)"
required: true required: true
type: string type: string
env: env:
PYTHON_VERSION: '3.11' PYTHON_VERSION: "3.11"
REGISTRY: git.b4l.co.th REGISTRY: git.b4l.co.th
IMAGE_NAME: b4l/northern-thailand-ping-river-monitor IMAGE_NAME: b4l/northern-thailand-ping-river-monitor
# GitHub token for better rate limits and authentication # GitHub token for better rate limits and authentication
@@ -71,7 +71,7 @@ jobs:
needs: create-release needs: create-release
strategy: strategy:
matrix: matrix:
python-version: ['3.9', '3.10', '3.11', '3.12'] python-version: ["3.9", "3.10", "3.11", "3.12"]
steps: steps:
- name: Checkout code - name: Checkout code
@@ -161,8 +161,6 @@ jobs:
with: with:
token: ${{ secrets.GITEA_TOKEN}} token: ${{ secrets.GITEA_TOKEN}}
# Test release deployment locally # Test release deployment locally
deploy-release: deploy-release:
name: Test Release Deployment name: Test Release Deployment
@@ -213,9 +211,16 @@ jobs:
echo "🔍 Running health checks against local container..." echo "🔍 Running health checks against local container..."
# Check if container is running
docker ps | grep ping-river-monitor-test || echo "⚠️ Container not found in docker ps"
# Check container logs for any startup issues
echo "📋 Recent container logs:"
docker logs --tail 10 ping-river-monitor-test || true
# Wait for the application to be ready # Wait for the application to be ready
for i in {1..12}; do for i in {1..12}; do
if curl -f http://localhost:8080/health; then if curl -f http://127.0.0.1:8080/health; then
echo "✅ Health endpoint responding" echo "✅ Health endpoint responding"
break break
else else
@@ -226,10 +231,10 @@ jobs:
# Test API endpoints # Test API endpoints
echo "🧪 Testing API endpoints..." echo "🧪 Testing API endpoints..."
curl -f http://localhost:8080/health || exit 1 curl -f http://127.0.0.1:8080/health || exit 1
curl -f http://localhost:8080/docs || exit 1 curl -f http://127.0.0.1:8080/docs || exit 1
curl -f http://localhost:8080/stations || exit 1 curl -f http://127.0.0.1:8080/stations || exit 1
curl -f http://localhost:8080/metrics || exit 1 curl -f http://127.0.0.1:8080/metrics || exit 1
echo "✅ All health checks passed!" echo "✅ All health checks passed!"
@@ -249,8 +254,6 @@ jobs:
echo "Status: Container tested successfully" echo "Status: Container tested successfully"
echo "Ready for production deployment" echo "Ready for production deployment"
# Notify stakeholders # Notify stakeholders
notify: notify:
name: Notify Release name: Notify Release