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

Zip the whl file before uploading for BCK step. #12201

Merged

Conversation

rtibbles
Copy link
Member

Summary

Zips the whl file into a zip file called kolibri.zip before uploading to the BCK GCS bucket.

@github-actions github-actions bot added DEV: dev-ops Continuous integration & deployment SIZE: very small labels May 23, 2024
Copy link
Member

@marcellamaki marcellamaki left a comment

Choose a reason for hiding this comment

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

Code changes match description of intended change. Will leave to @DXCanas to confirm that this is what we need on the infra side

@rtibbles rtibbles merged commit 095c5b4 into learningequality:release-v0.16.x May 23, 2024
34 checks passed
@rtibbles rtibbles deleted the really_bck_for_realz branch May 23, 2024 22:27
Copy link
Member

@DXCanas DXCanas left a comment

Choose a reason for hiding this comment

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

LGTM 👌🙌

@@ -204,8 +206,9 @@ jobs:
- name: Upload to BCK bucket
uses: 'google-github-actions/upload-cloud-storage@v2'
with:
path: 'dist/${{ needs.whl.outputs.whl-file-name }}'
path: 'dist/kolibri.zip'
Copy link
Member

Choose a reason for hiding this comment

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

That’s the ticket

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DEV: dev-ops Continuous integration & deployment SIZE: very small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants