-
-
Notifications
You must be signed in to change notification settings - Fork 290
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
test: enable coredump for the unit tests (#6107)
* Add support for core dump * Increase the ulimit for the debugger * Refactor the action steps as tasks * Refactor the action steps as tasks * Update the action path * Update the action path * Fix the syntax error * Fix the conditions
- Loading branch information
1 parent
3f3a207
commit 228d7ba
Showing
3 changed files
with
62 additions
and
5 deletions.
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
name: 'Take core dump files' | ||
description: 'List down and upload core dumps as artifacts' | ||
runs: | ||
using: "composite" | ||
steps: | ||
- run: ls -l | ||
shell: sh | ||
|
||
- name: Backup core dump | ||
uses: actions/upload-artifact@v2 | ||
with: | ||
name: core-dump | ||
path: core.* |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,39 @@ | ||
name: 'Setup node with debug support' | ||
description: 'Setup the nodejs version with debug support' | ||
inputs: | ||
node-version: | ||
description: 'Version of nodejs' | ||
required: true | ||
default: '20' | ||
debug: | ||
description: Enable the debug version | ||
required: true | ||
default: 'false' | ||
|
||
runs: | ||
using: "composite" | ||
steps: | ||
- uses: actions/setup-node@v3 | ||
with: | ||
node-version: ${{ inputs.node-version }} | ||
check-latest: true | ||
cache: yarn | ||
|
||
# For now we only have the Node 20 debug build | ||
- run: sudo apt-get install unzip && curl -L "https://drive.google.com/uc?export=download&id=1hlhbbQi-NJi8_WjULvOdo-K_tfZFzN3Z&confirm=t" > nodejs.zip && unzip nodejs.zip | ||
shell: sh | ||
if: ${{ inputs.debug == 'true' }} | ||
- run: sudo cp -f node $(which node) | ||
shell: sh | ||
if: ${{ inputs.debug == 'true' }} | ||
- run: sudo sh -c "ulimit -c unlimited" | ||
shell: sh | ||
if: ${{ inputs.debug == 'true' }} | ||
- run: sudo sh -c "echo core > /proc/sys/kernel/core_pattern" | ||
shell: sh | ||
if: ${{ inputs.debug == 'true' }} | ||
- run: echo $(node --print "process.version") | ||
shell: sh | ||
- run: echo $(node --print "process.features.debug") | ||
shell: sh | ||
|
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
228d7ba
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Possible performance regression was detected for some benchmarks.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold.
🚀🚀 Significant benchmark improvement detected
Full benchmark results