Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ComputeClientV3 methods to manage endpoints #1113

Merged
merged 2 commits into from
Nov 22, 2024

Conversation

rjmello
Copy link
Member

@rjmello rjmello commented Nov 22, 2024

Added .register_endpoint(), .update_endpoint(), .lock_endpoint(), and .get_endpoint_allowlist() methods to the ComputeClientV3 class.


📚 Documentation preview 📚: https://globus-sdk-python--1113.org.readthedocs.build/en/1113/

Copy link
Contributor

@khk-globus khk-globus left a comment

Choose a reason for hiding this comment

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

Also thinking at this time that you may want to add yourself (and anyone else appropriate from our team) to CODEOWNERS. Maybe not for this PR — or maybe for the associated paths in this PR?

@rjmello rjmello force-pushed the compute-client-endpoint-methods-v3 branch from ee939b3 to 7d7a12d Compare November 22, 2024 13:56
Added `.register_endpoint()`, `.update_endpoint()`, `.lock_endpoint()`,
and `.get_endpoint_allowlist()` methods to the `ComputeClientV3` class.
@rjmello rjmello force-pushed the compute-client-endpoint-methods-v3 branch from 7d7a12d to 16b1981 Compare November 22, 2024 14:34
Copy link
Contributor

@khk-globus khk-globus left a comment

Choose a reason for hiding this comment

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

Approving, so as to not block the general consensus.

@kurtmckee kurtmckee merged commit 7b5771e into globus:main Nov 22, 2024
7 checks passed
@rjmello rjmello deleted the compute-client-endpoint-methods-v3 branch November 22, 2024 19:02
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