Add url format in the index pattern #3979

monicasarbu · 2017-04-10T17:31:47Z

Add the Url format in the fields.yml, in order to generate an index pattern with an Url field. The Url link is needed in order to link the beat.name to a host-specific dashboard.

Here is an example on how to make beat.name a link to a dashboard:

name: beat.name
   metricbeat.format: url
   metricbeat.params: 
       urlTemplate: "../app/kibana#/dashboard/79ffd6e0-faa0-11e6-947f-177f697178b8?_a=(query:    (query_string:(analyze_wildcard:!t,query:'beat.hostname:{{value}}'"
       labelTemplate: "{{value}}"

As a result, the host name appears as a link to the dashboard in all the tables.

andrewkroh

Really great idea!

andrewkroh · 2017-04-10T20:20:42Z

libbeat/_meta/fields.common.yml

@@ -11,6 +11,11 @@
        set in the configuration file, then that value is used. If it is not
        set, the hostname is used. To set the Beat name, use the `name`
        option in the configuration file.
+      format: url
+      params: 
+        urlTemplate: "../app/kibana#/dashboard/79ffd6e0-faa0-11e6-947f-177f697178b8?_a=(query:(query_string:(analyze_wildcard:!t,query:'beat.hostname:{{value}}')))"


Is is possible to give the dashboard a name that we can reference here that is not a UUID? I think that would be a little better to maintainability.

Would it be possible to only apply this to the metricbeat-* index pattern instead of all beats? Like other beats might want to have their own host specific dashboards.

That's a good point, Andrew. Let me see if I can do that with the current functionality.

…w dashboard

This was accidentially removed in elastic#4055

…into export_index_pattern_with_url

ruflin · 2017-04-24T09:27:17Z

libbeat/_meta/fields.common.yml

@@ -15,6 +15,11 @@
      description: >
        The hostname as returned by the operating system on which the Beat is
        running.
+      metricbeat.format: url


If possible we should try not to mix beat specific things into libbeat.

This is the only solution that I found to set a different format for beat.hostname for Metricbeat than for the other beats. Other option would be to remove it from common libbeat fields, but this is a bigger change.

Ok, lets keep it then and figure out a way in the long run how we can make this nicer. Perhaps we can somehow have it also in metricbeat in the future and merge the fields values somehow.

Do you think this will also happen to other fields then only beat.hostname or is this hostname specific?

ruflin · 2017-04-24T18:35:51Z

libbeat/template/template.go

-	} else {
+	}
+
+	if t.esVersion.major >= 5 {


This change is also part of #4079

monicasarbu · 2017-04-24T20:05:04Z

After discussing internally, we decided to close this PR in favor of a custom widget where the user can configure the Dashboard where a field should point to.

monicasarbu added libbeat review v5.4.0 needs_backport PR is waiting to be backported to other branches. labels Apr 10, 2017

monicasarbu mentioned this pull request Apr 10, 2017

Add example dashboards using the time series visual builder #3981

Closed

monicasarbu force-pushed the export_index_pattern_with_url branch from f94922a to 07949eb Compare April 10, 2017 20:15

andrewkroh reviewed Apr 10, 2017

View reviewed changes

monicasarbu added v6.0.0-alpha1 and removed v5.4.0 needs_backport PR is waiting to be backported to other branches. labels Apr 10, 2017

monicasarbu added the discuss Issue needs further discussion. label Apr 20, 2017

ruflin added the in progress Pull request is currently in progress. label Apr 21, 2017

Generate beat.name in the index pattern as a link to the host overvie…

d2f5f2d

…w dashboard

monicasarbu force-pushed the export_index_pattern_with_url branch from 07949eb to d2f5f2d Compare April 21, 2017 11:48

ruflin and others added 3 commits April 21, 2017 14:13

Set 10000 field limit also for ES v5

a579669

This was accidentially removed in elastic#4055

Merge branch 'set-fields-limit-5' of https://github.com/ruflin/beats …

242cc65

…into export_index_pattern_with_url

Add support to define a format for a field only for one Beat

fb78c38

ruflin reviewed Apr 24, 2017

View reviewed changes

libbeat/template/template.go

} else {

}

if t.esVersion.major >= 5 {

Copy link

Contributor

ruflin Apr 24, 2017

Choose a reason for hiding this comment

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

This change is also part of #4079

monicasarbu closed this Apr 24, 2017

monicasarbu deleted the export_index_pattern_with_url branch April 24, 2017 20:05

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add url format in the index pattern #3979

Add url format in the index pattern #3979

monicasarbu commented Apr 10, 2017 •

edited

Loading

andrewkroh left a comment

andrewkroh Apr 10, 2017

andrewkroh Apr 10, 2017 •

edited

Loading

monicasarbu Apr 21, 2017

ruflin Apr 24, 2017

monicasarbu Apr 24, 2017

ruflin Apr 24, 2017

ruflin Apr 24, 2017

monicasarbu commented Apr 24, 2017

Add url format in the index pattern #3979

Add url format in the index pattern #3979

Conversation

monicasarbu commented Apr 10, 2017 • edited Loading

andrewkroh left a comment

Choose a reason for hiding this comment

andrewkroh Apr 10, 2017

Choose a reason for hiding this comment

andrewkroh Apr 10, 2017 • edited Loading

Choose a reason for hiding this comment

monicasarbu Apr 21, 2017

Choose a reason for hiding this comment

ruflin Apr 24, 2017

Choose a reason for hiding this comment

monicasarbu Apr 24, 2017

Choose a reason for hiding this comment

ruflin Apr 24, 2017

Choose a reason for hiding this comment

ruflin Apr 24, 2017

Choose a reason for hiding this comment

monicasarbu commented Apr 24, 2017

monicasarbu commented Apr 10, 2017 •

edited

Loading

andrewkroh Apr 10, 2017 •

edited

Loading