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: disable arm build for chainguard #3039

Merged
merged 14 commits into from
May 17, 2024
Merged

fix: disable arm build for chainguard #3039

merged 14 commits into from
May 17, 2024

Conversation

MthwRobinson
Copy link
Contributor

Summary

Temporarily disables the ARM build due to the error in this CI job. Will add back support for ARM using the rockylinux container once we show this works.

@MthwRobinson MthwRobinson requested a review from tabossert May 16, 2024 17:21
@MthwRobinson MthwRobinson enabled auto-merge May 16, 2024 23:49
@MthwRobinson MthwRobinson added this pull request to the merge queue May 17, 2024
Merged via the queue into main with commit 934f1a4 May 17, 2024
42 checks passed
@MthwRobinson MthwRobinson deleted the fix/only-amd-build branch May 17, 2024 00:53
github-merge-queue bot pushed a commit that referenced this pull request May 17, 2024
### Summary

Closes #3034 and reenables ARM64 in the docker build and publish job.
This was taken out in #3039 because we've only build `libreoffice` for
AMD64 and not ARM64. If Chainguard publishes an `apk` for `libreoffice`,
we can support a Chainguard image for both architectures. The smoke test
now differs for both architectures, to reflect differences in the
directory structure.

### Testing

Build and publish ran successfully for ARM64 (job
[here](https://github.com/Unstructured-IO/unstructured/actions/runs/9129712470/job/25104907497))
and AMD64 (job
[here](https://github.com/Unstructured-IO/unstructured/actions/runs/9129712470/job/25104907826)).
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