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

New Index Pattern Management UI Bugs #13735

Closed
alexfrancoeur opened this issue Aug 28, 2017 · 1 comment · Fixed by #13960
Closed

New Index Pattern Management UI Bugs #13735

alexfrancoeur opened this issue Aug 28, 2017 · 1 comment · Fixed by #13960
Labels
bug Fixes for quality problems that affect the customer experience

Comments

@alexfrancoeur
Copy link

alexfrancoeur commented Aug 28, 2017

@chrisronline We seem to have uncovered a few bugs with the new UI.

The counts seem to be off for some reason

In the below example, upon entering the page or using * you see that 16 indices are available to create a matching index pattern for
screen shot 2017-08-28 at 1 02 54 pm

However, if you were to specify an index pattern somehow the number of indices increased to 20.
screen shot 2017-08-28 at 1 02 58 pm

This was consistently reproducible. Are we doing any type of caching here for that initial count? Is it possible this view could be affected if a user is ingesting data in parallel?

Include internal indices can be confusing

The current implementation of include internal indices can get a bit confusing when you start to apply it to more general index patterns. Specifically * or any index pattern that starts off similar to an internal index .*

If you go through this wizard and create an index pattern for all indices (*) with the "Include internal indices" check box unchecked, the internal indices are still included. While technically this makes sense, a user wouldn't expect them to be included because they were not in the table below. The "include internal indices" option only seems to filter on the table. So if a user expects to select all indices (but none of the internal indices) they'll be presented with timestamps and fields from internal indices as well.

I'm not sure what the best approach here would be. Ideally, this option is still off by default. I had some initial thoughts here but none are ideal

  • Move the checkbox to the table and instead change the label to "Show internal indices" or "Hide internal indices". This way the configuration is tied directly to the table
  • Provide a separate tab to the table that is specific to internal indices (also weird)
  • Always show internal indices and rename the checkbox to "exclude internal indices". Checking this option would append the necessary exclusions (mentioned below) to the index pattern

We may be leaving out additional index patterns

You can technically search for index patterns that support inclusion / exclusion rules, example of using *,-.* below:
screen shot 2017-08-28 at 12 59 44 pm

This doesn't seem possible in today's index pattern management page either. However, I'm not sure if we decided not to support these types of index patterns in Kibana or an oversight.
screen shot 2017-08-28 at 1 40 32 pm

@epixa or @skearns64 may be able to provide some insight here.

@alexfrancoeur alexfrancoeur added :Management bug Fixes for quality problems that affect the customer experience labels Aug 28, 2017
@chrisronline
Copy link
Contributor

Thanks for bringing these up @alexfrancoeur! I'll take a look this week and get back to you on your questions.

chrisronline added a commit to chrisronline/kibana that referenced this issue Sep 12, 2017
- Ensure consistent counts by searching for twice as many as we show in the UI, this should avoid count issues due to system indices filtering (elastic#13735)
chrisronline added a commit that referenced this issue Oct 6, 2017
* Ensure we allow wildcards, #13956

* - Only search a single cluster by default (#13689)
- Ensure consistent counts by searching for twice as many as we show in the UI, this should avoid count issues due to system indices filtering (#13735)

* PR feedback

* Just return it

* Add representative sample copy

* PR feedback
chrisronline added a commit that referenced this issue Oct 6, 2017
* Ensure we allow wildcards, #13956

* - Only search a single cluster by default (#13689)
- Ensure consistent counts by searching for twice as many as we show in the UI, this should avoid count issues due to system indices filtering (#13735)

* PR feedback

* Just return it

* Add representative sample copy

* PR feedback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants