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

S3 delivery #3

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

S3 delivery #3

wants to merge 13 commits into from

Conversation

lumaghg
Copy link

@lumaghg lumaghg commented Jun 27, 2023

Hi,
this PR includes a pipeline to push the materials.json and molecules.json to an s3 bucket, where they can be made publicly accessible at a static endpoint. I have also moved the 2 json files into a folder, because the s3-upload-action only supports pushing folders for some reason.

I have tested this in my fork with my personal aws credentials. To get this working in your repository, please take the following steps

  • create a free AWS account
  • add your security credentials to the actions secrets of this repository (AWS_KEY_ID= AWS_SECRET_ACCESS_KEY=
  • navigate to S3
  • create a new bucket with the name "chemmatdata"
  • disable all rules regarding permitting public access
  • enable ACLs
  • push something to the repository to trigger the pipeline
  • the two files should appear in the s3 bucket in a folder "datasets"
  • mark the files and make them publicly available by clicking "Actions" => "make public via acl"
  • after that you should see the public URL when clicking on respective files

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