kruize / autotune

Autonomous Performance Tuning for Kubernetes!

Apache License 2.0

155 stars 53 forks source link

box plots data scripts #1108

Closed msvinaykumar closed 3 months ago

msvinaykumar commented 6 months ago

Add tests for box plots

Description

This PR will do the following changes -

add new tests for the box plots feature following the scenarios mentioned in #1116

Type of change

[ ] Refactoring Code
[ ] New feature
[x] Test changes

How has this been tested?

Using kruize-demo scripts. Image: quay.io/khansaad/autotune_operator:boxplots_fixes
Manually verified the plots data.

Test Configuration

Kubernetes clusters tested on: minikube

Checklist :dart:

[X] Followed coding guidelines
[X] Comments added
[ ] Dependent changes merged
[ ] Documentation updated
[x] Tests added or updated

Additional information

khansaad commented 4 months ago

PR check is failing because of the newly added box plot validations test. This should pass once PR #1189 is merged.

Also, have added the remaining test for validation. Running the regression on jenkins: https://ci.app-svc-perf.corp.redhat.com/job/ExternalTeams/job/Autotune/job/kruize_test_pull_request/96/

cc: @msvinaykumar @chandrams

khansaad commented 4 months ago

PR check is failing because of the newly added box plot validations test. This should pass once PR #1189 is merged.

Also, have added the remaining test for validation. Running the regression on jenkins: https://ci.app-svc-perf.corp.redhat.com/job/ExternalTeams/job/Autotune/job/kruize_test_pull_request/96/

cc: @msvinaykumar @chandrams

This one ran successfully. Re-running the whole suite after schema updates: https://ci.app-svc-perf.corp.redhat.com/job/ExternalTeams/job/Autotune/job/kruize_test_pull_request/97/

dinogun commented 3 months ago

@khansaad Please fix the conflicts