Add a new runkperf bench for listing configmaps #164

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

Jump to bottom

Merged

fuweid merged 18 commits into Azure:main from xinWeiWei24:runkperf_configmaps

May 27, 2025

Contributor

xinWeiWei24 commented May 5, 2025

Add a new runkperf bench for listing configmaps:

Add total-time parameter to specify the duration for the runner execution;
Add list_configmaps benchmark command to execute benchmarks on specified resources

xinWeiWei24 added 2 commits

May 5, 2025 11:38


          kperf: runner: Add --total-time flag to specify the duration for the …

d7d7ef1

…runner execution


          runkper/commands/bench: Add list_configmaps benchmark command to exec…

f7f4306

…ute benchmarks on specified resources

xinWeiWei24 requested a review from fuweid

May 5, 2025 04:39


          fix linter error

48956b2

wonderyl reviewed

View reviewed changes

api/types/load_traffic.go Outdated Show resolved Hide resolved

wonderyl reviewed

View reviewed changes

cmd/kperf/commands/runner/runner.go Outdated Show resolved Hide resolved

wonderyl reviewed

View reviewed changes

cmd/kperf/commands/runner/runner.go Outdated Show resolved Hide resolved

wonderyl reviewed

View reviewed changes

cmd/kperf/commands/runner/runner.go Outdated Show resolved Hide resolved

wonderyl reviewed

View reviewed changes

cmd/kperf/commands/runner/runner.go Outdated Show resolved Hide resolved

wonderyl reviewed

View reviewed changes

contrib/cmd/runkperf/commands/bench/list_configmaps.go Outdated Show resolved Hide resolved

wonderyl reviewed

View reviewed changes

contrib/cmd/runkperf/commands/bench/list_configmaps.go Show resolved Hide resolved

wonderyl reviewed

View reviewed changes

contrib/cmd/runkperf/commands/bench/list_configmaps.go Outdated Show resolved Hide resolved

wonderyl reviewed

View reviewed changes

request/random.go Outdated Show resolved Hide resolved

wonderyl reviewed

View reviewed changes

contrib/cmd/runkperf/commands/bench/utils.go Outdated Show resolved Hide resolved

wonderyl reviewed

View reviewed changes

contrib/cmd/runkperf/commands/bench/list_configmaps.go Outdated Show resolved Hide resolved

wonderyl reviewed

View reviewed changes

contrib/internal/manifests/loadprofile/list_configmaps.yaml Outdated Show resolved Hide resolved

wonderyl reviewed

View reviewed changes

contrib/internal/manifests/loadprofile/list_configmaps.yaml Outdated Show resolved Hide resolved

xinWeiWei24 added 9 commits

May 5, 2025 20:31


          Fix wrong comment

0a1be89


          Replace benchConfigmapsCase with benchListConfigmapsCase

e547f49


          Replace RunForDuration with RunForDuration

d79b42f


          Note the unit of size in usage

db990a0


          list_configmaps: make usage succincter

c7e5e97


          list_configmaps: Add group label in selector of requests

8cd9139


          list_configmaps: Just use staleList and quorumList with same shares

764d99e


          list_configmaps: Set profileCfg.Spec.Total if both Total and Duration…

… are not set.


          list_configmaps: Set default value of total flag

729e988

fuweid reviewed

View reviewed changes

api/types/load_traffic.go Outdated Show resolved Hide resolved

cmd/kperf/commands/runner/runner.go Show resolved Hide resolved

request/random.go Outdated Show resolved Hide resolved

cmd/kperf/commands/runner/runner.go Outdated Show resolved Hide resolved

contrib/cmd/runkperf/commands/bench/list_configmaps.go Outdated Show resolved Hide resolved

xinWeiWei24 and others added 3 commits

May 21, 2025 11:28


          Update contrib/cmd/runkperf/commands/bench/list_configmaps.go

6989d12

Co-authored-by: Fu Wei <fuweid89@gmail.com>


          Update cmd/kperf/commands/runner/runner.go

608975a

Co-authored-by: Fu Wei <fuweid89@gmail.com>


          Update api/types/load_traffic.go

61c2d52

Co-authored-by: Fu Wei <fuweid89@gmail.com>


          runkperf/list_configmaps: Use timeout context instead of new function

35c8698

fuweid approved these changes

View reviewed changes

request/random.go Outdated Show resolved Hide resolved

cmd/kperf/commands/runner/runner.go Outdated Show resolved Hide resolved

xinWeiWei24 added 2 commits

May 22, 2025 14:46


          kperf/runner: Use klog to print warning message

ab86e2d


          request/random: Delete useless function

ae1da5f

fuweid merged commit 1ef44ca into Azure:main

4 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet