feat(cli): add --remote flag for automated remote Ray dispatch of ado CLI commands#593
Open
michael-johnston wants to merge 30 commits intomainfrom
Open
feat(cli): add --remote flag for automated remote Ray dispatch of ado CLI commands#593michael-johnston wants to merge 30 commits intomainfrom
michael-johnston wants to merge 30 commits intomainfrom
Conversation
Member
Author
|
@AlessandroPomponio Docs not complete but take a look and test when you have a moment. |
1e0caaf to
01a6002
Compare
* refactor(remote): initial attempt at simplifying code Made with Bob Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): remove wrapper functions Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): rename models Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): remove unused function Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): move definitions Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): rename FlagDefinition to RemoteDispatchFlagDefinition Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * fix(tests): update mocks Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): rename remote-dispatch related models Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): rename files as well Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): rename field Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): remove duplicate argument Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): simplify arguments Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): strip -c flag immediately Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): use project context name for serialization Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * fix(tests): update assertions Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(core): use HttpUrl with constraint in ExecutionContext Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(core): rename ExecutionContext to RemoteExecutionContext Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * docs(agents): add instructions Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): use annotated defaults Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * docs(cli): update docstring for ParsedRemoteSubmissionFlags Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): rename variables Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): rename flag_name to name Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): rename variable Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): rename variables Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): rename variables Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * refactor(cli): remove filter_and_rewrite Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> * build(deps): add uv as a dependency It's now required for remote execution Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com> --------- Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
Member
AlessandroPomponio
left a comment
There was a problem hiding this comment.
I have left a couple of comments and opened #608 for converting docstrings to all use the Google format
Also avoid direct indexing of function_operation_descriptions just in case
Original code waited 30 seconds for port-forward to become ready. If port-forward failed after 1s it would still wait 30s New code polls directly for errors allowing faster failure.
Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
…_execution_contexts
The additionalFiles field can contain absolute or relative paths to files or directories. These will be symlinked into the workding dir package sent to the remote ray cluster.
plus related link updated
Member
Author
|
@christian-pinto @VassilisVassiliadis some modifications to the actuator docs so please check |
Member
The docs look fine to me |
plus related link updated
VassilisVassiliadis
previously requested changes
Feb 25, 2026
… after join Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
…xts' into maj_execution_contexts
Review comments answered
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.
What's Added
A new global
--remote <file>option for the ado CLI. When provided, any ado command is automatically dispatched to a remote Ray cluster instead of running locally. The entire workflow — copying files, building plugin wheels, generating a Ray runtime environment, setting up a port-forward, submitting the job, and tearing down — is handled by ado with no manual steps required.Example
Running an optimisation operation (using examples/optimization_test_functions/) with the port-forward execution context.
Where
execution_context.yamlcontains: