cloud-bulldozer / cloud-bulldozer/benchmark-wrapper

Inconsistent Benchmark and Quay Image Names

Open
#347 10 comments 1 reaction 4 assignees Claimed by @mohit-sheth View on GitHub
bug CI
Dominant language
Python
Stars
19
Forks
56
PR merge metrics
No merged PRs in 30d

Description

Looking at any of the build results after #344 (i.e. https://github.com/cloud-bulldozer/benchmark-wrapper/actions/runs/1221646689), we have some inconsistent names between the wrappers in our code and the wrapper images up on quay. Additionally, some wrappers are missing image repositories altogether. Here are the inconsistencies that I found:

|snafu wrapper module name|snafu wrapper tool name (`--tool`)|quay image name|
|---|---|---|
|iperf|n/a|iperf3|
|ycsb|ycsb|ycsb-server|
|fs_drift|fs-drift|fs-drift|
|dns_perf|dns_perf|No repo on quay|
|upgrade_openshift|upgrade|No repo on quay|

I can add logic into the [build_matrix.py](https://github.com/cloud-bulldozer/benchmark-wrapper/blob/master/ci/build_matrix.py) script in order to take care of this, or we can do some refactoring of our image names. What do we think?

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.