Skip to content
This repository was archived by the owner on Feb 6, 2024. It is now read-only.

emojivoto: use custom metrics-merger jobs#44

Open
t-lo wants to merge 1 commit into
masterfrom
t-lo/emojivoto-use-correct-exported-jobs
Open

emojivoto: use custom metrics-merger jobs#44
t-lo wants to merge 1 commit into
masterfrom
t-lo/emojivoto-use-correct-exported-jobs

Conversation

@t-lo

@t-lo t-lo commented May 10, 2021

Copy link
Copy Markdown
Member

This change leverages the metrics-merger feature of setting custom merge
jobs in order to merge results from Intel, AMD, Graviton, and Ampere
CPUs.

The metrics-merger feature was introduced with
f438e26fcba7cfa49e68bf963d2f778d50c5a6d7.

This PR requires kinvolk/service-mesh-benchmark#72 (and a respectively updated metrics-merger container image).

This change leverages the metrics-merger feature of setting custom merge
jobs in order to merge results from Intel, AMD, Graviton, and Ampere
CPUs.

THe metrics-merger feature was introduced with
f438e26fcba7cfa49e68bf963d2f778d50c5a6d7.

Signed-off-by: Thilo Fromm <thilo@kinvolk.io>
@t-lo t-lo requested a review from iaguis May 10, 2021 12:03
@iaguis

iaguis commented May 10, 2021

Copy link
Copy Markdown
Contributor

The way this is set up now, I copied the metrics-merge Helm chart from github.com/kinvolk/service-mesh-benchmark to this repo: https://github.com/kinvolk/benchmark-containers/tree/master/emojivoto-benchmark/helm/metrics-merger

What I didn't copy is the Dockerfile and python script. I propose to include the Helm chart changes here too, I can push on top of your branch.

@iaguis iaguis removed their request for review September 7, 2022 08:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants