fix(history): cap default backfill window at 30 days (mila sacct limit)

Author: wietzesuijker

cluv/__main__.py

@@ -206,8 +206,8 @@ def add_history_args(subparsers: Subparsers) -> argparse.ArgumentParser:
     history_parser.add_argument(
         "--since-days",
         type=int,
-        default=60,
-        help="How many days of sacct to pull for `backfill`. Default: 60.",
+        default=30,
+        help="How many days of sacct to pull for `backfill`. Default: 30.",
     )
     history_parser.set_defaults(func=history)
     return history_parser

cluv/history.py

@@ -191,7 +191,7 @@ def _records_from_sacct(output: str, cluster: str) -> list[JobRecord]:
 
 
 async def backfill_from_sacct(
-    remote: Remote, cluster: str, *, since_days: int = 60
+    remote: Remote, cluster: str, *, since_days: int = 30
 ) -> int:
     """Pull recent cluv-stamped jobs from sacct on ``cluster``, write to cache.
 
@@ -271,5 +271,5 @@ def clear(cluster: str | None = None, key: str | None = None) -> int:
 
 
 # Convenience for sync callers (CLI subcommands) that don't want to await.
-def backfill_sync(remote: Remote, cluster: str, *, since_days: int = 60) -> int:
+def backfill_sync(remote: Remote, cluster: str, *, since_days: int = 30) -> int:
     return asyncio.run(backfill_from_sacct(remote, cluster, since_days=since_days))