Skip to content
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.

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

Script to run a Porch load test #141

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

liamfallon
Copy link
Member

@liamfallon liamfallon commented Nov 20, 2024

This PR adds a script to run a load test on Porch.

The PR on the docs repo that documents the script is here: nephio-project/docs#185

Copy link
Contributor

nephio-prow bot commented Nov 20, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: liamfallon

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

The pull request process is described here

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

@nephio-prow nephio-prow bot added the approved label Nov 20, 2024
@liamfallon
Copy link
Member Author

/assign @kushnaidu

@liamfallon
Copy link
Member Author

/assign @radoslawc

@liamfallon
Copy link
Member Author

Screenshot 2024-11-20 at 16 01 10

Plot for the load test on 20 repos, with 20 packages, each package has 5 PRs, so a total of 2000 PRs. The left part is the time for init/copy to approval, the right part is the deletion-propose/deletion time.

Worst time is 22s, you can really see the effect of the 10 minute polling that refreshes the repos on the times.

scripts/run-load-test.sh Outdated Show resolved Hide resolved
scripts/run-load-test.sh Outdated Show resolved Hide resolved
scripts/run-load-test.sh Outdated Show resolved Hide resolved
scripts/run-load-test.sh Outdated Show resolved Hide resolved
scripts/run-load-test.sh Show resolved Hide resolved
scripts/run-load-test.sh Outdated Show resolved Hide resolved
scripts/run-load-test.sh Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants