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

ci: upload release artifact to supabase s3 bucket #1812

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hf
Copy link
Contributor

@hf hf commented Oct 22, 2024

Uploads the release artifact to a Supabase owned S3 bucket for easy releases.

@hf hf requested a review from a team as a code owner October 22, 2024 20:11
@coveralls
Copy link

coveralls commented Oct 22, 2024

Pull Request Test Coverage Report for Build 11467779604

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 57.172%

Totals Coverage Status
Change from base Build 11466551533: 0.0%
Covered Lines: 9574
Relevant Lines: 16746

💛 - Coveralls

Copy link
Contributor

@J0 J0 left a comment

Choose a reason for hiding this comment

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

Guessing the secrets have already ben added to Github?

Comment on lines +93 to +98
- name: configure aws credentials for uploading release artifacts
uses: aws-actions/configure-aws-credentials@v1
with:
role-to-assume: ${{ secrets.PROD_AWS_ROLE }}
aws-region: us-east-1

Copy link
Member

@kangmingtay kangmingtay Oct 30, 2024

Choose a reason for hiding this comment

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

we only upload the release artifacts when the release-please PR is merged and not the rc versions right? seems like this would push every rc to s3 since the action is also triggered on the release/* branch

@@ -126,6 +132,8 @@ jobs:
-f "sha=$GITHUB_SHA"
fi
fi

aws s3api put-object --bucket ${{ secrets.PROD_ARTIFACTS_BUCKET }} --key auth-v$RELEASE_VERSION-arm64.tar.gz --body ./auth-v$RELEASE_VERSION-arm64.tar.gz
Copy link
Member

@kangmingtay kangmingtay Nov 5, 2024

Choose a reason for hiding this comment

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

added PROD_ARTIFACTS_BUCKET this to the repo secrets

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