Sridhar Seshasayee

Results 45 comments of Sridhar Seshasayee

**Teuthology Test Result** http://pulpito.front.sepia.ceph.com/?branch=wip-yuri5-testing-2022-08-18-0812 @ifed01 Across both the runs from the link above, the following failed jobIDs appear to be related to this PR. - [6981593](http://pulpito.front.sepia.ceph.com/lflores-2022-08-19_21:39:29-rados-wip-yuri5-testing-2022-08-18-0812-distro-default-smithi/6981593) - [6981597](http://pulpito.front.sepia.ceph.com/lflores-2022-08-19_21:39:29-rados-wip-yuri5-testing-2022-08-18-0812-distro-default-smithi/6981597) - [6981601](http://pulpito.front.sepia.ceph.com/lflores-2022-08-19_21:39:29-rados-wip-yuri5-testing-2022-08-18-0812-distro-default-smithi/6981601)...

@amathuria The commit sequence can be improved a bit as I found a couple of things confusing with respect to passing cost parameter to the various methods. Here's my suggestion...

> fio-plot-stats appears to have been broken by updates to numpy, the versions of numpy and pandas are: matplotlib is 3.4.3-4.el9 numpy is 1.26.4 pandas is 2.2.1 > > Sridhar...

@neha-ojha Can you please approve this? @lee-j-sanders ran the tool with the latest changes and it looks good. This is a low risk PR since it's a standalone tool. Thanks!

@neha-ojha Please see the sample charts generated using the tool below. ### Sample Charts 1. Percentile latency comparison between test runs ![sample_pct_cmp](https://github.com/ceph/cbt/assets/36185324/b3b974c3-e51f-4f66-89f8-0cf8d6c0de61) 2. Completion latency comparison between test runs ![sample_lat_cmp](https://github.com/ceph/cbt/assets/36185324/eafebf3c-2a96-4e31-a9ed-95d1f67604e5)...

@perezjosibm I have started taking a look at this. Please allow me some time to understand the changes. I plan to complete the review by the end of this week...

> I have added my first set of review comments. Will continue the review. Hmm...looks like my initial comments were lost. I will add them again. Sorry about that.

**Teuthology Test Result** https://pulpito.ceph.com/sseshasa-2024-07-02_16:09:36-perf-basic-squid-release-distro-default-smithi/ All perf-basic tests passed.