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

sync GitHub action distribution to S3 #5

Merged
merged 22 commits into from
May 6, 2020

Conversation

npalm
Copy link
Member

@npalm npalm commented Apr 30, 2020

To avoid we have to download the github action dist everytime on runner is created we cach the dist in S3.

@npalm npalm requested a review from gertjanmaas April 30, 2020 18:59
import { handle } from './syncer/handler';

module.exports.handler = async (event: any, context: any, callback: any): Promise<any> => {
const statusCode = await handle();
Copy link
Collaborator

Choose a reason for hiding this comment

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

StatusCode isn't necessary, this lambda isn't part of the API

modules/action-runner-binary-cache/variables.tf Outdated Show resolved Hide resolved
@npalm
Copy link
Member Author

npalm commented May 4, 2020

@gertjanmaas solved review comments, also add tagging for resources and update default example. Did not change the used API. The one you suggested is not a public api.

@gertjanmaas gertjanmaas merged commit e219b4a into master May 6, 2020
@gertjanmaas gertjanmaas deleted the feature/sync-github-action-dist branch May 6, 2020 11:26
npalm added a commit that referenced this pull request Aug 5, 2021
# This is the 1st commit message:

Rebase

# This is the commit message #2:

fix: terraform error

# This is the commit message #3:

fix: terraform error

# This is the commit message #4:

fix: test and linter

# This is the commit message #5:

runner labels can be a subset

# This is the commit message #6:

runner labels can be a subset
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.

2 participants