Skip to content

Feature: Add support for Globus compute v3 Action Providers#305

Merged
NickolausDS merged 4 commits intoglobus-gladier:mainfrom
NickolausDS:compute-v3-support
Aug 13, 2025
Merged

Feature: Add support for Globus compute v3 Action Providers#305
NickolausDS merged 4 commits intoglobus-gladier:mainfrom
NickolausDS:compute-v3-support

Conversation

@NickolausDS
Copy link
Collaborator

This removes code debt necessary for allowing an upgrade to v3. Tests currently pass on v2, v3 functionality still needs to be added in addition to upgrading the 'chaining' behavior for the new refactor.

This removes code debt necessary for allowing an upgrade to v3
@NickolausDS NickolausDS force-pushed the compute-v3-support branch from 8a98e5c to a1d7e05 Compare June 4, 2025 17:59
v3 tools can be configured by setting the following attribute:

    action_url = "https://compute.actions.globus.org/v3"

The following modifiers are supported for v3 actions:

        "endpoint_id",
        "tasks",
        "task_group_id",
        "user_endpoint_config",
        "resource_specification",
        "create_queue",
@NickolausDS NickolausDS marked this pull request as ready for review June 16, 2025 18:21
@NickolausDS
Copy link
Collaborator Author

@ryanchard Ahh, sorry. I actually had a pretty good writeup on the v3 behavior already, but it's been long enough I didn't remember. There's a README and also code examples in the Gladier Client Examples PR here:

globus-gladier/gladier-client-examples#19

Copy link
Contributor

@ryanchard ryanchard left a comment

Choose a reason for hiding this comment

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

I see I approved the examples PR and not this one. Together this made sense and will let us start upgrading flows to use the v3 api.

@NickolausDS NickolausDS merged commit 87ddf5c into globus-gladier:main Aug 13, 2025
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.

2 participants