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

feat: build arm images #786

Merged
merged 1 commit into from
Jan 10, 2025
Merged

feat: build arm images #786

merged 1 commit into from
Jan 10, 2025

Conversation

6ixfalls
Copy link
Contributor

@6ixfalls 6ixfalls commented Jan 10, 2025

Kutt used to provide arm images for versions before v3. I'm assuming the build works fine because there's an arm version pushed for v3 which seems to be a mistake; but the Dockerfile may need to be adapted if it doesn't support ARM.

@poeti8
Copy link
Contributor

poeti8 commented Jan 10, 2025

The build kept failing for the ARM architecture because npm was hanging:
https://github.com/thedevs-network/kutt/actions/runs/12680476630/job/35343075489

I removed platforms for now until I find a solution. I can't merge this one unless we make sure that the image gets build successfully.

@6ixfalls
Copy link
Contributor Author

The workflow run/file you linked includes armv7 that shouldn't conflict with arm64, the 64 bit version of ARM. From those logs it looks like arm64 built fine, and the newer platforms run on that architecture.

@6ixfalls
Copy link
Contributor Author

For reference, I found this issue that seems to describe the same issue above.
nodejs/docker-node#2077

@poeti8
Copy link
Contributor

poeti8 commented Jan 10, 2025

Oh I see, I'll merge this one then.

@poeti8 poeti8 merged commit c253b75 into thedevs-network:main Jan 10, 2025
@6ixfalls
Copy link
Contributor Author

Any way to get a release tagged (or builds rerun on the latest version)?

@6ixfalls 6ixfalls deleted the patch-1 branch January 11, 2025 01:50
@poeti8
Copy link
Contributor

poeti8 commented Jan 11, 2025

Done

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