kruize / autotune

Autonomous Performance Tuning for Kubernetes!
Apache License 2.0
153 stars 52 forks source link

Format the durationInhours value #1203

Closed khansaad closed 3 weeks ago

khansaad commented 1 month ago

Format the DurationInHours value to two decimal places

Description

This PR will do the following changes -

Type of change

How has this been tested?

Test Configuration

Checklist :dart:

Additional information

khansaad commented 3 weeks ago

Test results attached: https://ci.app-svc-perf.corp.redhat.com/job/ExternalTeams/job/Autotune/job/kruize_test_pull_request/104/

@chandrams Do we need to add new test for this ?

chandrams commented 3 weeks ago

@khansaad - No need of a new test, you can add a check in the validation logic for recommendations to check for expected precision for duration in hours.