Skip to content

Also build for linux/riscv64 on release#22256

Merged
Jafaral merged 1 commit into
FRRouting:masterfrom
mmoll:riscv
Jun 10, 2026
Merged

Also build for linux/riscv64 on release#22256
Jafaral merged 1 commit into
FRRouting:masterfrom
mmoll:riscv

Conversation

@mmoll

@mmoll mmoll commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

I did a successful build on native hardware.

@greptile-apps

greptile-apps Bot commented Jun 8, 2026

Copy link
Copy Markdown

Greptile Summary

This PR adds linux/riscv64 to the list of target platforms in the docker buildx build command documented in the release procedure, enabling RISC-V 64-bit Docker images to be produced during releases.

  • The single-line change appends linux/riscv64 to the comma-separated --platform argument in doc/developer/frr-release-procedure.rst, consistent with how the existing architectures are listed.
  • The daily master CI workflow (docker-daily-master.yml) uses a smaller subset of platforms and is not updated here, which is consistent with the pre-existing pattern where the daily build already omits several architectures (e.g., linux/ppc64le, linux/s390x, linux/arm/v6) that the release procedure includes.

Confidence Score: 5/5

Safe to merge — a one-word documentation change that appends a new build target to a shell command example.

The change is a single addition to a comma-separated platform list in a documentation file. There is no runtime code involved, and the author reports a successful native-hardware build for the new target.

No files require special attention.

Important Files Changed

Filename Overview
doc/developer/frr-release-procedure.rst Adds linux/riscv64 to the docker buildx platform list in the release procedure build command

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Release Tag frr-X.Y.Z] --> B[docker buildx build --platform ...]
    B --> C[linux/amd64]
    B --> D[linux/arm64]
    B --> E[linux/ppc64le]
    B --> F[linux/s390x]
    B --> G[linux/arm/v7]
    B --> H[linux/arm/v6]
    B --> I[linux/riscv64 NEW]
    C & D & E & F & G & H & I --> J[quay.io/frrouting/frr:X.Y.Z]
    J --> K[git tag docker/X.Y.Z]
    K --> L[git push origin docker/X.Y.Z]
Loading

Reviews (1): Last reviewed commit: "Also build for linux/riscv64 on release" | Re-trigger Greptile

Signed-off-by: Michael Moll <kvedulv@kvedulv.de>
@donaldsharp

Copy link
Copy Markdown
Member

Before we can claim support we need to do some actual work on the backend to ensure that it is working and stays working

@Jafaral Jafaral merged commit 9494cef into FRRouting:master Jun 10, 2026
23 checks passed
@mmoll mmoll deleted the riscv branch June 10, 2026 19:17
@Jafaral

Jafaral commented Jun 12, 2026

Copy link
Copy Markdown
Member

@Mergifyio backport stable/10.7 stable/10.6

@mergify

mergify Bot commented Jun 12, 2026

Copy link
Copy Markdown

backport stable/10.7 stable/10.6

✅ Backports have been created

Details

donaldsharp added a commit that referenced this pull request Jun 12, 2026
Also build for linux/riscv64 on release (backport #22256)
donaldsharp added a commit that referenced this pull request Jun 12, 2026
Also build for linux/riscv64 on release (backport #22256)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants