E534 ci(docker): build and publish linux/riscv64 images by benevolentshrine · Pull Request #3578 · prometheus/node_exporter · GitHub
[go: up one dir, main page]

Skip to content

ci(docker): build and publish linux/riscv64 images#3578

Open
benevolentshrine wants to merge 1 commit intoprometheus:masterfrom
benevolentshrine:feat/riscv64-docker-images
Open

ci(docker): build and publish linux/riscv64 images#3578
benevolentshrine wants to merge 1 commit intoprometheus:masterfrom
benevolentshrine:feat/riscv64-docker-images

Conversation

@benevolentshrine
Copy link

Adds linux/riscv64 support for Docker image builds.

Fixes #3311

Changes

  • Add riscv64 to DOCKER_ARCHS in Makefile
  • Un-ignore .build/linux-riscv64 in .dockerignore

Notes

Previously attempted in #3312 but was closed when the author deleted
their repository. That PR had received LGTM from a member and was
awaiting final review from @SuperQ.

@SuperQ
Copy link
Member
SuperQ commented Mar 9, 2026

This needs a DCO sign-off. You can use git commit -s --amend to add it.

@benevolentshrine benevolentshrine force-pushed the feat/riscv64-docker-images branch from 7242104 to ef9f55d Compare March 9, 2026 17:33
@benevolentshrine
Copy link
Author

Hey @SuperQ! All checks are passing including DCO ✅, would love to get your eyes on this when you get a chance. Happy to make any changes if needed!

Copy link
Member
@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

Let's fix the sort order while we're here.

@benevolentshrine benevolentshrine force-pushed the feat/riscv64-docker-images branch from 809bf7b to 82617c1 Compare March 10, 2026 10:03
@benevolentshrine benevolentshrine requested a review from SuperQ March 10, 2026 10:17
@SuperQ
Copy link
Member
SuperQ commented Mar 10, 2026

Can you fix the commit history here?

@benevolentshrine benevolentshrine force-pushed the feat/riscv64-docker-images branch 2 times, most recently from 99dc128 to 662fdd6 Compare March 10, 2026 10:54
@benevolentshrine
Copy link
Author

Hey @SuperQ! Applied your suggestions and cleaned up the commit history. All checks passing including DCO ✅, ready for your re-review when you get a chance!

@SuperQ
Copy link
Member
SuperQ commented Mar 10, 2026

Sorry, it looks like the suggestions were not applied correctly and the git history is still incorrect.

Adds riscv64 to DOCKER_ARCHS in Makefile and un-ignores
.build/linux-riscv64 in .dockerignore.

Fixes prometheus#3311

Signed-off-by: benevolentshrine <proffersor45@gmail.com>
@benevolentshrine benevolentshrine force-pushed the feat/riscv64-docker-images branch from 3eeb6de to d1c11b6 Compare March 10, 2026 13:55
@benevolentshrine
Copy link
Author

Hey @SuperQ fixed the commit history, only one clean commit now. Ready for re-review

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.

Provide linux/riscv64 container images

2 participants

0