Skip to content

chore(makefile): update Go compiler image to golang:1.24.0-bullseye-c…#3312

Merged
LordofAvernus merged 1 commit into
mainfrom
dsb_add_parameters
May 21, 2026
Merged

chore(makefile): update Go compiler image to golang:1.24.0-bullseye-c…#3312
LordofAvernus merged 1 commit into
mainfrom
dsb_add_parameters

Conversation

@iwanghc
Copy link
Copy Markdown
Collaborator

@iwanghc iwanghc commented May 21, 2026

User description

…ross

关联的 issue

https://github.com/actiontech/sqle-ee/issues/2884

描述你的变更

修复go pipeline打arm包失败,更新golang版本,把 ARM 交叉编译器装进 bullseye 镜像

确认项(pr提交后操作)

Tip

请在指定复审人之前,确认并完成以下事项,完成后✅


  • 我已完成自测
  • 我已记录完整日志方便进行诊断
  • 我已在关联的issue里补充了实现方案
  • 我已在关联的issue里补充了测试影响面
  • 我已确认了变更的兼容性,如果不兼容则在issue里标记 not_compatible
  • 我已确认了是否要更新文档,如果要更新则在issue里标记 need_update_doc

link #3309


Description

  • 修改Makefile中Go编译器的镜像配置

  • 将镜像值更新为golang:1.24.0-bullseye-cross

  • 修复ARM交叉编译时使用错误镜像的问题


Diagram Walkthrough

flowchart LR
  A["Makefile更新"] -- "替换镜像值" --> B["golang:1.24.0-bullseye-cross"]
Loading

File Walkthrough

Relevant files
Bug fix
Makefile
更新编译器镜像配置支持ARM交叉编译                                                                             

Makefile


  • 将GO_COMPILER_IMAGE的值从"golang:1.24.0-bullseye"修改为"golang:1.24.0-bullseye-cross"
  • 确保ARM交叉编译器正确集成
+1/-1     

@actiontech-bot actiontech-bot requested review from LordofAvernus and removed request for LordofAvernus May 21, 2026 10:32
@github-actions
Copy link
Copy Markdown

PR Reviewer Guide 🔍

🎫 Ticket compliance analysis 🔶

2884 - Partially compliant

Compliant requirements:

  • 修复ARM交叉编译时使用错误镜像的问题

Non-compliant requirements:

  • 执行报错规则需单独分类显示到审核结果界面
  • 数据库结构对比审核结果
  • 文件审核count计数统计

Requires further human verification:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions
Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

@LordofAvernus LordofAvernus merged commit 71712b3 into main May 21, 2026
4 checks passed
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.

2 participants