Skip to content

[8.6] [Fix][Infrastructure UI] Incorrect payload in time range when landing on the Hosts View (#147390)#147424

Merged
kibanamachine merged 1 commit intoelastic:8.6from
kibanamachine:backport/8.6/pr-147390
Dec 13, 2022
Merged

[8.6] [Fix][Infrastructure UI] Incorrect payload in time range when landing on the Hosts View (#147390)#147424
kibanamachine merged 1 commit intoelastic:8.6from
kibanamachine:backport/8.6/pr-147390

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.6:

Questions ?

Please refer to the Backport tool documentation

… on the Hosts View (elastic#147390)

Closes elastic#146581

## Summary

This PR fixes the initial `from` date range calculation. The idea is to
convert first the initial range of minutes to milliseconds. Then in
order to get the date of `CALCULATED_DATE_RANGE_TO` - (the calculated
initial range in milliseconds) and call `getTime()` to receive the
calculated `from` date as timestamp.

To test that you can open the host page without the time range in the
URL. Then check the `from` value as in the screenshot attached to the
story and you can use an [Unix timestamp
converter](https://www.unixtimestamp.com/) to verify that the
calculation is correct

(cherry picked from commit 29841da)
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
infra 1.1MB 1.1MB -12.0B
Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 59 65 +6
osquery 108 113 +5
securitySolution 441 447 +6
total +19

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 68 74 +6
osquery 109 115 +6
securitySolution 518 524 +6
total +20

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @jennypavlova

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants