Refresh the package lists before installing in script/bootstrap (closes #508) #509

Merged
clawbot merged 1 commits from issue-508-apt-update-once into next 2026-10-06 07:21:31 +02:00
Collaborator

In script/bootstrap, the apt branch of pkg_install now runs apt-get update once per run, before its first apt-get install, with the same code and comment as the shared copy in sneak/prompts#116. The CI runner image starts with empty package lists, so installing Go failed with E: Unable to locate package golang and every CI run stopped at bootstrap (#508).

Worth knowing: the runner's apt package is Go 1.22. The go command fetches the newer toolchain that go.mod asks for, but the host-side gofmt in script/fmt-check stays at 1.22; it accepts this tree.

The CI job, script/cibuild, passed end to end in the runner's image (docker.gitea.com/runner-images:ubuntu-latest), run locally as root with the host's Docker socket, from a fresh clone of this branch.

Unverified: the branch's own CI run has not run; it is queued behind the runner outage (sneak/project-management#28) and must pass once the runner is back.

Deviation: script/bootstrap now differs from the vendored copy; the next re-vendor replaces it once sneak/prompts#116 lands.

Model: opus-5-5

In `script/bootstrap`, the apt branch of `pkg_install` now runs `apt-get update` once per run, before its first `apt-get install`, with the same code and comment as the shared copy in https://git.eeqj.de/sneak/prompts/pulls/116. The CI runner image starts with empty package lists, so installing Go failed with `E: Unable to locate package golang` and every CI run stopped at bootstrap (https://git.eeqj.de/sneak/webhooker/issues/508). Worth knowing: the runner's apt package is Go 1.22. The `go` command fetches the newer toolchain that `go.mod` asks for, but the host-side `gofmt` in `script/fmt-check` stays at 1.22; it accepts this tree. The CI job, `script/cibuild`, passed end to end in the runner's image (`docker.gitea.com/runner-images:ubuntu-latest`), run locally as root with the host's Docker socket, from a fresh clone of this branch. Unverified: the branch's own CI run has not run; it is queued behind the runner outage (https://git.eeqj.de/sneak/project-management/issues/28) and must pass once the runner is back. Deviation: `script/bootstrap` now differs from the vendored copy; the next re-vendor replaces it once https://git.eeqj.de/sneak/prompts/pulls/116 lands. Model: opus-5-5
clawbot added the needs-review label 2026-10-06 06:53:18 +02:00
clawbot self-assigned this 2026-10-06 06:53:18 +02:00
clawbot added 1 commit 2026-10-06 06:53:19 +02:00
The CI runner image starts with empty package lists, so installing Go
with apt-get failed with "Unable to locate package golang". The apt
branch of pkg_install now runs apt-get update once per run, before the
first apt-get install, with the same code as the shared copy in
sneak/prompts PR 116.

Deviation from the vendored copy, replaced by the next re-vendor once
that PR lands.

Model: opus-5-5
Author
Collaborator
  1. PR body, last two paragraphs: the branch's own CI run is mentioned only as queued, right after a sentence saying the CI job passed, so the open item reads as done. Acceptable: a one-line disclosure in the body, such as "Unverified: the branch's own CI run has not run; it is queued behind the runner outage (sneak/project-management#28) and must pass once the runner is back."

Model: opus-5-5

1. PR body, last two paragraphs: the branch's own CI run is mentioned only as queued, right after a sentence saying the CI job passed, so the open item reads as done. Acceptable: a one-line disclosure in the body, such as "Unverified: the branch's own CI run has not run; it is queued behind the runner outage (https://git.eeqj.de/sneak/project-management/issues/28) and must pass once the runner is back." Model: opus-5-5
Author
Collaborator

Review passed.

Judgement call: accepted under the amended definition of done (#508 (comment)) while the branch's own queued run waits for the runner, as the PR body discloses.

Model: opus-5-5

Review passed. Judgement call: accepted under the amended definition of done (https://git.eeqj.de/sneak/webhooker/issues/508#issuecomment-128399) while the branch's own queued run waits for the runner, as the PR body discloses. Model: opus-5-5
clawbot removed the needs-review label 2026-10-06 07:20:48 +02:00
clawbot merged commit faf3da9a75 into next 2026-10-06 07:21:31 +02:00
clawbot deleted branch issue-508-apt-update-once 2026-10-06 07:21:32 +02:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/webhooker#509