-
Notifications
You must be signed in to change notification settings - Fork 31
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.
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
Adding in user email and comparison config for kube-burner #164
Adding in user email and comparison config for kube-burner #164
Conversation
/assign @qiliRedHat |
@paigerube14
OCP Version Flexy Id Scale Ci Job UUID/Grafana URL Status Params Pod Ready Average (ms) Pod Scheduled Avg (ms) Initialized (ms) Containers Ready (ms) Time/Date Snapshot (done manually) 4.11.0-0.nightly-2022-05-18-171831 104086 56 15a42470-aa6b-4f45-863f-3bc171d9156a PASS 1 2405 0 42 2405 https://docs.google.com/spreadsheets/d/1NHU-mD1MqZcZizeJkKvAhpvXVKpKb3ogMEj70qjbhcs *************** 2022-05-24 04:53:58.448785-05:00 |
@qiliRedHat those errors need to be fixed upstream I believe. I will run it locally with the same parameters and report back with the results |
66e3bf6
to
4e5e0d3
Compare
If we set the metrics profile to be metrics-profiles/metrics-ovn.yaml it will properly get the missing metrics and runs cleaner Error message before setting metrics-ovn.yaml file Successful run: https://mastern-jenkins-csb-openshift-qe.apps.ocp-c1.prod.psi.redhat.com/job/scale-ci/job/paige-e2e-multibranch/job/kube-burner-3/63/ @qiliRedHat what are your thoughts on setting the default metrics profile to metrics-ovn.yaml for our runs? The 2 files are the exact same except for the addition of podCPU and podMemory metrics are indexed using the ovn yaml file and not the metrics.yaml file |
Created PR to not have to specifically set the metrics-ovn.yaml file as the metrics profile so we can properly get the podCPU and podMemory metrics in our google sheet comparison file |
@paigerube14 My run's google sheet is shared to you https://docs.google.com/spreadsheets/d/1NHU-mD1MqZcZizeJkKvAhpvXVKpKb3ogMEj70qjbhcs/edit?usp=sharing. |
@qiliRedHat no I have not seen that. Looks like the data is in elastic search for that run with 0 as the values. I have seen that nothing gets printed to the sheet though. Will have to test more |
4e5e0d3
to
c2badd7
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: paigerube14, qiliRedHat The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
As part of PR we added the ability to get benchmark-comparison google sheet output from kube-burner workloads. Need to add setting of comparison_config, google sheet service account and email to send it to
This only works on jenkins agents that start with "oc4*" for now. Still trying to fix for rhel8 agents