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

chore(IDX): create minimal image #682

Merged
merged 22 commits into from
Jul 31, 2024
Merged

chore(IDX): create minimal image #682

merged 22 commits into from
Jul 31, 2024

Conversation

cgundy
Copy link
Member

@cgundy cgundy commented Jul 30, 2024

This PR introduces a minimal runner images that can be used across all dfinity orgs. The reason for this is that self-hosted runners must use a container image, but using build-ic for all jobs not in the ic repo is impractical. Because the image is public it can be used in other orgs as well.

@github-actions github-actions bot added the chore label Jul 30, 2024
@cgundy cgundy marked this pull request as ready for review July 31, 2024 08:14
@cgundy cgundy requested a review from a team as a code owner July 31, 2024 08:14
@github-actions github-actions bot added the @idx label Jul 31, 2024
Copy link
Collaborator

@basvandijk basvandijk left a comment

Choose a reason for hiding this comment

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

What does the minimal-runner-image have to do with the ic repo? Is the plan to usr this image later on?

A small README file in this directory would help answer these questions for future readers.

@cgundy
Copy link
Member Author

cgundy commented Jul 31, 2024

What does the minimal-runner-image have to do with the ic repo? Is the plan to usr this image later on?

Yeah I'm open to suggestions, but essentially it needs to be in a public repo. I wanted to put it in the k8s repo but because that's private it can't create a public image. It's a limitation of GHCR. At the very least, I can add a README to explain that. 👍

@cgundy cgundy enabled auto-merge July 31, 2024 08:55
@cgundy cgundy added this pull request to the merge queue Jul 31, 2024
Merged via the queue into master with commit 6f11a00 Jul 31, 2024
24 checks passed
@cgundy cgundy deleted the create-minimal-runner-image branch July 31, 2024 09:09
levifeldman pushed a commit to levifeldman/ic that referenced this pull request Oct 1, 2024
This PR introduces a minimal runner images that can be used across all
dfinity orgs. The reason for this is that self-hosted runners must use a
container image, but using `build-ic` for all jobs not in the ic repo is
impractical. Because the image is public it can be used in other orgs as
well.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants