Skip to content

CLDSRV-829: Increase KMS load test tolerance from 10% to 20% #6054

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

Merged
bert-e merged 1 commit into from
Jan 20, 2026
Merged

CLDSRV-829: Increase KMS load test tolerance from 10% to 20% #6054

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 4 additions & 3 deletions tests/functional/sse-kms-migration/load.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,11 @@ const KMS_NODES = helpers.config.kmip.transport.length;
const TOTAL_OBJECTS_PER_NODE = Math.floor(TOTAL_OBJECTS / KMS_NODES);

/**
* 10% approximation for the number of packets per IP
* 20% approximation for the number of packets per IP
* As we might not have an exact match of packets and the
* round robin is confined to each nodejs cluster processes
*/
const APPROX = Math.floor(0.1 * TOTAL_OBJECTS_PER_NODE);
const APPROX = Math.floor(0.2 * TOTAL_OBJECTS_PER_NODE);
const EXPECTED_MIN = TOTAL_OBJECTS_PER_NODE - APPROX;
const EXPECTED_MAX = TOTAL_OBJECTS_PER_NODE + APPROX;

Expand Down Expand Up @@ -178,7 +178,8 @@ describe(`KMS load (kmip cluster ${KMS_NODES} nodes): ${OBJECT_NUMBER
assert(repartition.length === KMS_NODES, `Expected ${KMS_NODES} IPs but got ${repartition.length}`);
assert(repartitionCount.every(count =>
count >= EXPECTED_MIN && count <= EXPECTED_MAX),
`Repartition counts should be around ${TOTAL_OBJECTS_PER_NODE} but got ${repartitionCount}`);
`Repartition counts should be around ${TOTAL_OBJECTS_PER_NODE} ` +
`(±${APPROX}, min: ${EXPECTED_MIN}, max: ${EXPECTED_MAX}) but got ${repartitionCount}`);
}

it(`should encrypt ${TOTAL_OBJECTS} times in parallel, ~${TOTAL_OBJECTS_PER_NODE} per node`, async () => {
Expand Down
Loading