Skip to content

feat(arch): add local execution backend - #5

Merged
toladnertum merged 4 commits into
mainfrom
feature/local-execution-backend
Sep 3, 2026
Merged

feat(arch): add local execution backend#5
toladnertum merged 4 commits into
mainfrom
feature/local-execution-backend

Conversation

@pelin17

@pelin17 pelin17 commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Summary

Implements the ARCH-COMP specific wiring for the new local execution backend, allowing rapid local testing and development without Docker or EC2 provisioning.

Changes

  • core submodule update: Brings in the base LocalBackend engine and registry configuration.
  • arch_comp/scripts/arch/run_benchmark.sh: Added local execution mode. Bypasses SSH and tmux overhead; runs the benchmark directly as a background process and routes logs to /app/logs when the target IP is 127.0.0.1 or localhost.
  • arch_comp/steps.py:
    • ArchInstallHandler and ArchLoadHandler: Short-circuited to return success immediately for local IPs, completely skipping unnecessary network cloning and installation steps.
    • Path Routing: Dynamically switches base directories (/app vs /home/ubuntu) depending on the execution environment.
    • Process Management: Updated _kill_run to use local pkill instead of SSH kill commands for local environments.

Copilot AI lite review requested due to automatic review settings August 27, 2026 13:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@toladnertum

Copy link
Copy Markdown
Contributor

Thanks! I merged a few other branches, which caused some merge conflicts. Happy to merge once these are resolved.

@pelin17

pelin17 commented Sep 2, 2026

Copy link
Copy Markdown
Contributor Author

Thanks! I merged a few other branches, which caused some merge conflicts. Happy to merge once these are resolved.

@toladnertum The core submodule merge conflicts are resolved. The PR is ready to merge

@toladnertum
toladnertum merged commit 302e41a into main Sep 3, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants