Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

ThomasK33
Copy link
Member

@ThomasK33ThomasK33 commentedFeb 18, 2025
edited
Loading

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

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

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

Change-Id: I85432acc06a204229fa3675e0020bd3acacf775aSigned-off-by: Thomas Kosiewski <tk@coder.com>
@ThomasK33Graphite App
Copy link
MemberAuthor

This stack of pull requests is managed byGraphite. Learn more aboutstacking.

@ThomasK33ThomasK33 marked this pull request as ready for reviewFebruary 18, 2025 14:58
@@ -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
MemberAuthor

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
MemberAuthor

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 ofgit 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!

@ThomasK33ThomasK33 merged commit00e2703 intomainFeb 18, 2025
35 checks passed
@ThomasK33ThomasK33 deleted the thomask33/02-18-fix_flake.nix_add_procps_to_nix_dogfood_image branchFebruary 18, 2025 15:24
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsFeb 18, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers

@johnstcnjohnstcnjohnstcn approved these changes

Assignees

@ThomasK33ThomasK33

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@ThomasK33@johnstcn

[8]ページ先頭

©2009-2025 Movatter.jp