Skip to content

fix(docker): install libyaml in arm64#12329

Merged
SkyeYoung merged 1 commit into
apache:masterfrom
SkyeYoung:young/fix/docker/install-libyaml-in-arm64
Jun 16, 2025
Merged

fix(docker): install libyaml in arm64#12329
SkyeYoung merged 1 commit into
apache:masterfrom
SkyeYoung:young/fix/docker/install-libyaml-in-arm64

Conversation

@SkyeYoung

@SkyeYoung SkyeYoung commented Jun 13, 2025

Copy link
Copy Markdown
Member

Description

What:

This PR fixes the issue where the dev image could not be built in arm64 due to the direct copy of Dockerfile.local in the PR #12300.

How:

ref https://github.com/apache/apisix-docker/blob/d5322d57d58c569d98c225f89a8abd991429ec10/debian-dev/Dockerfile#L48

Why:

To allow .github/workflows/push-dev-image-on-commit.yml build the dev image in the arm64 runner.

It has already been tested in PR #12322 and passed CI on both architectures.

Which issue(s) this PR fixes:

Fixes #

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. bug Something isn't working labels Jun 13, 2025
@SkyeYoung
SkyeYoung merged commit ba4bb2b into apache:master Jun 16, 2025
23 checks passed
@SkyeYoung
SkyeYoung deleted the young/fix/docker/install-libyaml-in-arm64 branch June 16, 2025 02:42
laz-xyr pushed a commit to laz-xyr/apisix that referenced this pull request Jun 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants