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

Add support for RISC-V #56

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

Conversation

chazapis
Copy link

The work is part of an ongoing effort to run K3s on RISC-V. In K3s, klipper-lb is part of the default deployment.

The resulting container image is built with ARCH=riscv64 make.

@chazapis
Copy link
Author

@matttrach, @galal-hussein, @brandond Sorry to bother you, but it has been over 3 months with no response. This is a really helpful feature that requires no code changes. Could it possibly be included in the next release?

@brandond
Copy link
Member

Thanks for the PR. We're not yet committed to building for risc-v. We will need to tackle this at a project level (including deciding on a standard approach to building for this platform, and adding risc-v support to our mirrored images) before we are ready to start building things for risc-v.

@chazapis
Copy link
Author

@brandond, I understand. Thank you for the feedback. I just want to stress out that even though the RISC-V community is still small, it is constantly growing. Having this project (and eventually K3s) supporting the architecture would be great! Let me know if there is any way I can help.

Signed-off-by: Antony Chazapis <chazapis@ics.forth.gr>
@chazapis
Copy link
Author

chazapis commented Oct 22, 2024

Almost a year later, the tides are changing. As recorded in the K3s on RISC-V PR, most dependencies now support the riscv64 architecture, except klipper-helm and klipper-lb. It also seems that Rancher has started supporting RISC-V images, as is the case with Local Path Provisioner.

I have updated this PR, as the base Alpine image now also supports RISC-V, so there is no need to use another container.

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