-
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
[7.7] Fix sorting of scripted string fields (#72681) #72739
Conversation
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💔 Build Failed
Failed CI StepsTest FailuresKibana Pipeline / kibana-oss-agent / Chrome UI Functional Tests.test/functional/apps/dashboard/dashboard_filter_bar·js.dashboard app using current data dashboard filter bar filter editor field list shows index pattern of vis when one is addedStandard Out
Stack Trace
Kibana Pipeline / kibana-oss-agent / Chrome UI Functional Tests.test/functional/apps/dashboard/dashboard_filter_bar·js.dashboard app using current data dashboard filter bar filter editor field list shows index pattern of vis when one is addedStandard Out
Stack Trace
Kibana Pipeline / kibana-xpack-agent / Chrome X-Pack UI Functional Tests.x-pack/test/functional/apps/graph/graph·ts.graph app graph should show correct node labelsStandard Out
Stack Trace
and 1 more failures, only showing the first 3. History
To update your PR or re-run it, just comment with: |
@elastic/kibana-operations I am trying to merge this backport pr. It looks like these tests are failing in the current Any suggestions to merge this pr? |
Why backport this to 7.7? Once 7.8 is released that's where bugfixes should go |
We don't run the tests on 7.7 on any regular basis so I'm not surprised that the tests are now failing. The reason we don't run the tests on 7.7 though is because once 7.8 is released there are only very special circumstances where we would expect a new 7.7 version to be released. |
Oh got it, thanks @spalger. Is there usually only one patch version per minor? Or does that just depend on the circumstances? |
Definitely depends on the circumstances 👍 |
Right on thanks! |
Backports the following commits to 7.7: