Skip to content

Conversation

@toshihikoyanase
Copy link
Member

Motivation

This pull request updates the Docker image build workflow in .github/workflows/dockerimage.yml to modernize the setup and improve maintainability. The key changes include replacing custom scripting with standardized Docker GitHub Actions (see https://github.com/docker/build-push-action) and simplifying the handling of environment variables.

This PR will change Docker tags as follows:

Current main:

  • pull request trigger: py3.12 will be created, but not pushed
  • release trigger: v0.1.1 and latest will be pushed (unchanged)

This PR:

  • pull request trigger: ci will be created but not pushed
  • release trigger: v0.1.1 and latest will be pushed (unchanged)

Removing the Python version will simplify the CI configuration.

Description of the changes

  • Replaced manual Docker login, QEMU setup, and Buildx setup steps with standardized GitHub Actions.
  • Replaced the custom Docker build and push script with docker/build-push-action@v6, which supports multi-platform builds and integrates seamlessly with GitHub Actions.
  • Removed PYTHON_VERSION from the environment variables and updated the TAG_NAME logic to use a static "ci" tag for non-release events.

@c-bata
Copy link
Member

c-bata commented Jun 12, 2025

@y0z Could you review this PR?

Copy link
Member

@y0z y0z left a comment

Choose a reason for hiding this comment

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

LGTM

@y0z y0z merged commit 37fb1d0 into main Jun 12, 2025
3 checks passed
@y0z y0z deleted the multi-platform-docker branch June 12, 2025 07:55
@toshihikoyanase toshihikoyanase modified the milestones: v0.1.0, v0.2.0 Jun 13, 2025
@toshihikoyanase toshihikoyanase added the enhancement New feature or request label Jun 13, 2025
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.

4 participants