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

Fix gateway server bug and interrupt handling for chunk dispatching #868

Merged
merged 221 commits into from
Jun 15, 2023

Conversation

sarahwooders
Copy link
Contributor

@sarahwooders sarahwooders commented Jun 14, 2023

This fixes two bugs:

  • During chunk dispatching for CLI transfers, Ctrl-C could not be used to exit out of the transfer due to threads not being properly filled. This is fixed by adding exit signals to the tracker and stopped the multipart threads when the tracker is exited.
  • Previous self.auth in the Server instance was modified to contain credentials for all clouds that are initialized, however this was not properly updated for child classes.

lynnliu030 and others added 30 commits November 29, 2022 13:36
This change introduces an API for Skyplane Broadcast

Todos:
- [x] Fix provisioning in BroadcastDataplane
  - Reuse provision loop via inheritance
  - Move `_start_gateway` to a class method and override it
  - Adapt broadcast to use `bound_nodes`
- [x] Add BroadcastCopyJob (ideally extend CopyJob)
- [x] Update tracker to monitor broadcast jobs
- [x] Add multipart support 
- [x] Fix dependency issue via adding dockerfile and bc_requirements 
- [x] Integrate with gateway and test the monitoring side 

Co-authored-by: Paras Jain <[email protected]>
Co-authored-by: Sarah Wooders <[email protected]>
@sarahwooders sarahwooders changed the title Fix gateway server bug Fix gateway server bug and interrupt handling for chunk dispatching Jun 14, 2023
@sarahwooders sarahwooders merged commit 2696494 into skyplane-project:main Jun 15, 2023
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.

4 participants