Spaces:
Running
Running
chore: add more logging for updates/telemetry and order trees by updated_at desc
Browse files- app.py +6 -1
- supabase_database.py +3 -1
app.py
CHANGED
@@ -533,6 +533,7 @@ async def update_tree(tree_id: int, tree_update: TreeUpdate, request: Request):
|
|
533 |
)
|
534 |
|
535 |
# Update tree in database
|
|
|
536 |
updated_tree = await db.update_tree(tree_id, update_data)
|
537 |
|
538 |
# Process files and return with URLs
|
@@ -830,9 +831,13 @@ async def telemetry(event: TelemetryEvent, request: Request, user: Dict[str, Any
|
|
830 |
# Prefer Supabase persistent storage; fallback to file if not configured
|
831 |
if getattr(db, 'connected', False):
|
832 |
ok = db.log_telemetry(evt)
|
833 |
-
if
|
|
|
|
|
|
|
834 |
_write_telemetry(evt)
|
835 |
else:
|
|
|
836 |
_write_telemetry(evt)
|
837 |
return {"ok": True}
|
838 |
except Exception as e:
|
|
|
533 |
)
|
534 |
|
535 |
# Update tree in database
|
536 |
+
logger.info(f"Updating tree {tree_id} with fields: {list(update_data.keys())}")
|
537 |
updated_tree = await db.update_tree(tree_id, update_data)
|
538 |
|
539 |
# Process files and return with URLs
|
|
|
831 |
# Prefer Supabase persistent storage; fallback to file if not configured
|
832 |
if getattr(db, 'connected', False):
|
833 |
ok = db.log_telemetry(evt)
|
834 |
+
if ok:
|
835 |
+
logger.info(f"Telemetry stored: {evt.get('event_type')} status={evt.get('status')}")
|
836 |
+
else:
|
837 |
+
logger.warning("Telemetry DB insert failed, writing to file")
|
838 |
_write_telemetry(evt)
|
839 |
else:
|
840 |
+
logger.info("DB not connected, writing telemetry to file")
|
841 |
_write_telemetry(evt)
|
842 |
return {"ok": True}
|
843 |
except Exception as e:
|
supabase_database.py
CHANGED
@@ -83,7 +83,8 @@ class SupabaseDatabase:
|
|
83 |
pass
|
84 |
|
85 |
# Apply pagination and ordering
|
86 |
-
result = query.order('
|
|
|
87 |
.range(offset, offset + limit - 1) \
|
88 |
.execute()
|
89 |
|
@@ -289,6 +290,7 @@ class SupabaseDatabase:
|
|
289 |
'timestamp': event.get('timestamp')
|
290 |
}
|
291 |
self.client.table('telemetry_events').insert(payload).execute()
|
|
|
292 |
return True
|
293 |
except Exception as e:
|
294 |
logger.error(f"Failed to log telemetry: {e}")
|
|
|
83 |
pass
|
84 |
|
85 |
# Apply pagination and ordering
|
86 |
+
result = query.order('updated_at', desc=True) \
|
87 |
+
.order('created_at', desc=True) \
|
88 |
.range(offset, offset + limit - 1) \
|
89 |
.execute()
|
90 |
|
|
|
290 |
'timestamp': event.get('timestamp')
|
291 |
}
|
292 |
self.client.table('telemetry_events').insert(payload).execute()
|
293 |
+
logger.info("Telemetry event inserted into Supabase")
|
294 |
return True
|
295 |
except Exception as e:
|
296 |
logger.error(f"Failed to log telemetry: {e}")
|