Skip to content

Commit 0b84959

Browse files
committed
feat: remove disk space check
1 parent 62536ff commit 0b84959

File tree

3 files changed

+6
-40
lines changed

3 files changed

+6
-40
lines changed

src/cli.py

Lines changed: 1 addition & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -545,7 +545,6 @@ async def _perform_preflight_checks(self, args: argparse.Namespace) -> bool:
545545
True if all checks pass, False otherwise
546546
"""
547547
from pathlib import Path
548-
import shutil
549548

550549
def _try(message: str, bullets: list[str]) -> None:
551550
self.console.print(f"[red]{message}[/red]")
@@ -631,23 +630,7 @@ def _try(message: str, bullets: list[str]) -> None:
631630
self.console.print(f"[red]Error checking working tree: {e}[/red]")
632631
return False
633632

634-
# 4. Check disk space (20GB minimum as per spec)
635-
try:
636-
stat = shutil.disk_usage(str(repo_path))
637-
free_gb = stat.free / (1024**3)
638-
if free_gb < 20:
639-
_try(
640-
f"insufficient disk space: {free_gb:.1f}GB free (<20GB)",
641-
[
642-
"free space on this volume",
643-
"change repo to a disk with more space",
644-
],
645-
)
646-
return False
647-
except OSError as e:
648-
self.console.print(
649-
f"[yellow]Warning: Could not check disk space: {e}[/yellow]"
650-
)
633+
# 4. Disk space check removed: do not block runs based on free space.
651634

652635
# 5. Quick test of Docker image availability (non-blocking)
653636
# This is a quick check - actual image pull happens later if needed

src/instance_runner/runner.py

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -401,18 +401,7 @@ def emit_event(event_type: str, data: Dict[str, Any]) -> None:
401401
if not branch_name:
402402
raise ValidationError("Branch name must be provided by orchestration")
403403

404-
# Validate disk space (20GB minimum as per spec)
405-
try:
406-
import shutil
407-
408-
stat = shutil.disk_usage(str(repo_path))
409-
free_gb = stat.free / (1024**3)
410-
if free_gb < 20:
411-
raise ValidationError(
412-
f"Insufficient disk space: {free_gb:.1f}GB free (20GB required)"
413-
)
414-
except OSError as e:
415-
logger.warning(f"Could not check disk space: {e}")
404+
# Disk space check removed: proceed without enforcing a minimum.
416405

417406
# Initialize components
418407
docker_manager = DockerManager()

src/orchestration/orchestrator.py

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@
3232
DockerError,
3333
GitError,
3434
StrategyError,
35-
ValidationError,
3635
)
3736

3837

@@ -192,7 +191,7 @@ async def initialize(self) -> None:
192191
self.state_dir.mkdir(parents=True, exist_ok=True)
193192
self.logs_dir.mkdir(parents=True, exist_ok=True)
194193

195-
# Check disk space (20GB minimum as per spec)
194+
# Disk space check removed: keep informational logging only
196195
await self._check_disk_space()
197196

198197
# Initialize components
@@ -2614,8 +2613,8 @@ async def _run_instance_from_saved_state(self, instance_info) -> InstanceResult:
26142613

26152614
async def _check_disk_space(self) -> None:
26162615
"""
2617-
Check available disk space per spec section 8.1.
2618-
Requires at least 20GB free space for Docker images and temporary workspaces.
2616+
Log available disk space (informational only).
2617+
No enforcement; runs proceed regardless of free space.
26192618
"""
26202619
try:
26212620
import shutil
@@ -2632,12 +2631,7 @@ async def _check_disk_space(self) -> None:
26322631
f"Disk space: {free_gb:.1f}GB free of {total_gb:.1f}GB ({used_percent:.1f}% used)"
26332632
)
26342633

2635-
# Check minimum requirement (20GB as per spec)
2636-
if free_gb < 20:
2637-
raise ValidationError(
2638-
f"Insufficient disk space: {free_gb:.1f}GB free, need at least 20GB "
2639-
f"for Docker images and temporary workspaces"
2640-
)
2634+
# No minimum threshold enforced.
26412635

26422636
except (OSError, ImportError) as e:
26432637
logger.warning(f"Could not check disk space: {e}")

0 commit comments

Comments
 (0)