Skip to content

fix(flake.nix): add procps to nix dogfood image #16607

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

Conversation

ThomasK33
Copy link
Member

@ThomasK33 ThomasK33 commented Feb 18, 2025

Add procps to flake.nix and release name to Docker image

Adds the procps package to flake.nix to enable the free command, and includes a release name file in the Docker image at /etc/coderniximage-release.

Change-Id: I85432acc06a204229fa3675e0020bd3acacf775a
Signed-off-by: Thomas Kosiewski tk@coder.com

Change-Id: I85432acc06a204229fa3675e0020bd3acacf775a
Signed-off-by: Thomas Kosiewski <tk@coder.com>
Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@ThomasK33 ThomasK33 marked this pull request as ready for review February 18, 2025 14:58
@ThomasK33 ThomasK33 requested a review from johnstcn February 18, 2025 14:59
@@ -50,6 +50,10 @@ let
experimental-features = nix-command flakes
'';

etcReleaseName = writeTextDir "etc/coderniximage-release" ''
0.0.0
Copy link
Member

Choose a reason for hiding this comment

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

nit: wonder if it would make sense to have some kind of versioning here?

Copy link
Member Author

Choose a reason for hiding this comment

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

I agree. It would make sense to get some versioning here.

Right now, we're not versioning the dogfood images, so this file is mostly a stub/placeholder for the system info go library not to fail.

Copy link
Member Author

Choose a reason for hiding this comment

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

I could add the output of git describe --tags here, or we could consider versioning the dogfood images, in which case the image tag should go here.

Copy link
Member

Choose a reason for hiding this comment

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

That or git SHA would probably be fine!

@ThomasK33 ThomasK33 merged commit 00e2703 into main Feb 18, 2025
35 checks passed
@ThomasK33 ThomasK33 deleted the thomask33/02-18-fix_flake.nix_add_procps_to_nix_dogfood_image branch February 18, 2025 15:24
@github-actions github-actions bot locked and limited conversation to collaborators Feb 18, 2025
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.

2 participants