DaveTeng0
DaveTeng0
@umamaheswararao @kerneltime Feel free to take a look if the added message helps! Thanks!
> LGTM, please address failing CI tests sure! looking
> cc @DaveTeng0 Thank you Ritesh! checking.
> Sure!! will resolve the conflict and update unit tests!
> Sure!! will take a look the failed tests!
cc. @kerneltime @jojochuang @umamaheswararao @duongkame
> Yes! The command will potentially read some key which doesn't exist in the cluster. Currently the freon command would stop running & report failure, but let me think more...
> > Yes! The command will potentially read some key which doesn't exist in the cluster. > > Currently the freon command would stop running & report failure, but let...
> > Code can continue testing and add a metric at the end for successful reads vs. not found read. It is a valid test to look at the performance...
> There are failures in CI, please make sure CI pass before merge @kerneltime. Sorry!! I'll take a look!!