Skip to content

build(ci): pin Go in migration-script-lint like the other container jobs - #53

Closed
DoDiODev wants to merge 1 commit into
release-pr9096-basefrom
ci/pr-9096
Closed

DoDiODev wants to merge 1 commit into
release-pr9096-basefrom
ci/pr-9096

Conversation

@DoDiODev

Copy link
Copy Markdown
Owner

Validation only - DO NOT MERGE. Replays the unmodified upstream workflows for apache#9096 (head 0065127 on upstream/main 9abdf41). Base is a release-* branch so test.yml, test-e2e.yml and grafana-dashboards-check trigger like on a PR to main.

The job runs in mericodev/lake-builder:latest, which still ships Go 1.20.4,
while go.mod declares go 1.26. test.yml, test-e2e.yml and golangci-lint.yml
already install the pinned toolchain via backend/scripts/install-go.sh; this
job was the only container job left out.
It currently passes because Go <= 1.20 does not treat a higher go directive as
a hard error, but it compiles and runs the linter under a language version six
major releases behind what the project declares.
The step is idempotent and costs 2.8 seconds when nothing is pre-installed, on
a job of roughly 45 seconds. libgit2 and mockery are not needed here.
@DoDiODev

Copy link
Copy Markdown
Owner Author

Validation for apache#9096 completed (8/8 upstream workflows green). Closing.

@DoDiODev DoDiODev closed this Sep 23, 2026
@DoDiODev
DoDiODev deleted the ci/pr-9096 branch September 23, 2026 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant