Repository navigation
🌱 images: bump codex 0.167.0 -> 0.168.0 - #18062
Conversation
Automated pin bump (#8419). Digests recomputed from the downloaded release artifacts by src/scripts/cli-pin-bump.sh: VERSION=0.168.0 Signed-off-by: hive-release[bot] <hive-release@hive.kubestellar.io> Signed-off-by: clubanderson <407614+clubanderson@users.noreply.github.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Changelog: this PR changes code but carries no changelog entry If it is user-visible — a feature, a fix an operator would notice, a This is a reminder, not a gate; it never blocks a merge. |
Open PR overlap check✅ Resolved: no conflicting open PRs or same-file overlaps remain against |
|
Hive status — updated in place; edits do not notify. Full history is in the Activity Monitor. @clubanderson this PR is held at Failing checks:
Next: the hive does not push to your branch. Push a new head (or reply here); CI and the review swarm re-run automatically and the PR returns to the normal path. stage 080-awaiting-merge-admission → 071-needs-author: required-check-failing:coverage Evidence bundle `#18062a8f016e` sha256:279d6f3ccf45b13c0b1cf586f314f43897d17f2605b9cb11c1460f26608b796d — download from the dashboard or `hivectl review evidence #18062`. |
There was a problem hiding this comment.
The diff is correct; the red is a time-bombed test on v5, not this PR.
The change is the two ARG CODEX_VERSION=0.167.0 → 0.168.0 lines (src/Dockerfile:327, src/Dockerfile.contributor:265); the docker (hive) / docker (contributor) / docker (hub) jobs that actually consume the ARG are green on both arches. Nothing in pkg/github is touched.
Why test (rest 3/4) → test → coverage are red (same on #18063, #18064, #18065, and on every hourly "v2 Tests" run of v5@88f6c9c since 2026-11-13 03:31Z):
TestReopenedClaimPersistsAcrossLoad(src/pkg/github/prclaims_reopen_test.go:235) fails deterministically, 3/3 retries, withreloaded = {Repo: Issue:0 …}, falseat line 254.- Fixture
reopenTestMerged = 2026-10-14 01:46:50 UTC(prclaims_reopen_test.go:18); the claim isMergedPR: true(line 25) soSettled()is true (prclaims.go:316-321). settledClaimRetention = 30 * 24 * time.Hour(prclaims.go:92) — sosettledRetained()(prclaims.go:368-374) flipped to false at 2026-11-13 01:46:50 UTC.- The test reloads via
LoadClaimLedger(line 245), which uses the wall clock (loadClaimLedgerAt(path, logger, nil),prclaims.go:1242-1252). The load loop atprclaims.go:1273-1277hitsc.Settled() && !settledRetained(c, now)→continue, dropping the claim, soLookupAnyreturnsok=false.
The sibling tests in that file pin the clock with ledger.SetClock(...); this is the only one going through the wall-clock loader. One-line fix: call loadClaimLedgerAt(path, testLogger(), func() time.Time { return reopenTestReopened }) instead of LoadClaimLedger. No production code change needed.
I tried to file this as an issue but the reviewer tier may not create issues; @clubanderson, this needs a fix on v5 before any of the four pin bumps can go green. I have not changed anything.
— hive: agent=reviewer-2 backend=copilot model=claude-fable-5.1 copilot=1.0.88
|
🐝 Hive Agent: — hive: agent=scanner backend=copilot model=claude-fable-5.1 copilot=1.0.88 |
|
The hive's GitHub App lacks |
|
Needs a decision, not a merge: Clear the cause and the hive re-evaluates on its next pass; the stage moves to |
Picks up the clock-pinned TestReopenedClaimPersistsAcrossLoad so the wall-clock retention flake no longer fails this bump. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Signed-off-by: scanner <280983584+hivecommons-hive[bot]@users.noreply.github.com>
a8f016e
ℹ️ Sentinel notice — informational; author is trusted (owner), merge is not blockedHive flagged this PR (author @clubanderson, head
Hive did not add the |
There was a problem hiding this comment.
Decision needed: @clubanderson should confirm the codex 0.168.0 pin lands as-is, or first update the model lists the bump warned about.
Routing, not objecting. The diff is exactly what the body says: ARG CODEX_VERSION 0.167.0 -> 0.168.0 in src/Dockerfile:327 and src/Dockerfile.contributor:265, and the two values agree (the smoke test asserts this). Image manifests are a trust surface, since this changes code run in every agent image, so I am leaving the sign-off to a person.
One open item from the PR's own provenance block: it warns that the codex static-list comment still says 0.166.0, and the three manual checklist items (cli_models*.go, static/index.html, src/pkg/tokens/pricing.go) are unchecked. I did not verify whether 0.168.0 adds or removes models, so I can't say whether that follow-up is needed or can wait.
I have not approved or closed anything.
— hive: agent=reviewer backend=claude model=claude-sonnet-5-5 claude=2.1.322
Automated pin bump (#8419). Digests recomputed from the downloaded release artifacts by src/scripts/cli-pin-bump.sh: VERSION=0.168.0 Signed-off-by: hive-release[bot] <hive-release@hive.kubestellar.io> Signed-off-by: clubanderson <407614+clubanderson@users.noreply.github.com> Signed-off-by: scanner <280983584+hivecommons-hive[bot]@users.noreply.github.com> Co-authored-by: hive-release[bot] <hive-release@hive.kubestellar.io> Co-authored-by: scanner <280983584+hivecommons-hive[bot]@users.noreply.github.com> Co-authored-by: Claude Sonnet 5.5 <noreply@anthropic.com>
Part of #8419
Automated pin bump: codex 0.167.0 -> 0.168.0 in
src/Dockerfileandsrc/Dockerfile.contributor.Provenance
Every value below was resolved by
src/scripts/cli-pin-bump.shin run https://github.com/hivecommons/hive/actions/runs/43371821632; digests are computed from the downloaded artifacts and cross-checked against the vendor-published digest where one exists (never copied from a page).Smoke (before this PR was opened)
src/Dockerfilebuilt for linux/amd64 with the new pin; inside the image:Model lists (not automated)
src/pkg/dashboard/cli_models*.gomatch the new pin's model set, and their "matches CLI version" comments name itstatic/index.htmlmodel options updatedsrc/pkg/tokens/pricing.gohas prices for any new modelThe docker workflow on this PR builds both images for linux/amd64 and linux/arm64 (every hash-verified layer runs
sha256sum -c/sha512sum -c), and the backend smoke's pinned lane picks the new CLI up on its next run. Seesrc/docs/cli-pins.md.Opened by .github/workflows/cli-pin-bump.yml.