Skip to content

enable imagelist argument for image build script - #95

Merged
Rui Gao (hippogr) merged 3 commits into
devfrom
ruigao/build_single_image
Nov 7, 2025
Merged

enable imagelist argument for image build script#95
Rui Gao (hippogr) merged 3 commits into
devfrom
ruigao/build_single_image

Conversation

@hippogr

Copy link
Copy Markdown
Contributor

This PR is used to add -imagelist argument for pai-build script when building the docker image. With this argument, we can build a single docker image in a service without building the whole service. For example:

 ./build/pai_build.py build -c ../config-test/ -n -s log-manager -i log-manager-cleaner

to build log-manager-cleaner but not build log-manager-nginx

Copilot AI review requested due to automatic review settings October 14, 2025 06:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 adds support for an -imagelist argument to the pai-build script, enabling selective building of specific Docker images within a service instead of building all images in the service. This allows developers to build only the images they need, improving build efficiency and reducing unnecessary build time.

  • Added -i/--imagelist command-line argument to specify which images to build
  • Modified the build pipeline to pass the imagelist filter through the build components
  • Updated the build logic to conditionally build only specified images when the filter is provided

Reviewed Changes

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

File Description
build/pai_build.py Added imagelist argument parser and passed args to BuildCenter
build/core/build_center.py Modified constructor to accept arg_config and pass imagelist to build handler
build/core/build_handler.py Added imagelist parameter to build_single_component with conditional image building

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread build/core/build_center.py
Comment thread build/pai_build.py
@zhogu

zhogu commented Oct 29, 2025

Copy link
Copy Markdown
Contributor

can ./build/pai_build.py push -s [service] -i [image] support this feature?

@yukirora Yuting Jiang (yukirora) mentioned this pull request Nov 4, 2025
26 tasks
Comment thread build/pai_build.py
Comment thread build/pai_build.py
@hippogr
Rui Gao (hippogr) merged commit 35b2a3c into dev Nov 7, 2025
4 checks passed
@yukirora Yuting Jiang (yukirora) mentioned this pull request Nov 11, 2025
25 tasks
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.

4 participants