Antctl+zen#3
Merged
Merged
Conversation
Create new antctl+zen process manager that uses antctl defaults for paths. Key features: - Only specifies ports (node/metrics/rpc) and rewards address - Lets antctl choose its own defaults for data, log, and binary paths - Parses antctl add command output to extract actual paths - Updates database with configuration from antctl - Simplifies integration with antctl's native behavior This approach provides better compatibility with antctl while maintaining control over port assignments for router forwarding requirements. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit completes the integration of the AntctlZenManager into the process manager factory and configuration system. Changes: - Factory integration: Added AntctlZenManager import and special routing for "antctl+zen" manager type in get_process_manager() - Configuration support: Added "antctl+zen" to --process_manager choices with automatic user mode routing (zen only supports user mode) - Mode detection: Updated _detect_process_manager_mode() to recognize "+zen" suffix as user mode for proper platform-specific path selection - CHANGELOG: Added v0.4.0 release notes documenting the new zen manager The zen manager provides a minimal command specification approach that lets antctl choose its own defaults for binary, data, and log paths while maintaining explicit control over ports for router forwarding. Integration verified with factory instantiation tests.
Fixed SQLAlchemy session error when creating nodes with antctl+zen process manager. The error "Instance is not bound to a Session" occurred because the node object was created in one session (executor) then passed to the manager which tried to add it to a new session. Changed antctl_zen_manager.py to use session.merge() instead of session.add() to properly handle detached node instances. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
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
1 participant
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.
Create a new process manager to debug antctl failing to start a random node.