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

feature(sdk): support graph deletion #1021

Merged
merged 2 commits into from
Nov 13, 2024
Merged

Conversation

seriousben
Copy link
Member

Context

There is currently no way to delete a compute graph using the SDK.

What

A new delete_compute_graph function is added to the IndexifyClient.

Testing

Contribution Checklist

  • If the python-sdk was changed, please run make fmt in python-sdk/.
  • If the server was changed, please run make fmt in server/.
  • Make sure all PR Checks are passing.

@seriousben seriousben requested a review from diptanu November 12, 2024 21:09
cert_path=cert_path,
key_path=key_path,
ca_bundle_path=ca_bundle_path
cert_path=cert_path, key_path=key_path, ca_bundle_path=ca_bundle_path
Copy link
Collaborator

Choose a reason for hiding this comment

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

Did you run make fmt Feels like this would be re-formatted back to what we had before.

@seriousben seriousben force-pushed the seriousben/sdk-graph-deletion branch 2 times, most recently from 486dd16 to 814cb76 Compare November 13, 2024 13:27
@seriousben seriousben force-pushed the seriousben/sdk-graph-deletion branch 2 times, most recently from 79317c1 to fdd5dcd Compare November 13, 2024 13:43
@seriousben seriousben added the ci_compat_test Trigger testing previous SDK versions against the current server label Nov 13, 2024
@seriousben seriousben force-pushed the seriousben/sdk-graph-deletion branch from fdd5dcd to 91e6fa0 Compare November 13, 2024 13:50
@seriousben seriousben removed the ci_compat_test Trigger testing previous SDK versions against the current server label Nov 13, 2024
@seriousben seriousben force-pushed the seriousben/sdk-graph-deletion branch from 91e6fa0 to 261d47c Compare November 13, 2024 13:55
@seriousben seriousben force-pushed the seriousben/sdk-graph-deletion branch from 261d47c to c6ced75 Compare November 13, 2024 13:59
@seriousben
Copy link
Member Author

seriousben commented Nov 13, 2024

@diptanu - I made it so that linting will be enforced by CI starting now. I also made the last release acceptance gated behind a specific label since it is expected to fail a lot until we get to v1.0 - I do think that last release test is useful to be intentional about breaking changes.

Will be merging this asap.

@seriousben seriousben merged commit 17115cf into main Nov 13, 2024
6 checks passed
@seriousben seriousben deleted the seriousben/sdk-graph-deletion branch November 13, 2024 14:10
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