Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: run docker build separately #583

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Revolyssup
Copy link
Contributor

No description provided.

runs-on: ubuntu-latest

build-arm64:
runs-on: buildjet-4vcpu-ubuntu-2204-arm
Copy link
Member

Choose a reason for hiding this comment

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

you can't use buildjet in apache repo

nic-6443

This comment was marked as outdated.

Copy link
Member

@nic-6443 nic-6443 left a comment

Choose a reason for hiding this comment

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

There are a few issues in the process of building the dev image:

  1. The built image relies on api7/apisix-runtime:dev, which hasn't been updated for 5 months. It comes from https://github.com/api7/apisix-build-tools/blob/master/.circleci/config.yml, and I suspect this workflow is also not working anymore. Therefore, I hope to modify debian-dev/Dockerfile to be the same as debian/Dockerfile, so it no longer depends on api7/apisix-runtime:dev.
  2. You can temporarily modify the trigger conditions of this Workflow so that it can build once during a PR to verify that your modifications solve the problem, avoiding having to merge into the main branch each time before discovering issues.

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