Skip to content

MINOR: [CI][C++] Show machine info in Linux GHA jobs#49736

Merged
pitrou merged 1 commit intoapache:mainfrom
pitrou:gh-sve
Apr 14, 2026
Merged

MINOR: [CI][C++] Show machine info in Linux GHA jobs#49736
pitrou merged 1 commit intoapache:mainfrom
pitrou:gh-sve

Conversation

@pitrou
Copy link
Copy Markdown
Member

@pitrou pitrou commented Apr 14, 2026

Rationale for this change

Detailed info about available memory and CPU model can help in certain cases, for example to know whether a certain instruction set is available (example in #47769).

Are these changes tested?

N/A.

Are there any user-facing changes?

No.

@github-actions github-actions bot added the awaiting review Awaiting review label Apr 14, 2026
@pitrou pitrou changed the title MINOR: [CI] Show machine info in GHA jobs MINOR: [CI][C++] Show machine info in Linux GHA jobs Apr 14, 2026
@pitrou pitrou marked this pull request as ready for review April 14, 2026 10:00
Copy link
Copy Markdown
Member

@raulcd raulcd left a comment

Choose a reason for hiding this comment

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

LGTM, is this better than the following?

      - name: CPU Info
        run: |
          sysctl -a | grep cpu
          sysctl -a | grep "hw.optional"

Maybe we can update the macOS intel step to match this one?
See:
https://github.com/apache/arrow/actions/runs/24392607522/job/71242267624?pr=49736#step:2:29

@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting review Awaiting review labels Apr 14, 2026
@raulcd
Copy link
Copy Markdown
Member

raulcd commented Apr 14, 2026

Ok, those doesn't exist on macOS so I was just non-sensing here :)

@pitrou pitrou added the CI: Extra: C++ Run extra C++ CI label Apr 14, 2026
@pitrou pitrou merged commit 36c498d into apache:main Apr 14, 2026
24 checks passed
@pitrou pitrou removed the awaiting merge Awaiting merge label Apr 14, 2026
@pitrou pitrou deleted the gh-sve branch April 14, 2026 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI: Extra: C++ Run extra C++ CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants