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

ROX-22866 save core dumps to collector ci artifacts with test names #1595

Draft
wants to merge 41 commits into
base: master
Choose a base branch
from
Draft

ROX-22866 save core dumps to collector ci artifacts with test names #1595

wants to merge 41 commits into from

Conversation

JoukoVirtanen
Copy link
Contributor

@JoukoVirtanen JoukoVirtanen commented Mar 13, 2024 •
edited
Loading

This is an attempted improvement over #1589

The goal is to make it easier to match core dumps to the integration tests that they belong to.

@JoukoVirtanen
X-Smart-Branch-Parent: master
f7acef7
@JoukoVirtanen
Trying to get core dumps from CI artifacts.
8bc3ded
@JoukoVirtanen
Ansible should get the core dumps and move them to the host machine
1178910
@JoukoVirtanen
Added debugging
63173a2
@JoukoVirtanen
Moved the location of abort. Temporarily removed NetworkStatusNotifie…
72b4034 
…rTest
@JoukoVirtanen
More debugging
fa37810
@JoukoVirtanen
Not checking if the tests are running locally before modifying the co…
c42bec5 
…re dump file path
@JoukoVirtanen
Not using sudo to modify core_pattern file
95dfff1
@JoukoVirtanen
More debugging
bad2bf4
@JoukoVirtanen
Trying to change core dump file path in the ansible playbook
0d2516c
@JoukoVirtanen
Checking tmp directory of integration tests
a3c8d0b
@JoukoVirtanen
Trying to change the permissions of the core dump file
c9a5c44
@JoukoVirtanen
Core dump permissions are changed in the always section
3fda8d2
@JoukoVirtanen
Trying to check tmp directory
c41966b
@JoukoVirtanen
Better debugging
801f645
@JoukoVirtanen
Using root user to fetch core dump
9264a4e
@JoukoVirtanen
Getting core dump files of the form /tmp/core.out.* not just /tmp/cor…
564ae4e 
…e.out
@JoukoVirtanen
Fixed error so that all core dumps should be fetched
b4292c1
@JoukoVirtanen
Fetching core dump files in a loop
84e4aa3
@JoukoVirtanen
Trying to fix error
577afc4
@JoukoVirtanen
Using root to fetch core dump files
56328e2
@JoukoVirtanen
Trying to follow pattern for fetching log files
3c11cd4
@JoukoVirtanen
Moving core dump files to their own directory
f492202
@JoukoVirtanen
Clean up
4d7a7a7
@JoukoVirtanen
Removed intentional crash. Restored unit tests
f683eff
@JoukoVirtanen
Removed unneeded line
f9268d3
@JoukoVirtanen
Removed blank lines
f645a1d
@JoukoVirtanen
Trying to add test name to core dump path. Adding intentional seg fau…
25eab83 
…lt back in
@JoukoVirtanen
Fixed indentation
9e7f634
@JoukoVirtanen
Trying to fix Write integration test log
9144e14
@JoukoVirtanen JoukoVirtanen mentioned this pull request Mar 14, 2024
Merged
3 tasks
@JoukoVirtanen JoukoVirtanen changed the title Jv rox 22866 save core dumps to collector ci artifacts segfault Jv rox 22866 save core dumps to collector ci artifacts with test names Mar 14, 2024
@robbycochran robbycochran changed the title Jv rox 22866 save core dumps to collector ci artifacts with test names ROX-22866 save core dumps to collector ci artifacts with test names Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
debug do-not-merge/work-in-progress
Projects
None yet
Milestone
No milestone
Development

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