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

Upload artifacts and logs to S3 bucket #114

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

marbre
Copy link
Contributor

@marbre marbre commented Feb 25, 2025

  • Uses the configure-aws-credentials action to retrieve short-lived
    credentials using GitHub's OIDC provider
  • Creates index of files using the MIT-licensed indexer.py script
  • Uploads artifacts and logs to the S3 bucket
  • Adds links pointing to the bucket to the job summary

Creating index files and uploading files is to be moved to scripts in a
follow up.

@marbre marbre force-pushed the upload-artifacts branch 6 times, most recently from 694b322 to 87e3a92 Compare February 25, 2025 22:20
* Uses the `configure-aws-credentials` action to retrieve short-lived
  credentials using GitHub's OIDC provider
* Creates index of files using the MIT-licensed indexer.py script
* Uploads artifacts and logs to the S3 bucket
* Adds links pointing to the bucket to the job summary

Creating index files and uploading files is to be moved to scripts in a
follow up.
@marbre marbre changed the title Upload artifacts to S3 bucket Upload artifacts and logs to S3 bucket Feb 25, 2025
@marbre
Copy link
Contributor Author

marbre commented Feb 25, 2025

See for example https://github.com/nod-ai/TheRock/actions/runs/13532379300 for a run limited to build therock-archive-core-runtime.

@marbre marbre marked this pull request as ready for review February 25, 2025 23:24
@marbre marbre marked this pull request as draft February 26, 2025 00:33
@marbre
Copy link
Contributor Author

marbre commented Feb 26, 2025

Artifacts were successfully upload (https://therock-artifacts.s3.us-east-2.amazonaws.com/13532940414/index.html) but seems logs were not. Looking into it.

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.

1 participant