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

Exclude s3 from main distribution #392

Merged
merged 1 commit into from
Sep 19, 2023
Merged

Conversation

ivanyu
Copy link
Contributor

@ivanyu ivanyu commented Sep 19, 2023

The S3 and GCS should be built separately with :storage:s3:DistTar and :storage:gcs:DistTar. This comes with certain modifications in Docker building, demos, e2e tests.

@ivanyu ivanyu force-pushed the ivanyu/separate-distro branch from 95b2a66 to 6ab975c Compare September 19, 2023 12:28
@ivanyu ivanyu changed the title Split distribution into core, S3, and GCS Exclude s3 from main distribution Sep 19, 2023
@ivanyu ivanyu force-pushed the ivanyu/separate-distro branch 3 times, most recently from fb3dc2a to 9995d94 Compare September 19, 2023 12:31
The S3 and GCS should be built separately with `:storage:s3:DistTar` and `:storage:gcs:DistTar`. This comes with certain modifications in Docker building, demos, e2e tests.
@ivanyu ivanyu force-pushed the ivanyu/separate-distro branch from 9995d94 to 2639ed0 Compare September 19, 2023 12:33
@ivanyu ivanyu marked this pull request as ready for review September 19, 2023 12:44
@ivanyu ivanyu requested a review from a team as a code owner September 19, 2023 12:44
@AnatolyPopov AnatolyPopov merged commit 71b1ebb into main Sep 19, 2023
6 checks passed
@AnatolyPopov AnatolyPopov deleted the ivanyu/separate-distro branch September 19, 2023 13:03
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