Skip to content

feat(ci): Enable cruise-control on ppc64le arch#2326

Merged
CCisGG merged 1 commit intolinkedin:mainfrom
saitejx08:cruisebranch
Nov 6, 2025
Merged

feat(ci): Enable cruise-control on ppc64le arch#2326
CCisGG merged 1 commit intolinkedin:mainfrom
saitejx08:cruisebranch

Conversation

@saitejx08
Copy link
Contributor

Summary

  1. Why: To enable CI/CD builds for the ppc64le architecture, as requested in issue Enable cruise-control on ppc64le arch via CI/CD #1486.
  2. What: This PR adds 'ppc64le' to the hw_platform matrix within the build-platform job in the .github/workflows/ci.yaml file.

Actual Behavior

The CI pipeline does not currently run builds or tests on the ppc64le architecture.

Steps to Reproduce

N/A

Known Workarounds

N/A

Additional evidence

N/A

Categorization

  • documentation
  • bugfix
  • new feature
  • refactor
  • security/CVE
  • other (CI/CD change)

This PR resolves #1486.

@kyguy
Copy link
Contributor

kyguy commented Oct 29, 2025

Hi @saitejx08, to make sure I understand this will emulate ppc64le on the GitHub-hosted runner just like it does for s390x, is that correct?

@saitejx08
Copy link
Contributor Author

Yes, @kyguy that is exactly correct.

My change just adds ppc64le to the existing hw_platform matrix, so it will use the same QEMU emulation (through the existing tonistiigi/binfmt and docker run --platform steps) that was already in place for the s390x build.

Thanks for checking!

Copy link
Contributor

@kyguy kyguy left a comment

Choose a reason for hiding this comment

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

Thanks for the explanation @saitejx08, that should work fine!

@saitejx08 saitejx08 requested a review from kyguy October 30, 2025 17:17
Copy link
Contributor

@kyguy kyguy left a comment

Choose a reason for hiding this comment

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

These CI changes look fine, we just need a maintainer to approve and merge.

@saitejx08
Copy link
Contributor Author

saitejx08 commented Oct 31, 2025

Sounds good @kyguy, Thanks for the review.

@saitejx08
Copy link
Contributor Author

hey @CCisGG can you look into this, i really appreciate if it gets merged.

@CCisGG CCisGG merged commit 3c7f9c6 into linkedin:main Nov 6, 2025
7 checks passed
georgew5656 pushed a commit to DataDog/cruise-control that referenced this pull request Dec 4, 2025
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.

Enable cruise-control on ppc64le arch via CI/CD

3 participants