-
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
field_capabilites
handler swallows an upgrade-related exception from ES
#11379
Comments
field_capabilites
handler swallows an upgrade-related exception from ES
Related to #9571 (which is why you're seeing weird errors in Discover). At least one user near the end of that comment thread seems to be running into this issue. It looks like you're still seeing the geo_field marked as searchable and aggregatable because it's hardcoded to workaround elastic/elasticsearch#20707. @tsullivan does this only happen when upgrading from 2.4.4 to 5.3.1? In the short term we should definitely stop swallowing the error, long term it won't be a problem when we switch to the field_caps API. |
So we already handle most errors, but this is a shard failure so it doesn't fail the entire request. I'm wondering if we already have a common way of handling this. @spalger does the ES client handle shard failures at all? Or are we best off extracting them in the success handler? |
As a workaround users may find this community member's script useful to temporarily mark all fields as searchable and aggregatable. |
The es client doesn't do anything special with shard failures. The courier automatically warns when any number of shards fails, but that's obviously not applicable to all requests |
…11379) (#203396) ## Summary [Internal link](elastic/security-team#10820) to the feature details These changes add sorting functionality to the migration rules table. It is possible to sort migration rules by next columns: `Updated`, `Name`, `Status`, `Risk Score`, `Severity` and `Author`. ### Other changes Next fixes and adjustments were also implemented as part of this PR: * `Installed` status in migration rules table to indicate whether the rule was installed * Rules selection and installation of selected rules * Disable selection for not fully translated rules * `Author` column to show whether the translated rule matched one of the existing Elastic prebuilt rules * `Install and enable` and `Install without enabling` buttons within the migration rule details flyout
…lastic#11379) (elastic#203396) ## Summary [Internal link](elastic/security-team#10820) to the feature details These changes add sorting functionality to the migration rules table. It is possible to sort migration rules by next columns: `Updated`, `Name`, `Status`, `Risk Score`, `Severity` and `Author`. ### Other changes Next fixes and adjustments were also implemented as part of this PR: * `Installed` status in migration rules table to indicate whether the rule was installed * Rules selection and installation of selected rules * Disable selection for not fully translated rules * `Author` column to show whether the translated rule matched one of the existing Elastic prebuilt rules * `Install and enable` and `Install without enabling` buttons within the migration rule details flyout (cherry picked from commit 70a5bb3)
…table (#11379) (#203396) (#203486) # Backport This will backport the following commits from `main` to `8.x`: - [[Rules migration] Add sorting functionality to rules migration table (#11379) (#203396)](#203396) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Ievgen Sorokopud","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-09T19:21:16Z","message":"[Rules migration] Add sorting functionality to rules migration table (#11379) (#203396)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nThese changes add sorting functionality to the migration rules table. It\r\nis possible to sort migration rules by next columns: `Updated`, `Name`,\r\n`Status`, `Risk Score`, `Severity` and `Author`.\r\n\r\n### Other changes\r\n\r\nNext fixes and adjustments were also implemented as part of this PR:\r\n* `Installed` status in migration rules table to indicate whether the\r\nrule was installed\r\n* Rules selection and installation of selected rules\r\n* Disable selection for not fully translated rules\r\n* `Author` column to show whether the translated rule matched one of the\r\nexisting Elastic prebuilt rules\r\n* `Install and enable` and `Install without enabling` buttons within the\r\nmigration rule details flyout","sha":"70a5bb33c438912b64259ea4c7a3c77c41f93f45","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat Hunting","Team: SecuritySolution","backport:prev-minor"],"title":"[Rules migration] Add sorting functionality to rules migration table (#11379)","number":203396,"url":"https://github.com/elastic/kibana/pull/203396","mergeCommit":{"message":"[Rules migration] Add sorting functionality to rules migration table (#11379) (#203396)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nThese changes add sorting functionality to the migration rules table. It\r\nis possible to sort migration rules by next columns: `Updated`, `Name`,\r\n`Status`, `Risk Score`, `Severity` and `Author`.\r\n\r\n### Other changes\r\n\r\nNext fixes and adjustments were also implemented as part of this PR:\r\n* `Installed` status in migration rules table to indicate whether the\r\nrule was installed\r\n* Rules selection and installation of selected rules\r\n* Disable selection for not fully translated rules\r\n* `Author` column to show whether the translated rule matched one of the\r\nexisting Elastic prebuilt rules\r\n* `Install and enable` and `Install without enabling` buttons within the\r\nmigration rule details flyout","sha":"70a5bb33c438912b64259ea4c7a3c77c41f93f45"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/203396","number":203396,"mergeCommit":{"message":"[Rules migration] Add sorting functionality to rules migration table (#11379) (#203396)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nThese changes add sorting functionality to the migration rules table. It\r\nis possible to sort migration rules by next columns: `Updated`, `Name`,\r\n`Status`, `Risk Score`, `Severity` and `Author`.\r\n\r\n### Other changes\r\n\r\nNext fixes and adjustments were also implemented as part of this PR:\r\n* `Installed` status in migration rules table to indicate whether the\r\nrule was installed\r\n* Rules selection and installation of selected rules\r\n* Disable selection for not fully translated rules\r\n* `Author` column to show whether the translated rule matched one of the\r\nexisting Elastic prebuilt rules\r\n* `Install and enable` and `Install without enabling` buttons within the\r\nmigration rule details flyout","sha":"70a5bb33c438912b64259ea4c7a3c77c41f93f45"}}]}] BACKPORT--> --------- Co-authored-by: Ievgen Sorokopud <[email protected]>
…lastic#11379) (elastic#203396) ## Summary [Internal link](elastic/security-team#10820) to the feature details These changes add sorting functionality to the migration rules table. It is possible to sort migration rules by next columns: `Updated`, `Name`, `Status`, `Risk Score`, `Severity` and `Author`. ### Other changes Next fixes and adjustments were also implemented as part of this PR: * `Installed` status in migration rules table to indicate whether the rule was installed * Rules selection and installation of selected rules * Disable selection for not fully translated rules * `Author` column to show whether the translated rule matched one of the existing Elastic prebuilt rules * `Install and enable` and `Install without enabling` buttons within the migration rule details flyout
…lastic#11379) (elastic#203396) ## Summary [Internal link](elastic/security-team#10820) to the feature details These changes add sorting functionality to the migration rules table. It is possible to sort migration rules by next columns: `Updated`, `Name`, `Status`, `Risk Score`, `Severity` and `Author`. ### Other changes Next fixes and adjustments were also implemented as part of this PR: * `Installed` status in migration rules table to indicate whether the rule was installed * Rules selection and installation of selected rules * Disable selection for not fully translated rules * `Author` column to show whether the translated rule matched one of the existing Elastic prebuilt rules * `Install and enable` and `Install without enabling` buttons within the migration rule details flyout
Kibana version:
5.3.1
Elasticsearch version:
5.3.1
Browser/Server OS version:
Darwin
Browser version:
Chrome
Original install method (e.g. download page, yum, from source, etc.):
.tar.gz from download
Description of the problem including expected versus actual behavior:
When there is a problem in the Elasticsearch mapping
Steps to reproduce:
I added
console.log
logging in the code and found this is thecallWithRequest
response that Kibana gets at line https://github.com/elastic/kibana/blob/5.3/src/core_plugins/kibana/server/routes/api/ingest/register_field_capabilities.js#L20:Errors in browser console (if relevant):
field_capabilities
returns nothing:Provide logs and/or server output (if relevant):
With verbose logging, no errors are logged either when trying to load Discover, or when refreshing the Index Pattern
The text was updated successfully, but these errors were encountered: