Skip to content
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

Remove index lifecycle phase filter from Index Management #45486

Merged
merged 1 commit into from
Sep 12, 2019

Conversation

cjcenizal
Copy link
Contributor

Fixes #45484

Per discussion in elastic/elasticsearch#46357, this filter implies that you can filter down to indices that are in a particular phase. However, the ES API doesn't currently provide the information needed to do that. Because this filter causes confusion among users, we're removing it until we can implement the desired behavior.

Before

image

After

image

@cjcenizal cjcenizal added chore Feature:Index Management Index and index templates UI Feature:ILM v8.0.0 Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more v7.5.0 labels Sep 12, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/es-ui

… ES API doesn't provide the required information for us to implement the expected behavior.
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@alisonelizabeth alisonelizabeth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code LGTM. Did not test locally.

@@ -10368,4 +10363,4 @@
"xpack.watcher.watchActions.logging.logTextIsRequiredValidationMessage": "ログテキストが必要です。",
"xpack.watcher.watcherDescription": "アラートの作成、管理、監視によりデータへの変更を検知します。"
}
}
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: add back newline

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was a result of running node scripts/i18n_check --fix so I'm inclined to leave it in. Seems like something we should change in the tool itself.

@cjcenizal cjcenizal merged commit 7506c7f into elastic:master Sep 12, 2019
@cjcenizal cjcenizal deleted the bug/remove-ilm-phase-filter branch September 12, 2019 14:58
cjcenizal added a commit to cjcenizal/kibana that referenced this pull request Sep 12, 2019
… ES API doesn't provide the required information for us to implement the expected behavior. (elastic#45486)
cjcenizal added a commit that referenced this pull request Sep 12, 2019
… ES API doesn't provide the required information for us to implement the expected behavior. (#45486) (#45529)
cjcenizal added a commit to cjcenizal/kibana that referenced this pull request Feb 11, 2020
cjcenizal added a commit that referenced this pull request Feb 21, 2020
…#58145)

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
jloleysens added a commit to jloleysens/kibana that referenced this pull request Feb 21, 2020
…-out-of-legacy

* 'master' of github.com:elastic/kibana: (109 commits)
  document difference between log record formats (elastic#57798)
  Expose elasticsearch config schema (elastic#57655)
  [ui/agg_response/tabify] update types for search/expressions/build_tabular_inspector_data.ts (elastic#58130)
  [SIEM] Cleans Cypress tests code (elastic#58134)
  fix: 🐛 make dev server Storybook builds work again (elastic#58188)
  Prevent core savedObjects plugin from being overridden (elastic#58193)
  Expose serverBasePath on client-side (elastic#58070)
  Fix legend sizing on area charts (elastic#58083)
  Drilldown plugin (elastic#58097)
  [skip-ci] Fix broken links to saved objects APIs in MIGRATION.md (elastic#58033)
  [ML] New Platform server shim: update datafeed routes (elastic#57739)
  Add flag for building static storybook site (elastic#58050)
  add monaco to kbn/ui-shared-deps and load required features for all uses (elastic#58075)
  [SIEM] Let us try out code owners for a little while and see what happens
  Add throttle param to Alerting readme (elastic#57609)
  [NP] Move ui/saved_objects to NP (elastic#57452)
  [Logs UI]  Fix column reordering in settings page (elastic#58104)
  Fix browser date format (elastic#57714)
  Add filter for ILM phase to Index Management (revert elastic#45486) (elastic#57402)
  Clarify Precision function in Timelion Kibana (elastic#58031)
  ...

# Conflicts:
#	x-pack/.i18nrc.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Feature:ILM Feature:Index Management Index and index templates UI release_note:skip Skip the PR/issue when compiling release notes Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more v7.5.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove ILM phase filter from Index Management
3 participants