Kibana: Refresh interval for dashboards doesn't refresh data

[HenrikOssipoff]

Kibana version:
7.6.0

Elasticsearch version:
7.6.0

Server OS version:
Ubuntu 18.04.4 LTS (GNU/Linux 5.0.0-1029-gcp x86_64)

Browser version:
Chrome 79.0.3945.130

Browser OS version:
Mac OS Catalina 10.15.3

Original install method (e.g. download page, yum, from source, etc.):
deb packages

Describe the bug:
Since installing 7.6.0 (upgraded from 7.4.0) the "refresh every" interval for dashboards doesn't fetch new data. I've checked the console, and no requests are periodically made from the browser. Manually clicking the refresh button fetches new data.
The refresh interval seems to be working in every other part of Kibana for us.

Steps to reproduce:

1. Open a dashboard
2. Set the refresh interval (for example 5 seconds)
3. Observe that no requests are ever sent to the server to refresh data

Expected behavior:
Expected that data would be refreshed at the set interval.

[elasticmachine]

Pinging @elastic/kibana-app-arch (Team:AppArch)

[timroes]

@LeeDr @ppisljar This seems like this should be a blocker for 7.6.1?

[flash1293]

This affects the visualization and lens embeddable, but not saved searches and maps.

The dashboard container is passing down the refresh config to the embeddables, but visualizations don't act on it.

Saved searches are also not using the passed down refresh config, but subscribe to data.timefilter.getAutoRefreshFetch$() instead (which is in practice always identical to the value of the embeddable input): https://github.com/elastic/kibana/blob/master/src/legacy/core_plugins/kibana/public/discover/np_ready/embeddable/search_embeddable.ts#L132

Maps do use the passed down refresh config and are creating their own refresh source: https://github.com/elastic/kibana/blob/master/x-pack/legacy/plugins/maps/public/connected_components/gis_map/view.js#L123

It seems like the way maps is doing it is the best one because it allows embeddable containers to use a refresh config different than the global one.

So my suggestion for fixing this:

• Make saved search embeddable use the refresh config passed down via embeddable input (like maps)
• Add the same logic to the visualization embeddable

@timroes @ppisljar what do you think?

Edit: Did some more digging and I think the cause was that this specific functionality was not moved from visualize loader to the expression service variant: https://github.com/elastic/kibana/pull/46910

[ppisljar]

i think we should implement this on the base embeddable class for all embeddables.
dashboard already passes in refreshInterval to every embeddable on the input. Base embeddable should use that parameter to periodically call reload()

[arjun-kankanala]

Is there any tentative release date for v7.6.1?

[bevano8]

Is there any tentative release date for v7.6.1?

Any updates on this?

[timroes]

We are planning to release 7.6.1, which will contain this sometime over the upcoming week, if everything goes as planed.

[patrickmhoge]

Hi, I've also noticed in 7.6.0 in the Infra/Metrics application that when Auto-Refresh is enabled, the stats in the waffle chart seem to oscillate between 0 and the correct value. Just wondering if this is related to the above regression or if this is something different. Let me know if you want any more info.

[flash1293]

Thank you very much for reporting @patrickmhoge, this sounds like a separate problem. Can you kindly open a separate issue for this?

[patrickmhoge]

Done! -> https://github.com/elastic/kibana/issues/58473