Skip to content

[deckhouse-cli] Re-land stronghold v1.18.6 on the debian builder image - #436

Merged
ldmonster merged 2 commits into
mainfrom
test/reland-stronghold-new-builder
Aug 19, 2026
Merged

[deckhouse-cli] Re-land stronghold v1.18.6 on the debian builder image#436
ldmonster merged 2 commits into
mainfrom
test/reland-stronghold-new-builder

Conversation

@Glitchy-Sheep

@Glitchy-Sheep Glitchy-Sheep commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Brings back #415 (stronghold v1.18.6 + GOST Go toolchain), reverted in #433. The trdl release build now runs on builder/golang-debian-1.26 from base-images v1.3.24 instead of builder/golang-alt.

Why #415 broke the release

  • the trdl server runs mkdir -p /result as the image user; builder/golang-alt runs as 64535, so it failed
  • the commands in trdl.yaml are written for Debian (apt-get); the image is ALT

What changed vs #415

Only trdl.yaml. The rest is #415 as merged.

  • image: container-factory@sha256:de392f69... = builder/golang-debian-1.26: root, Debian 13, same GOST go1.26.5 as GO_IMAGE in CI
  • zlib1g-dev + CGO_LDFLAGS=-lz: Debian 13 links libcrypto.a with zlib
  • no libuv_a.a symlink: Debian 13 installs libuv.a where the linker looks for it
  • task comes from go install ...@v3.41.0: the image has no curl/wget

Tests

  • Reproduce trdl release build (added in [deckhouse-cli] Revert "Bump stronghold to v1.18.6" and guard CI #433) is green: real trdl Dockerfile, real image digest, all 4 binaries in /result
  • Build / Test / Lint are green on the GOST toolchain
  • a linux binary built the trdl way inside the image runs on a cluster node: d8 stronghold version = v1.18.6, kubectl and delivery-kit work

@Glitchy-Sheep
Glitchy-Sheep force-pushed the test/reland-stronghold-new-builder branch 2 times, most recently from 3a67d1f to 0b633ef Compare August 4, 2026 20:36
This reverts commit 399c8e5.

Signed-off-by: Roman Berezkin <roman.berezkin@flant.com>
The golang-alt builder from #415 cannot work under trdl: non-root
user and ALT userland. base-images v1.3.24 builders run as root, and
the debian flavor keeps the existing apt/dpkg commands working.

Command changes for the new image:
- task is installed via go install: the image ships no curl, and the
  module checksum db replaces the manual sha256 pin
- the libuv_a.a symlink workaround is dropped: debian 13 ships the
  static lib as libuv.a in the default linker path
- zlib1g-dev + CGO_LDFLAGS=-lz: debian 13 builds libcrypto.a with
  zlib, so the static link needs -lz appended

Signed-off-by: Roman Berezkin <roman.berezkin@flant.com>
@Glitchy-Sheep
Glitchy-Sheep force-pushed the test/reland-stronghold-new-builder branch from 0b633ef to 595d39b Compare August 17, 2026 05:34
@Glitchy-Sheep Glitchy-Sheep changed the title [deckhouse-cli] Re-land stronghold v1.18.6 with builder/golang-debian (test) [deckhouse-cli] Re-land stronghold v1.18.6 on the debian builder image Aug 17, 2026
@Glitchy-Sheep Glitchy-Sheep self-assigned this Aug 17, 2026
@Glitchy-Sheep Glitchy-Sheep added the enhancement New feature or request label Aug 17, 2026
@Glitchy-Sheep
Glitchy-Sheep marked this pull request as ready for review August 17, 2026 05:41
@ldmonster
ldmonster merged commit ca5aa94 into main Aug 19, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants