GH-46499: [CI][Crossbow][C++] Use apache/arrow for Meson

[kou]

Rationale for this change

We're using Crossbow to offload CI jobs to separated repository but it has some inconveniences. See #46014 for details.

What changes are included in this PR?

Use CI: Extra label to run Meson CI job as an extra CI job.

Are these changes tested?

Yes.

Are there any user-facing changes?

No.

• GitHub Issue: [CI][Crossbow][C++] Use apache/arrow for Meson #46499

[github-actions]

⚠️ GitHub issue #46499 has been automatically assigned in GitHub to PR creator.

[kou]

pull_request.types: [labeled] will not work well. We need another approach.

[WillAyd]

@WillAyd Can we avoid changing the source directory by Meson? Should we use --wrap-mode=nodownload in CI?

Hmm well I think this will prevent the dependencies from being automatically downloaded, but then you will have to have the dependency installed in the CI container some other way, so not sure that is really the fix.

IIUC the error message is coming from tests that belong to bzip2. I think we can avoid running subproject tests if we change:

meson test

to

meson test "arrow*"

[kou]

Can we disable bundled bzip2 tests entirely?

In general, we don't want to build tests of bundled dependencies. For example:

arrow/cpp/cmake_modules/ThirdpartyToolchain.cmake

Line 937 in f88c19f

-DBUILD_TESTING=OFF

[kou]

https://github.com/mesonbuild/wrapdb/blob/c068c1051eaf88b7c00618295846ddd94467b272/subprojects/packagefiles/bzip2/meson.build#L27-L106

It seems that we can't disable tests...

[WillAyd]

Yep exactly - I can try to upstream a change to the wrapdb, but in its current state you cannot

[WillAyd]

Though interestingly the bzip2 project has its own CMake and Meson configuration files upstream, and neither of them provide an option to disable tests:

https://github.com/libarchive/bzip2/blob/1ea1ac188ad4b9cb662e3f8314673c63df95a589/CMakeOptions.txt

We can probably start by requesting an option in that project directly

[kou]

It makes sense. Could you do it?

[WillAyd]

Sure - see https://gitlab.com/bzip2/bzip2/-/issues/59

[kou]

It seems that we can use meson test --suite arrow to ignore bzip2 tests.

This is ready.

I'll merge this in a few days if nobody objects it.

[WillAyd]

Is this still required?

[WillAyd]

+1

[kou]

No objection. I'll merge this.

[conbench-apache-arrow]

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit b8c9f76.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 2 possible false positives for unstable benchmarks that are known to sometimes produce them.