Skip to content

chore: reduce build time by converting docker image to build with nix #9099

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

Merged
merged 37 commits into from
Aug 18, 2023

Conversation

kylecarbs
Copy link
Member

@kylecarbs kylecarbs commented Aug 15, 2023

The build time goes from ~40m to ~1m and aligns us with our Nix users.

@kylecarbs kylecarbs self-assigned this Aug 15, 2023
@kylecarbs kylecarbs force-pushed the nixdev branch 2 times, most recently from 8dccc20 to 74d7f3a Compare August 15, 2023 17:59
@matifali
Copy link
Member

matifali commented Aug 17, 2023

Can this also have docker in docker?

@kylecarbs kylecarbs requested a review from code-asher August 17, 2023 22:30
@kylecarbs
Copy link
Member Author

@matifali it does indeed!

@kylecarbs kylecarbs marked this pull request as ready for review August 17, 2023 22:32
@kylecarbs kylecarbs changed the title chore: convert the docker image to build with nix chore: reduce build time by converting docker image to build with nix Aug 17, 2023
Copy link
Member

@code-asher code-asher left a comment

Choose a reason for hiding this comment

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

Very cool!

@@ -5,19 +5,26 @@ on:
branches:
- main
paths:
- "flake.nix"
- "flake.lock"
- "dogfood/**"
- ".github/workflows/dogfood.yaml"
# Uncomment these lines when testing with CI.
Copy link
Member

Choose a reason for hiding this comment

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

comment might be out of date?

Copy link
Member Author

Choose a reason for hiding this comment

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

Ahh, I almost missed this ty!

@kylecarbs kylecarbs merged commit c0a7853 into main Aug 18, 2023
@kylecarbs kylecarbs deleted the nixdev branch August 18, 2023 02:19
@github-actions github-actions bot locked and limited conversation to collaborators Aug 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants