Skip to content

[testing] Add local kube support for load tests #3986

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

Merged
merged 4 commits into from
Jun 4, 2025

Conversation

RodrigoVillar
Copy link
Contributor

@RodrigoVillar RodrigoVillar commented Jun 2, 2025

Why this should be merged

This PR extends the functionality of the load tests by allowing it to run against a deployed Kube network.

How this works

./scripts/run_task.sh test-load-kube

How this was tested

Ran the load test locally; the load dashboard when running against a deployed Kube network is being populated as well.

Need to be documented in RELEASES.md?

N/A

@RodrigoVillar RodrigoVillar self-assigned this Jun 2, 2025
@RodrigoVillar RodrigoVillar marked this pull request as ready for review June 2, 2025 20:30
@Copilot Copilot AI review requested due to automatic review settings June 2, 2025 20:30
@RodrigoVillar RodrigoVillar requested a review from maru-ava as a code owner June 2, 2025 20:30
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds Kubernetes (Kind) support for running load tests by introducing a helper script and Taskfile task.

  • Introduce scripts/tests.load.kube.sh to provision a Kind cluster, build the Docker image, and invoke the Go load test with --runtime=kube.
  • Extend Taskfile.yml with a test-load-kube task to wire up the new script under the Taskfile workflow.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
scripts/tests.load.kube.sh New script to start a Kind cluster and run load tests
Taskfile.yml Added test-load-kube task for executing the new script
Comments suppressed due to low confidence (1)

Taskfile.yml:237

  • [nitpick] Consider using the full term "Kubernetes" instead of the abbreviation "kube" for clarity in the description.
    desc: Runs load tests against a network deployed to kube

@RodrigoVillar RodrigoVillar requested a review from maru-ava June 3, 2025 13:16
@RodrigoVillar RodrigoVillar requested a review from Elvis339 June 3, 2025 14:03
@maru-ava maru-ava added the testing This primarily focuses on testing label Jun 4, 2025
@maru-ava maru-ava moved this to Ready 🚦 in avalanchego Jun 4, 2025
@maru-ava maru-ava added this pull request to the merge queue Jun 4, 2025
Merged via the queue into master with commit 4b2e034 Jun 4, 2025
28 checks passed
@maru-ava maru-ava deleted the load-test-local-kube branch June 4, 2025 16:24
@github-project-automation github-project-automation bot moved this from Ready 🚦 to Done 🎉 in avalanchego Jun 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing This primarily focuses on testing
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants