-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[6.3][CI Failure] dashboard app dashboard queries Pie chart attached to saved search filters data as expected #19810
Comments
@tylersmalley, yes #19688 seems to have exposed this problem the test needs to wait for the page to be loaded. |
This should fix it, I can do another PR for it, if someone has not already. |
Looks like the test has been rewritten in master, so this only impacts 6.3 |
Yes that'd be great if you could pull together a PR @liza-mae. Sorry if there was any confusion on the original PR. Going forward, even if you have changes that don't directly cause a failing test, but indirectly cause it to appear consistently (e.g. maybe that PR introduced a tiny timing change), we should really check in the fix as part of the original PR, or as a separate PR, but get it in there first. Otherwise the constant build failures can hold developers up (though luckily this is only 6.3 so it wasn't a big problem). Apologies if you thought I was on top of that change, I just didn't have time. I very much appreciate you figuring out the issue and how to fix it!! |
@stacey-gammon yes I will pull the PR for the fix. Sorry I just started running these tests so I am still learning best practices and such. I agree it makes sense to have just put the fix into the original PR, even though it appears to be an indirect failure. I needed my changes merged to test cloud on 6.3 for BC11 today. Will get to this fix shortly, thanks again! |
Merged #19828, closing. |
Consecutive failures.
https://kibana-ci.elastic.co/job/elastic+kibana+6.3+multijob-selenium/177/console
https://kibana-ci.elastic.co/job/elastic+kibana+6.3+multijob-selenium/178/console
https://kibana-ci.elastic.co/job/elastic+kibana+6.3+multijob-selenium/179/console
@elastic/kibana-sharing
The text was updated successfully, but these errors were encountered: