seems to be working

PostHog · Sep 13, 2023 · c996713 · c996713
1 parent 2cca8cd
commit c996713
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/housewatch/celery.py b/housewatch/celery.py
@@ -35,6 +35,8 @@ def setup_periodic_tasks(sender: Celery, **kwargs):
 def run_backup(backup_id: str, incremental: bool = False):
     from housewatch.clickhouse import backups
 
+    logger.info("Running backup", backup_id=backup_id, incremental=incremental)
+
     backups.run_backup(backup_id, incremental=incremental)
 
 

diff --git a/housewatch/clickhouse/backups.py b/housewatch/clickhouse/backups.py
@@ -175,7 +175,7 @@ def run_backup(backup_id, incremental=False):
     S3_LOCATION = f"https://{backup.bucket}.s3.amazonaws.com/{path}"
     if incremental:
         if not backup.last_run:
-            raise ValueError("Cannot run incremental backup without a base backup")
+            logger.info("Cannot run incremental backup without a base backup")
         base_backup = backup.last_base_backup
     if backup.is_database_backup():
         create_database_backup(