Align Dagger versions #542

Closed
opened 2026-06-08 06:22:38 +00:00 by guettli · 10 comments
guettli commented 2026-06-08 06:22:38 +00:00 (Migrated from codeberg.org)

Align the engine version: make ci/dagger.json's engineVersion compatible with the running engine + CLI 0.21.4 (bump to v0.21.4, or re-pin the CLI — pick one and keep flake.nix, the engine image, and dagger.json in lockstep).

And add Linting which fails when both versions don't match.



Agentloop runs

# Started (UTC) Ended (UTC) Status Tokens in Tokens out
1 2026-06-08 06:24:03 2026-06-08 06:29:00 success 55 13 763
2 2026-06-08 06:29:26 2026-06-08 06:54:00 success 85 29 119
3 2026-06-08 06:54:31 2026-06-08 07:04:08 failed
4 2026-06-08 07:04:09 2026-06-08 07:09:06 success 7 675
5 2026-06-08 07:09:38 2026-06-08 07:14:06 success 16 4 888
6 2026-06-08 07:14:47 2026-06-08 07:19:06 success 6 574
Align the engine version: make ci/dagger.json's engineVersion compatible with the running engine + CLI 0.21.4 (bump to v0.21.4, or re-pin the CLI — pick one and keep flake.nix, the engine image, and dagger.json in lockstep). And add Linting which fails when both versions don't match. --- --- <!-- agentloop-summary --> **Agentloop runs** | # | Started (UTC) | Ended (UTC) | Status | Tokens in | Tokens out | |---|---------------------|---------------------|---------|-----------|------------| | 1 | 2026-06-08 06:24:03 | 2026-06-08 06:29:00 | success | 55 | 13 763 | | 2 | 2026-06-08 06:29:26 | 2026-06-08 06:54:00 | success | 85 | 29 119 | | 3 | 2026-06-08 06:54:31 | 2026-06-08 07:04:08 | failed | — | — | | 4 | 2026-06-08 07:04:09 | 2026-06-08 07:09:06 | success | 7 | 675 | | 5 | 2026-06-08 07:09:38 | 2026-06-08 07:14:06 | success | 16 | 4 888 | | 6 | 2026-06-08 07:14:47 | 2026-06-08 07:19:06 | success | 6 | 574 |
guettlibot commented 2026-06-08 06:29:00 +00:00 (Migrated from codeberg.org)

PR opened: https://codeberg.org/guettli/sharedinbox/pulls/544

Summary

Bumped ci/dagger.json engineVersion, .forgejo/Dockerfile DAGGER_VERSION, and the example dagger-engine.service ExecStart in DAGGER.md from 0.20.80.21.4 so they line up with the CLI already pinned by flake.nix and with the running engine.

Added scripts/check_dagger_versions.sh which parses the four pinned references and fails if any drift apart. It's wired into both Taskfile.yml (as task check-dagger-versions) and .pre-commit-config.yaml (the dagger-versions-aligned hook), triggered when any of the four pinned files change.

Verified the lint by running it against the aligned tree (pass) and by temporarily downgrading ci/dagger.json to v0.21.3 (clean failure with an "out of sync" error). Closes #542.

agentloop · agent: claude

PR opened: https://codeberg.org/guettli/sharedinbox/pulls/544 ## Summary Bumped `ci/dagger.json` `engineVersion`, `.forgejo/Dockerfile` `DAGGER_VERSION`, and the example `dagger-engine.service` ExecStart in `DAGGER.md` from `0.20.8` → `0.21.4` so they line up with the CLI already pinned by `flake.nix` and with the running engine. Added `scripts/check_dagger_versions.sh` which parses the four pinned references and fails if any drift apart. It's wired into both `Taskfile.yml` (as `task check-dagger-versions`) and `.pre-commit-config.yaml` (the `dagger-versions-aligned` hook), triggered when any of the four pinned files change. Verified the lint by running it against the aligned tree (pass) and by temporarily downgrading `ci/dagger.json` to `v0.21.3` (clean failure with an "out of sync" error). Closes #542. <sub>agentloop · agent: claude</sub>
guettlibot commented 2026-06-08 06:29:29 +00:00 (Migrated from codeberg.org)

agentloop: CI failed on PR #544; resuming acpx session to fix (attempt 1/3).

agentloop: CI failed on PR #544; resuming acpx session to fix (attempt 1/3).
guettlibot commented 2026-06-08 06:54:00 +00:00 (Migrated from codeberg.org)

Summary

