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 upload automation #1564

Closed
wants to merge 4 commits into from
Closed

S3 upload automation #1564

wants to merge 4 commits into from

Conversation

jmervine
Copy link
Contributor

@jmervine jmervine commented May 26, 2020

Context

After a discussion with @jdorfman, we'd like to leverage using Github as the source of truth for automated syncing of cdn files to S3. This is to ensure prompt release of new files, as well as remove any question about state.

Solution

Leverage existing ops automation, from https://github.com/jmervine/bootstrapcdn-ops. Wrap it in a Docker image for execution and use CI to automate via Github Actions.

Testing

  1. Docker image should run if configured properly.
  2. CI should automatically run and sync when this branch is deployed, either at merge time, or actual deployment time, depending on how CI is implemented.

@jmervine jmervine temporarily deployed to bootstrapcdn-dev-pr-1564 May 26, 2020 16:58 Inactive
@XhmikosR
Copy link
Collaborator

I can't actually comment on how this works, but why don't we use an existent action?

@jmervine
Copy link
Contributor Author

jmervine commented May 26, 2020

Not sure what "an existent action" is or refers to.

@jmervine jmervine temporarily deployed to bootstrapcdn-dev-pr-1564 May 26, 2020 20:34 Inactive
@XhmikosR
Copy link
Collaborator

https://github.com/marketplace?type=actions&query=s3 or another search should give more insight.

@XhmikosR XhmikosR marked this pull request as draft May 27, 2020 06:37
@XhmikosR XhmikosR temporarily deployed to bootstrapcdn-dev-pr-1564 May 27, 2020 06:40 Inactive
@XhmikosR XhmikosR temporarily deployed to bootstrapcdn-dev-pr-1564 May 27, 2020 06:42 Inactive
@XhmikosR XhmikosR force-pushed the jmervine-upload-automation branch from ede17b1 to 9ff90d1 Compare May 27, 2020 06:44
@XhmikosR XhmikosR temporarily deployed to bootstrapcdn-dev-pr-1564 May 27, 2020 06:44 Inactive
@jmervine jmervine temporarily deployed to bootstrap-cd-jmervine-u-xli38o July 7, 2020 21:42 Inactive
@jimaek
Copy link
Member

jimaek commented Mar 7, 2021

I believe this is not necessary anymore, too bad because its a cool solution

@jimaek jimaek closed this Mar 7, 2021
@XhmikosR XhmikosR deleted the jmervine-upload-automation branch March 11, 2021 08:42
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.

None yet

3 participants