Skip to content

ci: update CI dockerfile for improved reliability and security#35168

Merged
feici02 merged 3 commits intomainfrom
ci/update-dockerfile-ci
Apr 21, 2026
Merged

ci: update CI dockerfile for improved reliability and security#35168
feici02 merged 3 commits intomainfrom
ci/update-dockerfile-ci

Conversation

@tomchon
Copy link
Copy Markdown
Contributor

@tomchon tomchon commented Apr 17, 2026

概述

更新 CI Dockerfile (tests/ci/dockerfile_ci),提升构建的可靠性和安全性。

主要改动

  • 固定基础镜像标签:将 python:3.9.25 改为 python:3.9.25-bookworm,确保构建环境可复现
  • 替换 PyPI 镜像源:将内部 IP 地址的镜像源 (192.168.0.212) 替换为公共的 nexus.tdengine.net,避免依赖内网环境
  • 改进 .NET SDK 安装方式:从运行时下载脚本改为使用预构建的 tarball (dotnet-sdk-6.0.100-linux-x64.tar.gz),提升构建速度和稳定性
  • 修复依赖包名称:将 zlib1g-dev 替换为 libz-dev 以提升兼容性
  • 完善 SSH 配置:创建 /var/run/sshd 目录,确保 SSH 服务可正常启动
  • 清理冗余注释:移除 SSH 配置块中的行内注释

tomchon and others added 2 commits April 14, 2026 18:04
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Pin base image to python:3.9.25-bookworm for reproducibility
- Replace internal PyPI mirror with public nexus.tdengine.net
- Switch dotnet SDK installation from script to pre-built tarball
- Replace zlib1g-dev with libz-dev for compatibility
- Create /var/run/sshd directory for SSH daemon
- Remove inline comment in SSH configuration block
Copilot AI review requested due to automatic review settings April 17, 2026 03:34
@tomchon tomchon requested review from a team, guanshengliang and zitsen as code owners April 17, 2026 03:34
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the CI Dockerfile to make CI builds more reproducible and less dependent on internal network resources, while also adjusting a few build dependencies and SSH setup.

Changes:

  • Pin the builder base image to python:3.9.25-bookworm and adjust Debian packages (e.g., libz-dev).
  • Switch pip index configuration from an internal IP-based mirror to nexus.tdengine.net.
  • Change .NET installation approach to use a prebuilt SDK tarball and ensure /var/run/sshd exists for SSH startup.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 3 comments.

File Description
tests/ci/dockerfile_ci Updates base image, apt dependencies, pip index configuration, .NET SDK installation, and SSH runtime directory creation.
.gitignore Ignores .worktrees/ and normalizes the debug_coverage/ entry formatting.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread tests/ci/dockerfile_ci Outdated
Comment thread tests/ci/dockerfile_ci
Comment thread tests/ci/dockerfile_ci
@feici02 feici02 merged commit c58fcff into main Apr 21, 2026
11 of 12 checks passed
@feici02 feici02 deleted the ci/update-dockerfile-ci branch April 21, 2026 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants