Passing user to getLatestDataTime call

[amitgalitz]

Description

Changed getLatestDataTime so we pass the correct user so we have the right user info when querying remote indices, otherwise we get a failure like:

nested: RemoteTransportException[[35692e6c255f][172.22.0.4:9300][indices:data/read/search]]; nested: OpenSearchSecurityException[No user found for indices:data/read/search];

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[codecov]

Codecov Report

Attention: Patch coverage is 70.58824% with 5 lines in your changes missing coverage. Please review.

Project coverage is 77.49%. Comparing base (3f0fc8c) to head (9f4cbb9).
Report is 12 commits behind head on main.

Files with missing lines	Patch %	Lines
...pensearch/timeseries/feature/SearchFeatureDao.java	63.63%	3 Missing and 1 partial ⚠️
...ies/rest/handler/ModelValidationActionHandler.java	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #1296      +/-   ##
============================================
+ Coverage     71.83%   77.49%   +5.66%     
- Complexity     4898     5443     +545     
============================================
  Files           518      533      +15     
  Lines         22879    23324     +445     
  Branches       2245     2311      +66     
============================================
+ Hits          16434    18075    +1641     
+ Misses         5410     4188    -1222     
- Partials       1035     1061      +26     

Flag	Coverage Δ
plugin	77.49% <70.58%> (+5.66%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
.../rest/handler/AbstractTimeSeriesActionHandler.java	95.45% <100.00%> (+30.64%)	⬆️
...h/timeseries/rest/handler/IntervalCalculation.java	93.20% <ø> (+93.20%)	⬆️
...h/timeseries/rest/handler/LatestTimeRetriever.java	87.83% <100.00%> (+87.83%)	⬆️
...earch/timeseries/util/CrossClusterConfigUtils.java	90.47% <ø> (ø)
...ies/rest/handler/ModelValidationActionHandler.java	74.26% <0.00%> (+74.26%)	⬆️
...pensearch/timeseries/feature/SearchFeatureDao.java	80.06% <63.63%> (-2.67%)	⬇️

... and 101 files with indirect coverage changes

[kaituo]

Can you add a test to reproduce the bug? If it is too time consuming, we can do it after cut off.

[amitgalitz]

Can you add a test to reproduce the bug? If it is too time consuming, we can do it after cut off.

It only occurs on remote index, not 100% sure on best way to have integration with remote clusters, might take me longer then cut off