Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Testing] Crash reporting CDT tests #25100

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from
Draft

[Testing] Crash reporting CDT tests #25100

wants to merge 1 commit into from
+3 −0

Conversation

pgellert
Copy link
Contributor

I'm just testing some failing CDT tests through this PR.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v24.3.x
  • v24.2.x
  • v24.1.x

Release Notes

  • none

@pgellert pgellert self-assigned this Feb 18, 2025
@pgellert
Copy link
Contributor Author

/cdt
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

1 similar comment
@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

@pgellert pgellert force-pushed the crashlog/fix-cdt-tests branch from c81ffcc to 7224f8a Compare February 18, 2025 14:54
@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

1 similar comment
@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

@pgellert pgellert force-pushed the crashlog/fix-cdt-tests branch from aa73468 to 3bbc68c Compare February 18, 2025 15:45
@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

@pgellert pgellert force-pushed the crashlog/fix-cdt-tests branch from 3bbc68c to 23c617b Compare February 19, 2025 09:52
@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

1 similar comment
@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

@pgellert pgellert force-pushed the crashlog/fix-cdt-tests branch from a0b7538 to abe2d49 Compare February 19, 2025 11:59
@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

@pgellert pgellert force-pushed the crashlog/fix-cdt-tests branch from abe2d49 to 54a736e Compare February 19, 2025 12:14
@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

@pgellert pgellert force-pushed the crashlog/fix-cdt-tests branch from 54a736e to 418605e Compare February 19, 2025 12:44
@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

2 similar comments
@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

@pgellert pgellert force-pushed the crashlog/fix-cdt-tests branch from c3c2b01 to f94e69f Compare February 19, 2025 14:48
@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

1 similar comment
@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

@pgellert pgellert force-pushed the crashlog/fix-cdt-tests branch from 1750595 to 68c962f Compare February 19, 2025 15:38
@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

@pgellert
dt: disable core dumps in crash tracker tests
d336b73 
I chose to set the core_pattern as it is the most reliable and simplest
option for our current setup.

Note that in our current CDT setup, core dumps are not written to a file
directly by the kernel but instead use a piping mechanism where they
pipe the coredump to `apport`. Because of this, the RLIMIT_CORE of the
process (set through ulimit) is not respected, meaning ulimit alone
cannot disable core dumps.

For reference, see the "Piping core dumps to a program" section of
https://www.man7.org/linux/man-pages/man5/core.5.html
@pgellert pgellert force-pushed the crashlog/fix-cdt-tests branch from 68c962f to d336b73 Compare February 19, 2025 16:14
@pgellert
Copy link
Contributor Author

/cdt
rp_repo=nightly
rp_version=latest
tests/rptest/tests/crash_loop_checks_test.py::CrashLoopChecksTest.test_crash_report_with_signal

@pgellert pgellert mentioned this pull request Feb 19, 2025
Draft
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@pgellert pgellert

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@pgellert