Skip to content

Switch from x86 to ARM build host for POWER8+ builds.#19575

Merged
Ferroin merged 2 commits intonetdata:masterfrom
Ferroin:ppc64le-alt-emu
Feb 7, 2025
Merged

Switch from x86 to ARM build host for POWER8+ builds.#19575
Ferroin merged 2 commits intonetdata:masterfrom
Ferroin:ppc64le-alt-emu

Conversation

@Ferroin
Copy link
Member

@Ferroin Ferroin commented Feb 4, 2025

Summary

This should improve build times at least a little bit.

Test Plan

Requires confirmation of 64-bit POWER8+ builds being successful in CI and taking less time than they currently do on the master branch.

This should improve build times at least a little bit.
It should have been added previously, but got missed.
@Ferroin Ferroin marked this pull request as ready for review February 6, 2025 13:06
@Ferroin Ferroin requested a review from a team as a code owner February 6, 2025 13:06
@Ferroin Ferroin requested a review from a team February 6, 2025 13:06
Copy link
Contributor

@thiagoftsm thiagoftsm left a comment

Choose a reason for hiding this comment

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

I am approving after CI success.

@Ferroin Ferroin merged commit a5c5936 into netdata:master Feb 7, 2025
117 checks passed
@Ferroin Ferroin deleted the ppc64le-alt-emu branch February 7, 2025 11:53
stelfrag pushed a commit to stelfrag/netdata that referenced this pull request Feb 11, 2025
* Switch from x86 to ARM build host for POWER8+ builds.

This should improve build times at least a little bit.

* Include distro config in Docker CI skip check.

It should have been added previously, but got missed.

(cherry picked from commit a5c5936)
@stelfrag stelfrag mentioned this pull request Feb 11, 2025
Ferroin added a commit that referenced this pull request Feb 12, 2025
* Switch from x86 to ARM build host for POWER8+ builds.

This should improve build times at least a little bit.

* Include distro config in Docker CI skip check.

It should have been added previously, but got missed.

(cherry picked from commit a5c5936)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants