Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Page refresh on Dashboard with saved time loses time range #20827

Closed
stacey-gammon opened this issue Jul 16, 2018 · 5 comments
Closed

Page refresh on Dashboard with saved time loses time range #20827

stacey-gammon opened this issue Jul 16, 2018 · 5 comments
Assignees
Labels
blocker bug Fixes for quality problems that affect the customer experience Feature:Dashboard Dashboard related features v6.4.0

Comments

@stacey-gammon
Copy link
Contributor

stacey-gammon commented Jul 16, 2018

On master. To repro:

  1. Create a dashboard with time range saved.
    screen shot 2018-07-16 at 11 09 56 am

  2. Refresh the page.
    See it go to "last 15 minutes"
    screen shot 2018-07-16 at 11 10 12 am

There are quite a few time range dashboard bugs open so it could be related:

@stacey-gammon stacey-gammon added bug Fixes for quality problems that affect the customer experience blocker Feature:Dashboard Dashboard related features :Sharing labels Jul 16, 2018
@timroes timroes added the v6.4.0 label Jul 16, 2018
@stacey-gammon
Copy link
Contributor Author

stacey-gammon commented Jul 16, 2018

As of June 21st, this was working.
As of July 5th, this was not working. I suspect #19852 may be the culprit. Will continue digging.
As of commit dbdbb68 - the PR right after the above, it's broken. Looks more and more likely the above PR is the culprit.

Looks like the time portion of the URL is missing.

@stacey-gammon stacey-gammon changed the title Page refresh on dashboard losing time range Page refresh on Dashboard/Visualize/Discover loses time range Jul 16, 2018
@stacey-gammon stacey-gammon changed the title Page refresh on Dashboard/Visualize/Discover loses time range Page refresh on Dashboard with saved time loses time range Jul 16, 2018
@stacey-gammon stacey-gammon self-assigned this Jul 16, 2018
@tylersmalley
Copy link
Contributor

Checking in on the status of this since it's a blocker for 6.4

@stacey-gammon
Copy link
Contributor Author

I have a PR that I think fixes it, but I did not get it over the line when I went out on vacation last week. There are failing tests on it, but that's the norm nowadays so now I need to wade through whether they are legit failures or flaky ones (last week there were some legit ones which I thought I fixed right before I left, now I need to go back to verify as the tests are still failing).

@walterra
Copy link
Contributor

@stacey-gammon I'm afraid the PR #20858 caused a regression in the ML plugin documented here #21326

I was able to pin down the regression to be caused by this change: https://github.com/elastic/kibana/pull/20858/files#diff-82d343a079857a203816647e3c33f54eR73

I'm currently investigating how we can fix it on the ML side without any further changes to kbn_global_timepicker.js.

@stacey-gammon
Copy link
Contributor Author

Oh no! Was the ML code that was affected introduced within the last month? The code that originally caused the bug was introduced on June 26th. If your code was written after that, maybe it was written with incorrect assumptions? If the code that broke has not been touched in awhile then perhaps I incorrectly changed something from how it worked prior to June 26th.

I'm technically off today, but let me know if I can help when I get back on Monday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocker bug Fixes for quality problems that affect the customer experience Feature:Dashboard Dashboard related features v6.4.0
Projects
None yet
Development

No branches or pull requests

4 participants