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

feat: docker builder #5

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

feat: docker builder #5

wants to merge 17 commits into from

Conversation

jakeyheath
Copy link
Collaborator

@jakeyheath jakeyheath commented May 1, 2024

Summary

This PR tags the docker builder action that uses a branch of the real action. The branch has a few changes that make it usable by org outside of CZI. For now, we can continue to use this branch until the hackathon is over.

Things we needed to do to get it working

  • Add org and repo to gh action role trust relationship
  • Set up docker builder action to not use self-hosted runners
  • Set up docker builder action to use a local path for ECR settings (before it was using a private repo to grab settings from)
  • Set up the team and permissions on the API
  • Register the app to Argus
  • Set up the secrets and ID token permissions to work with personal Github projects (orgs have different rules apparently about thread through secrets)

@jakeyheath jakeyheath added stack and removed stack labels May 1, 2024
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