Skip to content

Commit 205bafe

Browse files
fix: re-enable linux perf listener for walltime
It was removed as a test and mistakenly not restored.
1 parent 36834e8 commit 205bafe

2 files changed

Lines changed: 2 additions & 8 deletions

File tree

.github/workflows/codspeed.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,8 +86,6 @@ jobs:
8686
CODSPEED_WALLTIME_PROFILER: samply
8787
with:
8888
mode: walltime
89-
# TODO: Remove this once runner is released
90-
runner-version: v4.17.7-alpha.2
9189
run: |
9290
pnpm turbo run bench --filter=@codspeed/tinybench-plugin
9391
pnpm turbo run bench --filter=@codspeed/vitest-plugin

packages/core/src/index.ts

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -57,9 +57,7 @@ export const setupCore = () => {
5757

5858
native_core.InstrumentHooks.setIntegration("codspeed-node", __VERSION__);
5959
// In walltime, we use node's native profiling options rather than our own, cf `getV8Flags`
60-
if (getCodspeedRunnerMode() !== "walltime") {
61-
linuxPerf.start();
62-
}
60+
linuxPerf.start();
6361
checkV8Flags();
6462

6563
// Collect Node.js runtime environment to detect changes that could
@@ -71,9 +69,7 @@ export const setupCore = () => {
7169
};
7270

7371
export const teardownCore = () => {
74-
if (getCodspeedRunnerMode() !== "walltime") {
75-
linuxPerf.stop();
76-
}
72+
linuxPerf.stop();
7773
};
7874

7975
export type {

0 commit comments

Comments
 (0)