Skip to content

Fix benchmark names - #398

Draft
chentex wants to merge 3 commits into
cloud-bulldozer:masterfrom
chentex:fix-benchmark-names
Draft

Fix benchmark names#398
chentex wants to merge 3 commits into
cloud-bulldozer:masterfrom
chentex:fix-benchmark-names

Conversation

@chentex

Copy link
Copy Markdown
Member

Description

Inconsistencies on naming have been having negative effects on our automation on image building.

  • Rename fs-drift tool to fs_drift
  • Rename upgrade tool to upgrade_openshift
  • Rename scale tool to scale_openshift

Fixes

#347

Signed-off-by: Vicente Zepeda Mas <vzepedam@redhat.com>
@chentexchentex added the ok to test Kick off our CI framework label Nov 29, 2021
Fixing systemd analyze image for arm64
Signed-off-by: Vicente Zepeda Mas <vzepedam@redhat.com>
@sjug

sjug commented Nov 29, 2021

Copy link
Copy Markdown
Collaborator

Don't you need to change ycsb-server as well?

@chentex

Copy link
Copy Markdown
MemberAuthor

Don't you need to change ycsb-server as well?

Good catch!

Signed-off-by: Vicente Zepeda Mas <vzepedam@redhat.com>
@codecov-commenter

codecov-commenter commented Nov 29, 2021

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 39.35%. Comparing base (b9ddc34) to head (b411745).
⚠️ Report is 49 commits behind head on master.

Additional details and impacted files
@@ Coverage Diff @@## master #398 +/- ##
=======================================
Coverage 39.35% 39.35% =======================================
Files 15 15 Lines 869 869 =======================================
Hits 342 342 Misses 527 527 
FlagCoverage Δ
gha39.35% <ø> (ø)
python-3.639.35% <ø> (ø)
unit39.35% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@chentex
chentex marked this pull request as draft November 29, 2021 16:31
@comet-perf-ci

Copy link
Copy Markdown
Collaborator

Results for SNAFU CI Test

TestResultRuntime
snafu/hammerdbPASS00:06:04
snafu/smallfile_wrapperPASS00:05:26
snafu/flent_wrapperPASS00:07:29
snafu/scale_openshift_wrapperFAIL00:03:10
snafu/sysbenchPASS00:02:52
snafu/benchmarks/uperfPASS00:21:53
snafu/image_pull_wrapperPASS00:03:28
snafu/ycsb_wrapperFAIL00:13:02
snafu/stressng_wrapperPASS00:03:36
snafu/fio_wrapperFAIL00:31:47
snafu/log_generator_wrapperFAIL00:01:54
snafu/upgrade_openshift_wrapperPASS00:00:00
snafu/iperfFAIL00:00:27
snafu/vegeta_wrapperFAIL00:01:43
snafu/fs_drift_wrapperFAIL00:01:47
snafu/pgbench_wrapperFAIL00:02:24

@learnitalllearnitall left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great! Can't find anything missing, but I'll try to do a more thorough double check with grep tomorrow.

@@ -1,11 +1,12 @@
FROM registry.access.redhat.com/ubi8:latest

RUN dnf install -y --nodocs git python3-pip && dnf clean all

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Was this change intentional here?

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

Labels

ok to testKick off our CI framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@chentex@sjug@codecov-commenter@comet-perf-ci@learnitall