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 image build for ppc64le architecture #339
Conversation
Signed-off-by: Carlos de Paula <me@carlosedp.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. Before merging this, I'd like to find out why we don't do ppc64le anywhere except for node_exporter.
@SuperQ do you know the context around building node_exporter for ppc64le, while we don't build ppc64le for anything else in the Prometheus org?
Yes, only node_exporter had it. I've also sent PRs to prometheus, alertmanager, snmp_exporter and blackbox_exporter. The last three were already merged. |
@beorn7 It was only on the node_exporter as a test to make sure everything worked ok before we pushed it to the rest of the repos. |
And did it work? (o: |
Both Prometheus and Alertmanager works fine built on ppc64le: Alertmanager
Prometheus:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I've created the quay.io repo for this. (Docker hub auto-creates)
Sorry, I've put the test outputs but not the machine info:
|
Thanks 1M @carlosedp and @SuperQ . |
Add image build for ppc64le architecture
Signed-off-by: Carlos de Paula me@carlosedp.com