bakery-admin/bakery-ia
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: Fix orchestrator dashboard service errors

Browse Source 
Fixed two critical issues preventing dashboard from loading:

1. OrchestrationStatus enum case mismatch:
   - Changed OrchestrationStatus.COMPLETED → .completed
   - Changed OrchestrationStatus.COMPLETED_WITH_WARNINGS → .partial_success
   - Enum values are lowercase, not uppercase

2. Service hostname resolution errors:
   - Fixed all service URLs to include -service suffix:
     - http://inventory:8000 → http://inventory-service:8000
     - http://production:8000 → http://production-service:8000
     - http://procurement:8000 → http://procurement-service:8000
     - http://alert-processor:8000 → http://alert-processor-service:8000

This fixes the AttributeError and "Name or service not known" errors
preventing the dashboard from loading demo data.
This commit is contained in:
Claude
2025-11-07 21:53:51 +00:00
parent eabd5eedc7
commit b732c0742b
2 changed files with 15 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
services/orchestrator/app/api/dashboard.py
View File

@@ -181,7 +181,7 @@ async def get_bakery_health_status(
# Get alerts # Get alerts
try: try:
alerts_response = await client.get( alerts_response = await client.get(
f"http://alert-processor:8000/api/v1/tenants/{tenant_id}/alerts/summary" f"http://alert-processor-service:8000/api/v1/tenants/{tenant_id}/alerts/summary"
) )
alerts_data = alerts_response.json() if alerts_response.status_code == 200 else {} alerts_data = alerts_response.json() if alerts_response.status_code == 200 else {}
critical_alerts = alerts_data.get("critical_count", 0) critical_alerts = alerts_data.get("critical_count", 0)
@@ -192,7 +192,7 @@ async def get_bakery_health_status(
# Get pending PO count # Get pending PO count
try: try:
po_response = await client.get( po_response = await client.get(
f"http://procurement:8000/api/v1/tenants/{tenant_id}/purchase-orders", f"http://procurement-service:8000/api/v1/tenants/{tenant_id}/purchase-orders",
params={"status": "pending_approval", "limit": 100} params={"status": "pending_approval", "limit": 100}
) )
po_data = po_response.json() if po_response.status_code == 200 else {} po_data = po_response.json() if po_response.status_code == 200 else {}
@@ -204,7 +204,7 @@ async def get_bakery_health_status(
# Get production delays # Get production delays
try: try:
prod_response = await client.get( prod_response = await client.get(
f"http://production:8000/api/v1/tenants/{tenant_id}/production-batches", f"http://production-service:8000/api/v1/tenants/{tenant_id}/production-batches",
params={"status": "ON_HOLD", "limit": 100} params={"status": "ON_HOLD", "limit": 100}
) )
prod_data = prod_response.json() if prod_response.status_code == 200 else {} prod_data = prod_response.json() if prod_response.status_code == 200 else {}
@@ -216,7 +216,7 @@ async def get_bakery_health_status(
# Get inventory status # Get inventory status
try: try:
inv_response = await client.get( inv_response = await client.get(
f"http://inventory:8000/api/v1/tenants/{tenant_id}/inventory/dashboard/stock-status" f"http://inventory-service:8000/api/v1/tenants/{tenant_id}/inventory/dashboard/stock-status"
) )
inv_data = inv_response.json() if inv_response.status_code == 200 else {} inv_data = inv_response.json() if inv_response.status_code == 200 else {}
out_of_stock_count = inv_data.get("out_of_stock_count", 0) out_of_stock_count = inv_data.get("out_of_stock_count", 0)
@@ -270,7 +270,7 @@ async def get_orchestration_summary(
async with httpx.AsyncClient(timeout=10.0) as client: async with httpx.AsyncClient(timeout=10.0) as client:
try: try:
po_response = await client.get( po_response = await client.get(
f"http://procurement:8000/api/v1/tenants/{tenant_id}/purchase-orders", f"http://procurement-service:8000/api/v1/tenants/{tenant_id}/purchase-orders",
params={"status": "pending_approval", "limit": 10} params={"status": "pending_approval", "limit": 10}
) )
if po_response.status_code == 200: if po_response.status_code == 200:
@@ -290,7 +290,7 @@ async def get_orchestration_summary(
async with httpx.AsyncClient(timeout=10.0) as client: async with httpx.AsyncClient(timeout=10.0) as client:
try: try:
batch_response = await client.get( batch_response = await client.get(
f"http://production:8000/api/v1/tenants/{tenant_id}/production-batches/today" f"http://production-service:8000/api/v1/tenants/{tenant_id}/production-batches/today"
) )
if batch_response.status_code == 200: if batch_response.status_code == 200:
batches = batch_response.json().get("batches", []) batches = batch_response.json().get("batches", [])
@@ -332,7 +332,7 @@ async def get_action_queue(
pending_pos = [] pending_pos = []
try: try:
po_response = await client.get( po_response = await client.get(
f"http://procurement:8000/api/v1/tenants/{tenant_id}/purchase-orders", f"http://procurement-service:8000/api/v1/tenants/{tenant_id}/purchase-orders",
params={"status": "pending_approval", "limit": 20} params={"status": "pending_approval", "limit": 20}
) )
if po_response.status_code == 200: if po_response.status_code == 200:
@@ -344,7 +344,7 @@ async def get_action_queue(
critical_alerts = [] critical_alerts = []
try: try:
alerts_response = await client.get( alerts_response = await client.get(
f"http://alert-processor:8000/api/v1/tenants/{tenant_id}/alerts", f"http://alert-processor-service:8000/api/v1/tenants/{tenant_id}/alerts",
params={"severity": "critical", "resolved": False, "limit": 20} params={"severity": "critical", "resolved": False, "limit": 20}
) )
if alerts_response.status_code == 200: if alerts_response.status_code == 200:
@@ -409,7 +409,7 @@ async def get_production_timeline(
async with httpx.AsyncClient(timeout=10.0) as client: async with httpx.AsyncClient(timeout=10.0) as client:
try: try:
response = await client.get( response = await client.get(
f"http://production:8000/api/v1/tenants/{tenant_id}/production-batches/today" f"http://production-service:8000/api/v1/tenants/{tenant_id}/production-batches/today"
) )
if response.status_code == 200: if response.status_code == 200:
batches = response.json().get("batches", []) batches = response.json().get("batches", [])
@@ -459,7 +459,7 @@ async def get_insights(
sustainability_data = {} sustainability_data = {}
try: try:
response = await client.get( response = await client.get(
f"http://inventory:8000/api/v1/tenants/{tenant_id}/sustainability/widget" f"http://inventory-service:8000/api/v1/tenants/{tenant_id}/sustainability/widget"
) )
if response.status_code == 200: if response.status_code == 200:
sustainability_data = response.json() sustainability_data = response.json()
@@ -470,7 +470,7 @@ async def get_insights(
inventory_data = {} inventory_data = {}
try: try:
response = await client.get( response = await client.get(
f"http://inventory:8000/api/v1/tenants/{tenant_id}/inventory/dashboard/stock-status" f"http://inventory-service:8000/api/v1/tenants/{tenant_id}/inventory/dashboard/stock-status"
) )
if response.status_code == 200: if response.status_code == 200:
inventory_data = response.json() inventory_data = response.json()

8
services/orchestrator/app/services/dashboard_service.py
View File

@@ -213,8 +213,8 @@ class DashboardService:
select(OrchestrationRun) select(OrchestrationRun)
.where(OrchestrationRun.tenant_id == tenant_id) .where(OrchestrationRun.tenant_id == tenant_id)
.where(OrchestrationRun.status.in_([ .where(OrchestrationRun.status.in_([
OrchestrationStatus.COMPLETED, OrchestrationStatus.completed,
OrchestrationStatus.COMPLETED_WITH_WARNINGS OrchestrationStatus.partial_success
])) ]))
.order_by(desc(OrchestrationRun.started_at)) .order_by(desc(OrchestrationRun.started_at))
.limit(1) .limit(1)
@@ -259,8 +259,8 @@ class DashboardService:
select(OrchestrationRun) select(OrchestrationRun)
.where(OrchestrationRun.tenant_id == tenant_id) .where(OrchestrationRun.tenant_id == tenant_id)
.where(OrchestrationRun.status.in_([ .where(OrchestrationRun.status.in_([
OrchestrationStatus.COMPLETED, OrchestrationStatus.completed,
OrchestrationStatus.COMPLETED_WITH_WARNINGS OrchestrationStatus.partial_success
])) ]))
.order_by(desc(OrchestrationRun.started_at)) .order_by(desc(OrchestrationRun.started_at))
.limit(1) .limit(1)