Run Dockerfile build, test and lint steps at nice -n 19 #64

Closed
clawbot wants to merge 1 commits from issue-63-nice-docker-steps into next
Collaborator

Closes #63.

Every RUN step that builds, tests or lints now runs under nice -n 19, per the owner's order. Installs, dependency downloads, copies and the runtime ENTRYPOINT are unchanged.

  • prompts/REPO_POLICIES.md gains the rule as one bullet, next to the Dockerfile rule, and last_modified moves to today.
  • The canonical Go Dockerfile in REPO_POLICIES.md follows it: the lint run, both go test runs (including the verbose rerun) and the go build.
  • This repo's own Dockerfile follows it: the prettier lint step, and the test placeholder echo, so a repo seeded from it keeps nice when it puts a real test command there.
  • nice comes from busybox in the alpine images and coreutils in the golangci-lint image, so no stage needed anything installed.

Disclosures:

  • next did not exist; it was created from main for this PR.
  • Judgement call: TODO.md gets a Completed Steps entry for this work; its Next Step is unrelated and left as is.
  • Not done here: README.md says the template repos (template-app-go, template-app-js, template-app-python) must be kept in sync with policy changes; they are not on the rollout list at sneak/project-management#15.

Model: opus-5-5

Closes https://git.eeqj.de/sneak/prompts/issues/63. Every `RUN` step that builds, tests or lints now runs under `nice -n 19`, per the owner's order. Installs, dependency downloads, copies and the runtime `ENTRYPOINT` are unchanged. - `prompts/REPO_POLICIES.md` gains the rule as one bullet, next to the `Dockerfile` rule, and `last_modified` moves to today. - The canonical Go `Dockerfile` in `REPO_POLICIES.md` follows it: the lint run, both `go test` runs (including the verbose rerun) and the `go build`. - This repo's own `Dockerfile` follows it: the prettier lint step, and the test placeholder `echo`, so a repo seeded from it keeps `nice` when it puts a real test command there. - `nice` comes from busybox in the alpine images and coreutils in the golangci-lint image, so no stage needed anything installed. Disclosures: - `next` did not exist; it was created from `main` for this PR. - Judgement call: `TODO.md` gets a Completed Steps entry for this work; its Next Step is unrelated and left as is. - Not done here: `README.md` says the template repos (`template-app-go`, `template-app-js`, `template-app-python`) must be kept in sync with policy changes; they are not on the rollout list at https://git.eeqj.de/sneak/project-management/issues/15. Model: opus-5-5
clawbot self-assigned this 2026-09-29 13:06:45 +02:00
clawbot added 1 commit 2026-09-29 13:06:46 +02:00
The owner ordered every build, test and lint step in every Dockerfile to
run at the lowest priority. REPO_POLICIES.md gains that rule in one
line, and both Dockerfiles this repo hands out follow it: its own, and
the canonical Go example in REPO_POLICIES.md. Installs, dependency
downloads and the runtime entrypoint are left at normal priority.

Model: opus-5-5
clawbot added the needs-review label 2026-09-29 13:06:46 +02:00
Author
Collaborator

Stopped on sneak's word, 2026-09-29 in chat (verbatim):

stop the rollout.

Why: a nice value only orders processes within their own CPU group, and every container (and every RUN step of a build) is its own group, so nice -n 19 inside a Dockerfile does not lower a build's priority against other containers. Closed without changes.

Model: opus-5-5

Stopped on sneak's word, 2026-09-29 in chat (verbatim): > stop the rollout. Why: a nice value only orders processes within their own CPU group, and every container (and every `RUN` step of a build) is its own group, so `nice -n 19` inside a Dockerfile does not lower a build's priority against other containers. Closed without changes. Model: opus-5-5
clawbot closed this pull request 2026-09-29 13:12:24 +02:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/prompts#64