Skip to content

Add volume lifecycle churn scale test#2880

Open
torredil wants to merge 1 commit intokubernetes-sigs:masterfrom
torredil:scale-test-58123
Open

Add volume lifecycle churn scale test#2880
torredil wants to merge 1 commit intokubernetes-sigs:masterfrom
torredil:scale-test-58123

Conversation

@torredil
Copy link
Member

What type of PR is this?

ebs-scale-test enhancement.

What is this PR about? / Why do we need it?

Our existing scale tests all follow a sequential pattern where volumes are created, attached, then torn down. That doesn't capture what happens with workloads like Spark pipelines, where volumes are constantly being created and destroyed in overlapping phases.

This PR adds a new test type that models that pattern. It runs waves of short lived jobs, each with its own PVC. When a wave's jobs complete, their resources are deleted and the next wave is immediately submitted so the driver is handling both teardown and provisioning concurrently, which is the behavior we want to benchmark.

Works with both pre-allocated and karpenter cluster types using the existing scale test framework.

How was this change tested?

export TEST_TYPE="volume-lifecycle-churn"
export REPLICAS=500
export WAVES=3

./scale-test setup
./scale-test run
./scale-test cleanup
make update && make verify && make test

Does this PR introduce a user-facing change?

NONE.

Signed-off-by: Eddie Torres <torredil@amazon.com>
@k8s-ci-robot k8s-ci-robot added the release-note-none Denotes a PR that doesn't merit a release note. label Feb 27, 2026
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign jsafrane for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Feb 27, 2026
@github-actions
Copy link

Code Coverage Diff

This PR does not change the code coverage

Copy link
Member

@ElijahQuinones ElijahQuinones left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants