Skip to content

Remove unnecessary nix build dependencies from docker image #404

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

Open
tegefaulkes opened this issue May 6, 2025 · 2 comments
Open

Remove unnecessary nix build dependencies from docker image #404

tegefaulkes opened this issue May 6, 2025 · 2 comments
Labels
development Standard development

Comments

@tegefaulkes
Copy link
Contributor

Specification

Following #399 We've reduced the size of the docker image to less that the previous size. But digging deeper into the image we found that there are extra nix store paths that are being included which may not need to be there. Mainly there is the inclusion of the polykey-cli so there is at munimum, 42MB that doesn't need to be included.

The nix store seems to be determined by the nix dependencies of the derivation being copied to the docker image. When testing I saw that these are included after the contents are copied to root. So its not possible to remove the nix dependencies using an extra build script.

Image

Additional context

Tasks

  1. Identify why polykey-cli is being included in the nix store paths of the docker image.
  2. Ensure that only runtime dependencies are included in the docker image.
  3. Apply the fix.
@tegefaulkes tegefaulkes added the development Standard development label May 6, 2025
Copy link

linear bot commented May 6, 2025

ENG-613

@CMCDragonkai
Copy link
Member

CMCDragonkai commented May 7, 2025 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
development Standard development
Development

No branches or pull requests

2 participants