[github-actions] upload core dump, so libs and binaries for crashed programs #5861
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
Irving-cl
force-pushed
the
upload_core_dump_on_fail
branch
from
34249c2 to
3e347af
Compare
Size Report of OpenThread
|
Codecov Report
@@ Coverage Diff @@
## master #5861 +/- ##
==========================================
- Coverage 87.14% 87.12% -0.02%
==========================================
Files 372 372
Lines 47044 47107 +63
==========================================
+ Hits 40995 41043 +48
- Misses 6049 6064 +15
|
moandor-y
reviewed
Nov 25, 2020
Irving-cl
force-pushed
the
upload_core_dump_on_fail
branch
from
3e347af to
1f6b08a
Compare
|
Can we add docs somewhere that includes the steps in #5812 (comment)? |
@jwhui |
@Irving-cl , sounds good to me! |
…er testing when programs crash
Irving-cl
force-pushed
the
upload_core_dump_on_fail
branch
2 times, most recently
from
b1323ea to
bd7a4de
Compare
|
Updated |
Irving-cl
force-pushed
the
upload_core_dump_on_fail
branch
from
bd7a4de to
29bcbc4
Compare
Irving-cl
force-pushed
the
upload_core_dump_on_fail
branch
from
29bcbc4 to
5365cec
Compare
Vyrastas
requested changes
Nov 30, 2020
|
The suggested version is much better. Thanks! |
Vyrastas
approved these changes
Dec 1, 2020
jwhui
approved these changes
Dec 2, 2020
gabekassel
pushed a commit
to gabekassel/openthread
that referenced
this pull request
Jan 21, 2021
…rograms (openthread#5861) This commit adds action to upload core dumps, so libs and binaries for 1.2 certification. This is intended to solve issues where programs intermittently crash however hard to reproduce it. This is added first for thread-1-2 and low-power-packet-verification first as an experimental work. We can add same actions for other items when it's necessary.
Abhayakara
pushed a commit
to Abhayakara/openthread
that referenced
this pull request
Oct 6, 2021
…rograms (openthread#5861) This commit adds action to upload core dumps, so libs and binaries for 1.2 certification. This is intended to solve issues where programs intermittently crash however hard to reproduce it. This is added first for thread-1-2 and low-power-packet-verification first as an experimental work. We can add same actions for other items when it's necessary.
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.
This PR adds action to upload core dumps, so libs and binaries for 1.2 certification. This is intended to solve issues like #5802, #5774 where programs intermittently crash however hard to reproduce it. Related issue: #5805.
I added this for
thread-1-2andlow-power-packet-verificationfirst as an experimental work. We can add same actions for other items when it's necessary.I've verified that we can analyze the core dumps with the artifacts using gdb successfully on our local PC: #5812 (comment)