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

add support for gateway-api #72

Merged
merged 6 commits into from
Nov 8, 2024

Conversation

mohit-sheth
Copy link
Contributor

@mohit-sheth mohit-sheth commented Oct 22, 2024

Type of change

  • Refactor
  • New feature
  • Bug fix
  • Optimization
  • Documentation Update

Description

Adds support for Gateway API testing

Creates

  • GatewayClass
  • Gateway
  • HTTPRoute
  • and the usual client/server pods and service

Run with
./bin/ingress-perf run --cfg cfg.yaml --loglevel=debug --uuid $(uuidgen) --gateway-api=true

Check that it targets the httproute

time="2024-10-22 12:00:34" level=info msg="Running sample 1/1: 10s" file="exec.go:101"
time="2024-10-22 12:00:34" level=debug msg="Running [wrk -s json.lua -c 200 -d 10 --latency https://nginx.gwapi.$DOMAIN/1024.html --timeout 10] in client pods" file="exec.go:111"

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

@rsevilla87 rsevilla87 added the enhancement New feature or request label Oct 24, 2024
pkg/runner/exec.go Outdated Show resolved Hide resolved
return err
}
log.Debugf("Waiting 4 minutes for Gateway to be ready...")
time.Sleep(4 * time.Minute)
Copy link
Member

Choose a reason for hiding this comment

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

Why so long?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it takes ~3mins for the gateway to be up

pkg/runner/types.go Outdated Show resolved Hide resolved
pkg/runner/types.go Outdated Show resolved Hide resolved
pkg/runner/runner.go Outdated Show resolved Hide resolved
Signed-off-by: Raul Sevilla <[email protected]>
@rsevilla87 rsevilla87 merged commit 1fe256c into cloud-bulldozer:main Nov 8, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants