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

DEV: Build base images for ARM64 platform in a seperate daily job #744

Closed
wants to merge 4 commits into from

Conversation

tgxworld
Copy link
Contributor

Why this change?

We're currently building a base image for arm64 platforms as part of the
CI workflow. However, building it on Github action via emulation is too
slow and makes it meaningless to include as part of the CI workflow.

What does this change do?

This change moves the building of the bage image targeting the arm64
platform completely to a scheduled job that runs outside of the normal
CI workflow. This will also enable us to build the images for the test
and dev environment eventually as part of the workflow which we intend
to do in a follow up commit.

@tgxworld tgxworld force-pushed the build_arm_images_daily branch 3 times, most recently from 2724164 to d2447d1 Compare October 16, 2023 03:00
@@ -0,0 +1,138 @@
on:
pull_request:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Enabling it on pull requests first so that we run the job in this PR and I know I got things right. Will remove before merging.

env:
BUILDKIT_PROGRESS: plain

jobs:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There is alot of duplication between this new workflow and build.yml but I want to get things running and see if we can even run everything in a reasonable amount of time first.

@davidtaylorhq
Copy link
Member

My preference would be to keep things as-is, so that we continue to get feedback on the ARM build in PRs/commits. If we have a time-sensitive change, we can still choose to merge before it completes.

If we do decide to proceed with this approach, I think we should avoid duplicating the workflow definition, and instead add a conditional to the existing ARM job. We do something similar in plugin/theme workflows here.

So in this case, to ignore push/pr events, we could do something like:

aarch64:
  if: ${{ github.event_name == 'schedule' }}

@tgxworld tgxworld force-pushed the build_arm_images_daily branch 4 times, most recently from 166f117 to a07ef89 Compare October 17, 2023 08:14
Why this change?

We're currently building a base image for arm64 platforms as part of the
CI workflow. However, building it on Github action via emulation is too
slow and makes it meaningless to include as part of the CI workflow.

What does this change do?

This change moves the building of the bage image targeting the arm64
platform completely to a scheduled job that runs outside of the normal
CI workflow. This will also enable us to build the images for the test
and dev environment eventually as part of the workflow which we intend
to do in a follow up commit.
@tgxworld tgxworld force-pushed the build_arm_images_daily branch from a07ef89 to efd4424 Compare October 17, 2023 08:18
@tgxworld
Copy link
Contributor Author

No longer required because we have access to Github hosted ARM runners now.

@tgxworld tgxworld closed this Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants