Update stacked: 5 packages - #522
Merged
Merged
Conversation
The 0.157.0 bump built clean at the version level but FTBFS on both arches:
go: go.opentelemetry.io/obi@v0.10.0 (replaced by ../../../internal/obi-src):
reading /internal/obi-src/go.mod: no such file or directory
Three upstream build-topology changes between 0.150.1 and 0.157.0, none visible
to a version check:
1. OBI is now compiled INTO otelcol-contrib. The contrib manifest gained
`- gomod: go.opentelemetry.io/obi v0.10.0` plus
`replaces: go.opentelemetry.io/obi => ../../../internal/obi-src`.
That directory is not in the repo (internal/ holds only tools/) — upstream
materializes it via scripts/prepare-obi.sh, which curls the
"source-generated" tarball (pre-generated BPF objects, so no clang/bpf2go).
Declare that tarball as a checksummed Source and stage it in build.sh, per
repo convention that every input is pinned by sha256.
2. ocb must run FROM distributions/otelcol-contrib. `output_path: ./_build` is
CWD-relative and the replace path is resolved from the output dir, so running
at the source root put _build at <root>/_build and ../../../ escaped above the
root — hence the absolute /internal/obi-src/go.mod in the error.
3. The builder pin was hardcoded at v0.150.0 while the package version was
0.150.1, so pkgmgr's build.sh version rewrite never matched it and it stayed
behind — the failing log shows otelcol/service resolving at v0.150.0 while
every other component came from 0.157.0. Pin it byte-equal to the package
version so the rewrite carries it forward on the next bump.
NOT yet build-verified — needs a clean-room build before it ships.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Member
|
Pushed a fix for the The failure's real error was the last line under ~60 lines of Three upstream build-topology changes between 0.150.1 and 0.157.0, none of them visible to a version check:
Note the other 3 non-shipping members are unrelated and unchanged: |
| { | ||
| url = "https://github.com/cython/cython/releases/download/%{version}/cython-%{version}.tar.gz", | ||
| sha256 = "f4f23a56b25221a06f91817fe8f3114ab8b48a4fac73187dbb64bc2c4a87961f", | ||
| url = "https://github.com/cython/cython/releases/download/3.2.9/cython-%{version}.tar.gz", |
Member
There was a problem hiding this comment.
This seems like a regression?
twitchyliquid64
approved these changes
Jul 24, 2026
twitchyliquid64
left a comment
Member
There was a problem hiding this comment.
Aside from the version string thing, lgtm
Tom caught this: the URL came out half-templated —
releases/download/3.2.9/cython-%{version}.tar.gz
so the tag segment was pinned to a literal while the filename stayed
%{version}. The next bump would move the filename and leave the tag behind,
producing a 404.
Cause is pkgmgr's rewrite_release_download_tag (pkgmgr-rs#578), added so
ghostscript's non-version-derivable tag (gs10060 -> gs10071) gets rewritten on a
bump. It replaces the /releases/download/<tag>/ segment unconditionally, so a
tag that was ALREADY %{version} got clobbered with a literal. Correct for
ghostscript, wrong wherever the tag is derivable — the tool fix follows
separately so this cannot recur.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update base-soup (5 packages)
Note
These packages declare
replace_on_cyclein their build.ncl, sothey participate in the toolchain rebuild graph and one hash change
cascades through the set. Bundling ensures the cascading rebuild
lands as a single unit, even when only one package is bumping —
avoids back-to-back full rebuilds from singleton PRs.
Warning
3 requested members did NOT ship — each needs separate follow-up.
A member is dropped when it fails to update, has its gs:// mirror
withheld by the scan gate, or is peeled to keep the shipped set a
closed dependency closure (a survivor built against a dropped member's
old version would be a broken closure).
charon0.1.2230.1.225prek0.4.100.4.11cloud-sql-proxy2.23.02.24.0Note
Build risk — 1 dependent across the tree. Package(s) that
build- or runtime-depend on a member of this bundle may need a rebuild,
or could FTBFS on an API/ABI change. Informational (not blocking) — a
heads-up for the reviewer on what this bump can ripple into.
cythonnumpySummary
3.2.83.2.9github:cython/cython:release-asset:operator-pinned0.17.30.17.4github:modem-dev/hunk:operator-pinned0.150.10.157.0github:open-telemetry/opentelemetry-collector-releases:operator-pinned3.253.03.254.0github:pulumi/pulumi:operator-pinned8.8.08.8.1github:redis/redis:operator-pinnedPer-package details
cython 3.2.8 → 3.2.9f4f23a56b25221a0...→d249c9022ab13286...https://github.com/cython/cython/releases/download/3.2.8/cython-3.2.8.tar.gz→https://github.com/cython/cython/releases/download/3.2.9/cython-3.2.9.tar.gzApache-2.0(source: GitHub + tarball)3.2.9(the upstream tag for 3.2.9); the previous tag did not track the version and would 404 at build time.hunk 0.17.3 → 0.17.486c37686352fcc69...→f37c7e7c492a60f8...https://github.com/modem-dev/hunk/archive/refs/tags/v0.17.3.tar.gz→https://github.com/modem-dev/hunk/archive/refs/tags/v0.17.4.tar.gzMIT(source: GitHub + tarball)otel-collector 0.150.1 → 0.157.0082a7d7816733676...→b8058145292617c8...https://github.com/open-telemetry/opentelemetry-collector-releases/archive/refs/tags/v0.150.1.tar.gz→https://github.com/open-telemetry/opentelemetry-collector-releases/archive/refs/tags/v0.157.0.tar.gzApache-2.0(source: GitHub + tarball)pulumi 3.253.0 → 3.254.06bffd86f18ebeefb...→dfec607f9030b48a...gs://minimal-staging-archives/pulumi/pulumi/v3.253.0.tar.gz→gs://minimal-staging-archives/pulumi/pulumi/v3.254.0.tar.gzApache-2.0(source: GitHub + tarball)redis 8.8.0 → 8.8.119736ce6117d90b3...→56008a7e9c87ed4e...gs://minimal-staging-archives/redis/redis/8.8.0.tar.gz→gs://minimal-staging-archives/redis/redis/8.8.1.tar.gzCreated by pkgmgr