-
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
Temporarily disable scripted field filter tests #15169
Conversation
jenkins, test this |
Does the functional test runner support |
@tylersmalley it does |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Commenting out 4 tests that are failing due to elastic/elasticsearch#26760 so we can bump ESVM snapshots. We'll need to revert this PR once elastic/elasticsearch#26760 is fixed.
Commenting out 4 tests that are failing due to elastic/elasticsearch#26760 so we can bump ESVM snapshots. We'll need to revert this PR once elastic/elasticsearch#26760 is fixed.
Commenting out 4 tests that are failing due to elastic/elasticsearch#26760 so we can bump ESVM snapshots. We'll need to revert this PR once elastic/elasticsearch#26760 is fixed.
Commenting out 4 tests that are failing due to elastic/elasticsearch#26760 so we can bump ESVM snapshots. We'll need to revert this PR once elastic/elasticsearch#26760 is fixed.
@Bargs Can you create a blocker issue for reenabling these tests? We don't want to ship 6.1 without the upstream issue being fixed in ES and CI running successfully in Kibana. |
We already have a blocker for this issue, I've added a note in bold at the top to remind us that we need to reenable these tests before it can be closed. #14159 |
This reverts commit 2911418.
Commenting out 4 tests that are failing due to elastic/elasticsearch#26760 so we can bump ESVM snapshots. We'll need to revert this PR once elastic/elasticsearch#26760 is fixed.