MINOR: [CI][C++] Enable core dumps and stack traces in Linux/macOS jobs #43937
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pitrou
requested review from
assignUser,
jonkeane,
kou and
raulcd
as code owners
pitrou
force-pushed
the
minor-core-dumps
branch
from
d16d213
to
32ab6de
Compare
|
@github-actions crossbow submit -g cpp |
|
@github-actions crossbow submit cuda wheelcp312* |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
|
@kou Does this look ok? |
kou
reviewed
Sep 4, 2024
github-actions
bot
added
awaiting changes
pitrou
force-pushed
the
minor-core-dumps
branch
from
80b2205
to
44b7271
Compare
|
@github-actions crossbow submit -g cpp |
github-actions
bot
added
Component: C++
awaiting change review
|
Revision: 44b7271 Submitted crossbow builds: ursacomputing/crossbow @ actions-3a98f95e07
|
|
Ok, we're now getting automatic backtraces on Linux x86, macOS x86, macOS ARM64 (but for some reason the backtrace on macOS ARM64 is incomplete). On the Linux ARM64 self-hosted runner, setting a custom core pattern is forbidden. |
|
@github-actions crossbow submit test-fedora-39-cpp test-ubuntu-20.04-cpp-bundled test-ubuntu-20.04-cpp-minimal-with-formats test-ubuntu-24.04-cpp-gcc-13-bundled |
|
Revision: 5aa9253 Submitted crossbow builds: ursacomputing/crossbow @ actions-262efe6260
|
|
@github-actions crossbow submit -g cpp -g python -g r |
|
@github-actions crossbow submit wheellinux* wheelmacos* |
pitrou
changed the title
|
@kou This PR went a bit further as it now re-enables automatic tracebacks on Linux and macOS (see example builds above). Do you want to give it another review? |
|
Revision: c65527a Submitted crossbow builds: ursacomputing/crossbow @ actions-272dc0e9a9
|
|
Revision: c65527a Submitted crossbow builds: ursacomputing/crossbow @ actions-20f4348f5c
|
bkietz
approved these changes
Sep 10, 2024
github-actions
bot
added
awaiting merge
kou
approved these changes
Sep 11, 2024
There was a problem hiding this comment.
+1
Great! It'll be very helpful to debug CI failures.
It seems that Flight related core dump in "test-ubuntu-24.04-cpp-gcc-14" doesn't work well but we can work on this later:
Test #77: arrow-flight-test ............................***Failed 0.28 sec
Running arrow-flight-test, redirecting output into /build/cpp/build/test-logs/arrow-flight-test.txt (attempt 1/1)
/arrow/cpp/build-support/run-test.sh: line 88: 47057 Aborted (core dumped) $TEST_EXECUTABLE "$@" > $LOGFILE.raw 2>&1
Running main() from ./googletest/src/gtest_main.cc
kou
reviewed
Sep 11, 2024
Comment on lines
103
to
+104
| run: > | ||
| source ci/scripts/util_enable_core_dumps.sh |
There was a problem hiding this comment.
Oh, this breaks integration test.
(archery docker run ... isn't executed.)
github-actions
bot
added
awaiting changes
|
After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit e710b6e. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. It also includes information about 5 possible false positives for unstable benchmarks that are known to sometimes produce them. |
khwilson
pushed a commit
to khwilson/arrow
that referenced
this pull request
Sep 14, 2024
…bs (apache#43937) ### Rationale for this change In apache#43936 I noticed that core dumps were not written out for crashing C++ tests. One problem is that, by default, Ubuntu hosts pipe core dumps to `apport`, but it is not available inside containers. Another is that the `ulimit` must be set in the host, not in the container. In addition, this PR restores automatic traceback generation when running C++ tests, on Linux and macOS jobs. ### Are these changes tested? Manually by introducing a spurious segfault and running Docker containers. ### Are there any user-facing changes? No. Lead-authored-by: Antoine Pitrou <[email protected]> Co-authored-by: Antoine Pitrou <[email protected]> Co-authored-by: Sutou Kouhei <[email protected]> Signed-off-by: Sutou Kouhei <[email protected]>
|
I'm seeing the following on the R binary builds, which looks related? https://github.com/ursacomputing/crossbow/actions/runs/10808019175/job/29980088987 |
|
Oh, it should be |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Rationale for this change
In #43936 I noticed that core dumps were not written out for crashing C++ tests. One problem is that, by default, Ubuntu hosts pipe core dumps to
apport, but it is not available inside containers. Another is that theulimitmust be set in the host, not in the container.In addition, this PR restores automatic traceback generation when running C++ tests, on Linux and macOS jobs.
Are these changes tested?
Manually by introducing a spurious segfault and running Docker containers.
Are there any user-facing changes?
No.