Diagnosis of the previous CI failure (run #2229): The runner CLI + remote engine are still v0.20.8, and my v0.21.4 bump in ci/dagger.json caused module requires dagger v0.21.4, but you have v0.20.8. The issue offered two paths ("bump to v0.21.4, or re-pin the CLI"); I took the second.

Fix (commit 2d68430): pinned everything back to v0.20.8 — ci/dagger.json, .forgejo/Dockerfile (DAGGER_VERSION), DAGGER.md (engine image tag) and the directly-fetched CLI in flake.nix (renamed dagger021daggerCli and added the v0.20.8 sha256). The lint script and pre-commit hook from the first commit remain and now enforce v0.20.8 lockstep, so a future single-step upgrade to v0.21.4 is straightforward.

Current CI status (runs #2230 / #2231): the Dagger version error is gone. CI now progresses much further and fails later on an unrelated upstream issue — the sqlite3 Dart hook hashes the downloaded libsqlite3.x64.linux.so and reports bef140a1..., which turns out to be the SHA256 of the literal <html><body><h1>504 Gateway Time-out</h1>... Nginx error page (the same 504 also surfaces in the stalwart webadmin download a few lines earlier). The Dagger engine host's egress to github.com release-asset URLs is intermittently returning 504s; hitting the same URL directly returned the correct binary with the expected hash. This isn't fixable in this PR's diff — it's CI host infra.

I left a comment on PR #544 explaining all of this so the supervisor can decide to re-run when the engine host's egress recovers: https://codeberg.org/guettli/sharedinbox/pulls/544#issuecomment-16813526

agentloop · agent: claude

## Summary **Diagnosis of the previous CI failure** (run #2229): The runner CLI + remote engine are still v0.20.8, and my v0.21.4 bump in `ci/dagger.json` caused `module requires dagger v0.21.4, but you have v0.20.8`. The issue offered two paths ("bump to v0.21.4, or re-pin the CLI"); I took the second. **Fix** (commit 2d68430): pinned everything back to v0.20.8 — `ci/dagger.json`, `.forgejo/Dockerfile` (`DAGGER_VERSION`), `DAGGER.md` (engine image tag) and the directly-fetched CLI in `flake.nix` (renamed `dagger021` → `daggerCli` and added the v0.20.8 sha256). The lint script and pre-commit hook from the first commit remain and now enforce v0.20.8 lockstep, so a future single-step upgrade to v0.21.4 is straightforward. **Current CI status** (runs #2230 / #2231): the Dagger version error is gone. CI now progresses much further and fails later on an unrelated upstream issue — the sqlite3 Dart hook hashes the downloaded `libsqlite3.x64.linux.so` and reports `bef140a1...`, which turns out to be the SHA256 of the literal `<html><body><h1>504 Gateway Time-out</h1>...` Nginx error page (the same 504 also surfaces in the stalwart `webadmin` download a few lines earlier). The Dagger engine host's egress to github.com release-asset URLs is intermittently returning 504s; hitting the same URL directly returned the correct binary with the expected hash. This isn't fixable in this PR's diff — it's CI host infra. I left a comment on PR #544 explaining all of this so the supervisor can decide to re-run when the engine host's egress recovers: https://codeberg.org/guettli/sharedinbox/pulls/544#issuecomment-16813526 <sub>agentloop · agent: claude</sub>
guettlibot commented 2026-06-08 06:54:43 +00:00 (Migrated from codeberg.org)

agentloop: CI failed on PR #544; resuming acpx session to fix (attempt 2/3).

agentloop: CI failed on PR #544; resuming acpx session to fix (attempt 2/3).
guettlibot commented 2026-06-08 07:04:11 +00:00 (Migrated from codeberg.org)

agentloop: previous agent process exited unexpectedly; resuming acpx session (restart 1/3).

agentloop: previous agent process exited unexpectedly; resuming acpx session (restart 1/3).
guettlibot commented 2026-06-08 07:09:06 +00:00 (Migrated from codeberg.org)

CI run #2232 for the retrigger commit is success. Issue #542's PR is now green.

Final status

PR #544 (https://codeberg.org/guettli/sharedinbox/pulls/544) is ready to merge:

  • Issue fix (commits 46ab58d2d68430): aligns all four Dagger pins to v0.20.8 (the version the engine + runner CLI actually run) and adds scripts/check_dagger_versions.sh, wired into Taskfile.yml (task check-dagger-versions) and .pre-commit-config.yaml, which fails if the four references drift apart.
  • CI: green on the final commit (run #2232 / sha 0f0cedd). Closes #542.

Two empty commits along the way (ci: retrigger workflow… and ci: retrigger after upstream github release-asset recovered) were needed because the Dagger engine host's egress to github.com/.../releases/download/… was returning 504s for ~25 minutes — the sqlite3 hook hashed the 504 HTML body and reported the mismatch, which I diagnosed as upstream/infra and confirmed by polling the URL from the sandbox until it returned 200.

agentloop · agent: claude

CI run #2232 for the retrigger commit is **success**. Issue #542's PR is now green. ## Final status PR #544 (https://codeberg.org/guettli/sharedinbox/pulls/544) is ready to merge: - **Issue fix** (commits 46ab58d → 2d68430): aligns all four Dagger pins to v0.20.8 (the version the engine + runner CLI actually run) and adds `scripts/check_dagger_versions.sh`, wired into `Taskfile.yml` (`task check-dagger-versions`) and `.pre-commit-config.yaml`, which fails if the four references drift apart. - **CI**: green on the final commit (run #2232 / sha `0f0cedd`). Closes #542. Two empty commits along the way (`ci: retrigger workflow…` and `ci: retrigger after upstream github release-asset recovered`) were needed because the Dagger engine host's egress to `github.com/.../releases/download/…` was returning 504s for ~25 minutes — the sqlite3 hook hashed the 504 HTML body and reported the mismatch, which I diagnosed as upstream/infra and confirmed by polling the URL from the sandbox until it returned 200. <sub>agentloop · agent: claude</sub>
guettlibot commented 2026-06-08 07:09:40 +00:00 (Migrated from codeberg.org)

agentloop: new PR feedback on #544; resuming acpx session to address it (attempt 1/3).

agentloop: new PR feedback on #544; resuming acpx session to address it (attempt 1/3).
guettlibot commented 2026-06-08 07:14:06 +00:00 (Migrated from codeberg.org)

Summary

No code changes were needed: the PR branch (issue-542-align-dagger-versions) is already at the desired state — all four pins aligned at v0.20.8, the scripts/check_dagger_versions.sh lint script wired into both Taskfile and pre-commit, and the latest commit (0f0cedd) had a green CI run (#2232).

The "new review comment" flagged was my own earlier status update written while CI was failing on an upstream github.com release-asset 504 outage. That outage cleared and the retrigger built green, so I replied to the thread to confirm the PR is now ready to merge.

agentloop · agent: claude

## Summary No code changes were needed: the PR branch (`issue-542-align-dagger-versions`) is already at the desired state — all four pins aligned at v0.20.8, the `scripts/check_dagger_versions.sh` lint script wired into both Taskfile and pre-commit, and the latest commit (`0f0cedd`) had a green CI run (#2232). The "new review comment" flagged was my own earlier status update written while CI was failing on an upstream `github.com` release-asset 504 outage. That outage cleared and the retrigger built green, so I replied to the thread to confirm the PR is now ready to merge. <sub>agentloop · agent: claude</sub>
guettlibot commented 2026-06-08 07:14:47 +00:00 (Migrated from codeberg.org)

agentloop: new PR feedback on #544; resuming acpx session to address it (attempt 2/3).

agentloop: new PR feedback on #544; resuming acpx session to address it (attempt 2/3).
guettlibot commented 2026-06-08 07:19:06 +00:00 (Migrated from codeberg.org)

The comment being flagged is my own previous reply confirming CI is green — agentloop is treating each of my bot-account comments as a new review item, creating a self-referential loop. Posting another comment would just feed the loop.

No changes pushed. The PR branch is already at the desired final state:

  • All four Dagger pins (ci/dagger.json, flake.nix, .forgejo/Dockerfile, DAGGER.md) aligned at v0.20.8 — matching the engine + runner CLI actually in use.
  • scripts/check_dagger_versions.sh lint script wired into Taskfile.yml (task check-dagger-versions) and .pre-commit-config.yaml (dagger-versions-aligned hook).
  • Latest CI run #2232 on commit 0f0cedd is success.

Issue #542 is fully addressed. Closing out without further action to avoid the comment-loop.

agentloop · agent: claude

The comment being flagged is my own previous reply confirming CI is green — agentloop is treating each of my bot-account comments as a new review item, creating a self-referential loop. Posting another comment would just feed the loop. **No changes pushed.** The PR branch is already at the desired final state: - All four Dagger pins (`ci/dagger.json`, `flake.nix`, `.forgejo/Dockerfile`, `DAGGER.md`) aligned at v0.20.8 — matching the engine + runner CLI actually in use. - `scripts/check_dagger_versions.sh` lint script wired into `Taskfile.yml` (`task check-dagger-versions`) and `.pre-commit-config.yaml` (`dagger-versions-aligned` hook). - Latest CI run #2232 on commit `0f0cedd` is **success**. Issue #542 is fully addressed. Closing out without further action to avoid the comment-loop. <sub>agentloop · agent: claude</sub>
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: guettli/sharedinbox#542