Add suggestions fix wrong crontab

safe-global · Jan 13, 2025 · e07f147 · e07f147
1 parent 759b95b
commit e07f147
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/app/datasources/db/models.py b/app/datasources/db/models.py
@@ -281,7 +281,9 @@ async def get_abi_by_contract_address(
         return None
 
     @classmethod
-    async def get_contracts_without_abi(cls, session: AsyncSession, max_retries: int):
+    async def get_contracts_without_abi(
+        cls, session: AsyncSession, max_retries: int = 0
+    ):
         """
         Fetches contracts without an ABI and fewer retries than max_retries, streaming results in batches to reduce memory usage for large datasets.
         More information about streaming results can be found here: https://docs.sqlalchemy.org/en/20/core/connections.html#streaming-with-a-dynamically-growing-buffer-using-stream-results

diff --git a/app/workers/tasks.py b/app/workers/tasks.py
@@ -68,7 +68,7 @@ async def get_contract_metadata_task(
         logging.debug(f"Skipping contract with address {address} and chain {chain_id}")
 
 
-@dramatiq.actor(periodic=cron("* * * * *"))  # Every midnight
+@dramatiq.actor(periodic=cron("0 0 * * *"))  # Every midnight
 @database_session
 async def get_missing_contract_metadata_task(session: AsyncSession) -> None:
     async for contract in Contract.get_contracts_without_abi(