Conversation
- Add build-arm64, build-cpu-arm64, build-lb-cpu-arm64 targets - Add push-arm64 target for pushing ARM64 images to Docker Hub - Fix Makefile awk regex to properly match target names with numbers - Update version to 0.7.3 in uv.lock
…viceRegistry Replace hub-and-spoke architecture with peer-to-peer model where all endpoints query State Manager directly for dynamic endpoint discovery. Integrates tetra-rp's ServiceRegistry for consistent cross-endpoint function routing. Key Changes: - Update tetra-rp ServiceRegistry to use StateManagerClient directly (peer-to-peer) - Replace manifest-based routing with async ServiceRegistry in RemoteExecutor - Support provisioning-time endpoint IDs through State Manager - Add optional tetra-rp import with graceful fallback for local execution - Remove old static routing methods (_get_target_endpoint, _is_local_endpoint) - Update tests to verify new ServiceRegistry-based routing Architecture: - All endpoints query State Manager directly (no single point of failure) - 300s cache reduces State Manager queries to ~1 per endpoint per 5 minutes - ServiceRegistry determines local vs remote based on FLASH_RESOURCE_NAME - Graceful degradation if State Manager unavailable (fallback to local) Benefits: ✓ Handles provisioning-time endpoint IDs correctly ✓ No hub-and-spoke dependency (more resilient) ✓ Consistent with tetra-rp client-side routing ✓ Automatic manifest updates via State Manager ✓ Simpler error handling with clear fallback paths Test Results: - All 205 tests pass - Code coverage: 78.78% (required: 35%) - All quality checks pass (ruff format, ruff check, mypy, pytest) - Handler tests: 14/14 pass
- Removed .gitmodules and tetra-rp submodule registration - Added tetra-rp as a git-based dependency in pyproject.toml - Removed tetra-rp exclusions from mypy and ruff config - For local development, install with: uv pip install -e ~/Github/python/tetra-rp All 205 worker-tetra tests pass with 78% coverage. Benefits: - Cleaner project structure - Easier dependency management - Allows flexible local development with -e flag - Both projects can have independent release cycles
- Updated DEVELOPMENT.md with local development workflow using pip -e flag - Renamed 'Submodule Management' to 'tetra-rp Dependency Management' - Added instructions for switching between remote and local tetra-rp - Updated CI/CD workflows to remove all submodule references: - Removed 'submodules: recursive' from checkout actions - Removed 'git submodule update' commands - Removed 'cp tetra-rp/src/...' protocol sync commands - Simplified setup workflow - uv sync now handles all dependencies
Update all code, tests, and documentation to use tetra-rp as a pip dependency instead of a git submodule. This includes updating import paths, test fixtures, and documentation references across the codebase.
- Add manifest_reconciliation.py with request-scoped refresh - Integrate refresh into RemoteExecutor before routing - Add comprehensive unit and integration tests - Update remote_executor to call refresh_manifest_if_stale() Serverless-compatible approach using TTL-based staleness detection. No boot-time blocking, no background threads. Manifest refreshes from State Manager only when stale (>5 min) before cross-endpoint routing. Test coverage: 30+ unit tests, 15+ integration tests Quality checks: All 241 tests pass, 79% code coverage
Contributor
There was a problem hiding this comment.
Pull request overview
This PR implements TTL-based manifest reconciliation for state manager integration and converts tetra-rp from a git submodule to a pip dependency. The changes enable peer-to-peer cross-endpoint routing with automatic manifest refresh while simplifying dependency management and adding ARM64 Docker support.
Changes:
- Convert tetra-rp from git submodule to pip dependency for better package management
- Add TTL-based manifest reconciliation with State Manager integration
- Implement peer-to-peer cross-endpoint routing via ServiceRegistry
- Add ARM64 Docker image build support
- Update all imports from local
remote_execution.pytotetra_rp.protos.remote_execution
Reviewed changes
Copilot reviewed 36 out of 37 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tetra-rp | Removed git submodule reference |
| .gitmodules | Removed submodule configuration |
| pyproject.toml | Added tetra-rp as pip dependency and aiohttp for HTTP routing |
| src/remote_executor.py | Added manifest reconciliation, ServiceRegistry integration, and cross-endpoint routing |
| src/manifest_reconciliation.py | New module for TTL-based manifest refresh from State Manager |
| src/constants.py | Added manifest path and timeout constants |
| Makefile | Removed submodule commands, added ARM64 build targets |
| .github/workflows/*.yml | Removed submodule checkout steps |
| tests/unit/test_manifest_reconciliation.py | New comprehensive unit tests for manifest reconciliation |
| tests/integration/test_manifest_state_manager.py | New integration tests for State Manager interaction |
| All test files | Updated imports to use tetra_rp.protos.remote_execution |
| docs/Manifest_Reconciliation.md | New comprehensive documentation |
| DEVELOPMENT.md | Updated for pip dependency workflow |
| CLAUDE.md | Updated references from submodule to pip dependency |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 36 out of 37 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Remove unnecessary fallback in remote_executor.py line 383: function_name is guaranteed non-None by FunctionRequest validation, so the 'or ""' fallback is dead code that would fail anyway - Improve Makefile help regex pattern to use /^[a-zA-Z_][a-zA-Z0-9_-]*:.*# / instead of /^[a-zA-Z0-9_-]+:.*# / to ensure targets start with a letter or underscore, preventing unintended matches of targets starting with digits
KAJdev
approved these changes
Jan 26, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 38 out of 39 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
KAJdev
approved these changes
Jan 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
See Manifest Reconciliation Documentation for technical details on the TTL-based caching strategy.
Test plan
make quality-checkto verify all tests passmake test-handlerto validate handler functionality