cloud-bulldozer / benchmark-wrapper

Python Library to run benchmarks
https://benchmark-wrapper.readthedocs.io
Apache License 2.0
19 stars 56 forks source link

Uperf workload to support nodeport or loadbalancer type svc #402

Closed mukrishn closed 2 years ago

mukrishn commented 2 years ago

Depends-On: 721

Description

To support different service types for uperf workload NodePort:

      serviceip: true
      servicetype: "nodeport"

Loadbalancer: only tested for metallb loadbalancers on baremetal.

      serviceip: true
      servicetype: "metallb"
      metallb:
        addresspool: "addresspool-l3"
        service_etp: "Cluster"

Dependencies: https://github.com/cloud-bulldozer/benchmark-operator/pull/721

Fixes

codecov-commenter commented 2 years ago

Codecov Report

Merging #402 (58a9f18) into master (b8ccd6e) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #402   +/-   ##
=======================================
  Coverage   39.35%   39.35%           
=======================================
  Files          15       15           
  Lines         869      869           
=======================================
  Hits          342      342           
  Misses        527      527           
Flag Coverage Δ
gha 39.35% <ø> (ø)
python-3.6 39.35% <ø> (ø)
unit 39.35% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update b8ccd6e...58a9f18. Read the comment docs.

rsevilla87 commented 2 years ago

/rerun all snafu/benchmarks/uperf

comet-perf-ci commented 2 years ago

Results for SNAFU CI Test

Test Result Runtime
snafu/benchmarks/uperf PASS 00:20:01
rsevilla87 commented 2 years ago

@mohit-sheth, last thought before merging? We should do it along with https://github.com/cloud-bulldozer/benchmark-operator/pull/721