From 031cd2a644f59f2aca8f97833a34bd49c3c40450 Mon Sep 17 00:00:00 2001 From: Alexey Zorkaltsev Date: Fri, 19 Jan 2024 12:24:09 +0300 Subject: [PATCH 1/7] chore: create SLO testing branch --- README.md | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/README.md b/README.md index 5fff5cf3..868a38d3 100644 --- a/README.md +++ b/README.md @@ -1,3 +1,8 @@ +# !!! DO NOT APPROVE + +This branch is purely for testing SLO by creating a PR to the main branch. That PR should NOT be merged. +There are no useful changes in it. + # Roadmap You can take a look at our Roadmap for the near future [here](ROADMAP.md). From 9093a61193fba191fd5cfb8a19ae4c15bfb2de98 Mon Sep 17 00:00:00 2001 From: Alexey Zorkaltsev Date: Fri, 19 Jan 2024 12:46:54 +0300 Subject: [PATCH 2/7] chore: switch to debug branch of slo-tests --- .github/workflows/slo.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/slo.yml b/.github/workflows/slo.yml index 01090552..bf9da747 100644 --- a/.github/workflows/slo.yml +++ b/.github/workflows/slo.yml @@ -7,7 +7,7 @@ name: SLO jobs: test-slo: if: github.event.pull_request.head.repo.full_name == 'ydb-platform/ydb-nodejs-sdk' - uses: ydb-platform/slo-tests/.github/workflows/slo.yml@main + uses: ydb-platform/slo-tests/.github/workflows/slo.yml@debug secrets: inherit with: language_id: nodejs From 8a60cef52a2ef4a3825405a2f74b7521c89e9d03 Mon Sep 17 00:00:00 2001 From: Alexey Zorkaltsev Date: Fri, 19 Jan 2024 13:08:12 +0300 Subject: [PATCH 3/7] chore: change 1 --- README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/README.md b/README.md index 868a38d3..327aebf6 100644 --- a/README.md +++ b/README.md @@ -6,6 +6,8 @@ There are no useful changes in it. # Roadmap You can take a look at our Roadmap for the near future [here](ROADMAP.md). +Dummy change 1 + # How to use ## Install package from NPM ```bash From de53a1ee7e2912f7bbe5eacfa548ee3aa7ada2d1 Mon Sep 17 00:00:00 2001 From: Alexey Zorkaltsev Date: Fri, 19 Jan 2024 14:08:38 +0300 Subject: [PATCH 4/7] chore: Dummy change 2 --- README.md | 1 + 1 file changed, 1 insertion(+) diff --git a/README.md b/README.md index 327aebf6..a4f669b2 100644 --- a/README.md +++ b/README.md @@ -7,6 +7,7 @@ There are no useful changes in it. You can take a look at our Roadmap for the near future [here](ROADMAP.md). Dummy change 1 +Dummy change 2 # How to use ## Install package from NPM From 7f2c66e8cad07fa684e0611fb477b146af1d28bd Mon Sep 17 00:00:00 2001 From: Alexey Zorkaltsev Date: Fri, 19 Jan 2024 14:16:42 +0300 Subject: [PATCH 5/7] Dummy change 3 --- README.md | 1 + 1 file changed, 1 insertion(+) diff --git a/README.md b/README.md index a4f669b2..e333039b 100644 --- a/README.md +++ b/README.md @@ -8,6 +8,7 @@ You can take a look at our Roadmap for the near future [here](ROADMAP.md). Dummy change 1 Dummy change 2 +Dummy change 3 # How to use ## Install package from NPM From b18ff02ad5447de3b0574ee12641adb8b5df93d6 Mon Sep 17 00:00:00 2001 From: Alexey Zorkaltsev Date: Fri, 19 Jan 2024 14:56:39 +0300 Subject: [PATCH 6/7] Dummy change 4 --- README.md | 1 + 1 file changed, 1 insertion(+) diff --git a/README.md b/README.md index e333039b..07279603 100644 --- a/README.md +++ b/README.md @@ -9,6 +9,7 @@ You can take a look at our Roadmap for the near future [here](ROADMAP.md). Dummy change 1 Dummy change 2 Dummy change 3 +Dummy change 4 # How to use ## Install package from NPM From 4942d062773e6206e25b8cfc07fa042ad15b740c Mon Sep 17 00:00:00 2001 From: Alexey Zorkaltsev Date: Fri, 19 Jan 2024 15:16:48 +0300 Subject: [PATCH 7/7] chore: SLO-workload-fix --- slo-workload/src/metricsJob.ts | 2 +- slo-workload/src/utils/RateLimiter.ts | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/slo-workload/src/metricsJob.ts b/slo-workload/src/metricsJob.ts index 9ed10a57..f956ac82 100644 --- a/slo-workload/src/metricsJob.ts +++ b/slo-workload/src/metricsJob.ts @@ -1,7 +1,7 @@ import Executor from './utils/Executor' export class MetricsJob { - private intervalId: NodeJS.Timer | undefined + private intervalId: NodeJS.Timeout | undefined private endTime: number private promise: Promise diff --git a/slo-workload/src/utils/RateLimiter.ts b/slo-workload/src/utils/RateLimiter.ts index c00a897f..c5a9cdb0 100644 --- a/slo-workload/src/utils/RateLimiter.ts +++ b/slo-workload/src/utils/RateLimiter.ts @@ -3,7 +3,7 @@ import NanoTimer from 'nanotimer' export default class RateLimiter { private delayMicroSec: number private count: number = 0 - private realRPSObserverId: NodeJS.Timer + private realRPSObserverId: NodeJS.Timeout private returnerTimer: NanoTimer private real: number = 0 private returner = () => {}